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

os/exec: panic in (*Cmd).Start.func2 on openbsd-amd64-68 #52801

Closed
bcmills opened this issue May 9, 2022 · 4 comments
Closed

os/exec: panic in (*Cmd).Start.func2 on openbsd-amd64-68 #52801

bcmills opened this issue May 9, 2022 · 4 comments
Labels
FrozenDueToAge NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. OS-OpenBSD
Milestone

Comments

@bcmills
Copy link
Contributor

bcmills commented May 9, 2022

https://storage.googleapis.com/go-build-log/84130882/openbsd-amd64-68_97af332c.log (a TryBot):

            > go test -v foo_test.go
            [stderr]
            panic: runtime error: invalid memory address or nil pointer dereference
            [signal SIGSEGV: segmentation violation code=0x1 addr=0x1 pc=0x55f005]
            
            goroutine 441 [running]:
            panic({0xa57620, 0xf64e30})
            	/tmp/workdir/go/src/runtime/panic.go:969 +0x3ba fp=0xc00073a778 sp=0xc00073a6b8 pc=0x437bda
            runtime.panicmem(...)
            	/tmp/workdir/go/src/runtime/panic.go:242
            runtime.sigpanic()
            	/tmp/workdir/go/src/runtime/signal_unix.go:823 +0x336 fp=0xc00073a7c8 sp=0xc00073a778 pc=0x44e396
            os/exec.(*Cmd).Start.func2()
            	/tmp/workdir/go/src/os/exec/exec.go:537 +0x25 fp=0xc00073a7e0 sp=0xc00073a7c8 pc=0x55f005
            runtime.goexit()
            	/tmp/workdir/go/src/runtime/asm_amd64.s:1594 +0x1 fp=0xc00073a7e8 sp=0xc00073a7e0 pc=0x46bba1
            created by os/exec.(*Cmd).Start
            	/tmp/workdir/go/src/os/exec/exec.go:535 +0x6be
@bcmills bcmills self-assigned this May 9, 2022
@bcmills bcmills added this to the Go1.19 milestone May 9, 2022
@bcmills
Copy link
Contributor Author

bcmills commented May 9, 2022

Marking as release-blocker until I can determine whether this is related to CL 401896.

@bcmills bcmills added NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. release-blocker labels May 9, 2022
@bcmills
Copy link
Contributor Author

bcmills commented May 9, 2022

greplogs -l -e \\s+os/exec\\.\\\(\\\*Cmd\\\)\\.Start\\.func2\\\(\\\)
2022-05-08T17:26:10-df57592/plan9-amd64-0intro
2021-12-13T16:44:13-27fc764-acc65b4/netbsd-amd64-9_0-n2d

(But neither of those matches this failure mode.)

@bcmills bcmills added the WaitingForInfo Issue is not actionable because of missing required information, which needs to be provided. label May 9, 2022
@bcmills
Copy link
Contributor Author

bcmills commented May 13, 2022

greplogs -l -e '(?ms)panic: runtime error: invalid memory address.*^os/exec\.\(\*Cmd\)\.Start' --since=2022-05-08

(0 matching logs)

Looks to me like this was some kind of memory corruption (CC @golang/openbsd).

AFAICT it is not a systematic problem with the os/exec change.

@bcmills bcmills removed WaitingForInfo Issue is not actionable because of missing required information, which needs to be provided. release-blocker labels May 13, 2022
@bcmills bcmills modified the milestones: Go1.19, Backlog May 13, 2022
@bcmills bcmills removed their assignment May 13, 2022
@bcmills
Copy link
Contributor Author

bcmills commented Oct 4, 2022

Duplicate of #34988

@bcmills bcmills marked this as a duplicate of #34988 Oct 4, 2022
@bcmills bcmills closed this as not planned Won't fix, can't repro, duplicate, stale Oct 4, 2022
@golang golang locked and limited conversation to collaborators Oct 4, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
FrozenDueToAge NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. OS-OpenBSD
Projects
None yet
Development

No branches or pull requests

2 participants