`staticcheck` doesn't work in v1.47.2

[sukki37]

Welcome

• Yes, I'm using a binary release within 2 latest major releases. Only such installations are supported.
• Yes, I've searched similar issues on GitHub and didn't find any.
• Yes, I've included all information below (version, config, etc).
• Yes, I've tried with the standalone linter if available. (https://golangci-lint.run/usage/linters/)

Description of the problem

staticcheck seems doesn’t work. With the same code and configuration, using staticcheck command directly would report many errors, but using golangci-lint there seems everything is ok.

Version of golangci-lint

$ golangci-lint --version
# Paste output here

Configuration file

$ cat .golangci.yml
# paste output here
v1.47.2

Go environment

$ go version && go env
# paste output here
go version go1.18 linux/amd64
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/root/.cache/go-build"
GOENV="/root/.config/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GOMODCACHE="/opt/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/opt/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/linux_amd64"
GOVCS=""
GOVERSION="go1.18"
GCCGO="gccgo"
GOAMD64="v1"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/dev/null"
GOWORK=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build1617367600=/tmp/go-build -gno-record-gcc-switches"

Verbose output of running

$ golangci-lint cache clean
$ golangci-lint run -v
# paste output here
golangci-lint run -v -c .golangci-lint.yml ./...
INFO [config_reader] Used config file .golangci-lint.yml 
INFO [lintersdb] Active 15 linters: [deadcode depguard errcheck exportloopref gofmt gosimple govet ineffassign prealloc rowserrcheck staticcheck structcheck typecheck unused varcheck] 
INFO [loader] Go packages loading at mode 575 (name|types_sizes|compiled_files|deps|exports_file|files|imports) took 42.402616908s 
INFO [runner/filename_unadjuster] Pre-built 2 adjustments in 123.558001ms 
INFO [linters context/goanalysis] analyzers took 2m52.987225587s with top 10 stages: buildir: 56.87517251s, gofmt: 6.060680897s, S1038: 4.752459806s, unused: 4.190751314s, inspect: 3.451040317s, directives: 2.554453153s, S1039: 1.927667183s, printf: 1.888487294s, SA1012: 1.658661914s, varcheck: 1.609130118s 
WARN [linters context] rowserrcheck is disabled because of go1.18. You can track the evolution of the go1.18 support by following the https://github.com/golangci/golangci-lint/issues/2649. 
WARN [linters context] structcheck is disabled because of go1.18. You can track the evolution of the go1.18 support by following the https://github.com/golangci/golangci-lint/issues/2649. 
INFO [runner/skip dirs] Skipped 5 issues from dir pkg/sql/plan/function/builtin/multi by pattern (^|/)builtin($|/) 
INFO [runner/skip dirs] Skipped 6 issues from dir pkg/sql/plan/function/builtin/unary by pattern (^|/)builtin($|/) 
INFO [runner/skip dirs] Skipped 2 issues from dir pkg/sql/plan/function/builtin/binary by pattern (^|/)builtin($|/) 
INFO [runner/skip dirs] Skipped 2 issues from dir pkg/common/morpc/examples/message by pattern (^|/)examples($|/) 
INFO [runner] Issues before processing: 945, after processing: 0 
INFO [runner] Processors filtering stat (out/in): skip_files: 797/797, identifier_marker: 289/289, filename_unadjuster: 797/797, autogenerated_exclude: 289/782, cgo: 797/945, exclude-rules: 0/289, path_prettifier: 797/797, skip_dirs: 782/797, exclude: 289/289 
INFO [runner] processing took 6.870061ms with stages: identifier_marker: 3.172762ms, autogenerated_exclude: 1.523984ms, path_prettifier: 1.063913ms, skip_dirs: 525.354µs, exclude-rules: 497.362µs, cgo: 46.968µs, filename_unadjuster: 28.483µs, max_per_file_from_linter: 3.206µs, max_from_linter: 3.046µs, nolint: 1.212µs, max_same_issues: 983ns, uniq_by_line: 490ns, source_code: 430ns, skip_files: 360ns, exclude: 300ns, path_shortener: 290ns, sort_results: 290ns, severity-rules: 259ns, diff: 250ns, path_prefixer: 119ns 
INFO [runner] linters took 13.30072062s with stages: goanalysis_metalinter: 13.293687318s, rowserrcheck: 15.338µs, structcheck: 11.902µs 
INFO File cache stats: 0 entries of total size 0B 
INFO Memory: 552 samples, avg is 364.2MB, max is 1913.3MB 
INFO Execution took 55.835466979s                 

Code example or link to a public repository

// add your code here
https://github.com/sukki37/matrixone/tree/sukki37-patch-1
make install-static-check-tools
make static-check

[ldez]

Hello,

the integration works well.
golangci-lint contains by default some exclusions (exclude-use-default) that will hide by default some report of staticcheck.

golangci-lint/pkg/config/issues.go

Lines 10 to 104 in 886fbd7

	var DefaultExcludePatterns = []ExcludePattern{
	{
	ID: "EXC0001",
	Pattern: "Error return value of .((os\\.)?std(out|err)\\..*|.*Close" +
	"|.*Flush|os\\.Remove(All)?|.*print(f|ln)?|os\\.(Un)?Setenv). is not checked",
	Linter: "errcheck",
	Why: "Almost all programs ignore errors on these functions and in most cases it's ok",
	},
	{
	ID: "EXC0002",
	Pattern: "(comment on exported (method|function|type|const)|" +
	"should have( a package)? comment|comment should be of the form)",
	Linter: "golint",
	Why: "Annoying issue about not having a comment. The rare codebase has such comments",
	},
	{
	ID: "EXC0003",
	Pattern: "func name will be used as test\\.Test.* by other packages, and that stutters; consider calling this",
	Linter: "golint",
	Why: "False positive when tests are defined in package 'test'",
	},
	{
	ID: "EXC0004",
	Pattern: "(possible misuse of unsafe.Pointer|should have signature)",
	Linter: "govet",
	Why: "Common false positives",
	},
	{
	ID: "EXC0005",
	Pattern: "ineffective break statement. Did you mean to break out of the outer loop",
	Linter: "staticcheck",
	Why: "Developers tend to write in C-style with an explicit 'break' in a 'switch', so it's ok to ignore",
	},
	{
	ID: "EXC0006",
	Pattern: "Use of unsafe calls should be audited",
	Linter: "gosec",
	Why: "Too many false-positives on 'unsafe' usage",
	},
	{
	ID: "EXC0007",
	Pattern: "Subprocess launch(ed with variable|ing should be audited)",
	Linter: "gosec",
	Why: "Too many false-positives for parametrized shell calls",
	},
	{
	ID: "EXC0008",
	Pattern: "(G104|G307)",
	Linter: "gosec",
	Why: "Duplicated errcheck checks",
	},
	{
	ID: "EXC0009",
	Pattern: "(Expect directory permissions to be 0750 or less|Expect file permissions to be 0600 or less)",
	Linter: "gosec",
	Why: "Too many issues in popular repos",
	},
	{
	ID: "EXC0010",
	Pattern: "Potential file inclusion via variable",
	Linter: "gosec",
	Why: "False positive is triggered by 'src, err := ioutil.ReadFile(filename)'",
	},
	{
	ID: "EXC0011",
	Pattern: "(comment on exported (method|function|type|const)|" +
	"should have( a package)? comment|comment should be of the form)",
	Linter: "stylecheck",
	Why: "Annoying issue about not having a comment. The rare codebase has such comments",
	},
	{
	ID: "EXC0012",
	Pattern: `exported (.+) should have comment( \(or a comment on this block\))? or be unexported`,
	Linter: "revive",
	Why: "Annoying issue about not having a comment. The rare codebase has such comments",
	},
	{
	ID: "EXC0013",
	Pattern: `package comment should be of the form "(.+)...`,
	Linter: "revive",
	Why: "Annoying issue about not having a comment. The rare codebase has such comments",
	},
	{
	ID: "EXC0014",
	Pattern: `comment on exported (.+) should be of the form "(.+)..."`,
	Linter: "revive",
	Why: "Annoying issue about not having a comment. The rare codebase has such comments",
	},
	{
	ID: "EXC0015",
	Pattern: `should have a package comment, unless it's in another file for this package`,
	Linter: "revive",
	Why: "Annoying issue about not having a comment. The rare codebase has such comments",
	},
	}

https://golangci-lint.run/usage/configuration/#issues-configuration