Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cmd/go: stack overflow caused due to typo in package name #45863

Open
sachinkumarsingh092 opened this issue Apr 29, 2021 · 0 comments
Open

cmd/go: stack overflow caused due to typo in package name #45863

sachinkumarsingh092 opened this issue Apr 29, 2021 · 0 comments

Comments

@sachinkumarsingh092
Copy link

@sachinkumarsingh092 sachinkumarsingh092 commented Apr 29, 2021

What version of Go are you using (go version)?

$ go version
go version go1.16.3 linux/amd64

Does this issue reproduce with the latest release?

Yes.

What operating system and processor architecture are you using (go env)?

go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/sachin/.cache/go-build"
GOENV="/home/sachin/.config/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GOMODCACHE="/home/sachin/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/home/sachin/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.16.3"
GCCGO="gccgo"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/home/sachin/go/src/github.com/spiffe/spire/go.mod"
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-build2078549597=/tmp/go-build -gno-record-gcc-switches"

What did you do?

The wrong package name in the go file led to stack overflow by go list.
Particularly in this commit, the datastore_health.go was accidentally made a file in the sql package though it should have been in datastore package.

What did you expect to see?

An error that possibly terminates the program without overflowing memory.

What did you see instead?

A stack overflow caused due to go list.

/cc @bcmills @heschi

@cherrymui cherrymui added this to the Go1.17 milestone Apr 30, 2021
@bcmills bcmills self-assigned this Apr 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants