-
Notifications
You must be signed in to change notification settings - Fork 17.9k
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/cgo: panic in main.(*Package).rewriteRef(0x1043c4d0, 0x1040a080) #8980
Comments
Dave, if you can reproduce this, please cd into os/user and run go tool cgo lookup_unix.go. Please post a comment with the crash message and attach the contents of the os/user/_cgo directory it leaves behind. Thanks. Labels changed: added release-go1.4maybe, removed release-go1.4. Status changed to Accepted. |
dfc@qnap:~/go/src/os/user$ env CC=clang go tool cgo lookup_unix.go /home/dfc/go/src/os/user/lookup_unix.go:49:10: type C.struct_passwd: undefined C type 'struct passwd' /home/dfc/go/src/os/user/lookup_unix.go:50:14: type C.struct_passwd: undefined C type 'struct passwd' /home/dfc/go/src/os/user/lookup_unix.go:52:14: type C.long: undefined C type 'long' /home/dfc/go/src/os/user/lookup_unix.go:59:13: call of non-function C.sysconf /home/dfc/go/src/os/user/lookup_unix.go:59:23: unable to find value of constant C._SC_GETPW_R_SIZE_MAX /home/dfc/go/src/os/user/lookup_unix.go:64:9: call of non-function C.malloc panic: runtime error: invalid memory address or nil pointer dereference [signal 0xb code=0x1 addr=0x0 pc=0x1d9ec] goroutine 1 [running]: main.(*Package).rewriteRef(0x1043a540, 0x10434040) /home/dfc/go/src/cmd/cgo/gcc.go:610 +0xee4 main.(*Package).Translate(0x1043a540, 0x10434040) /home/dfc/go/src/cmd/cgo/gcc.go:184 +0x188 main.main() /home/dfc/go/src/cmd/cgo/main.go:231 +0xf54 Attachments:
|
@davecheney Can you see if http://golang.org/11222 fixes the problem? It looks right to me but I have no way to test it. |
CL https://golang.org/cl/11222 mentions this issue. |
@ianlancetaylor i'll take a look, it might take some time, that machine has been turned off and packed away. |
I guess anybody with an ARM GNU/Linux system could try it. |
I'll try tonight, but I ended up uninstalling clang from all my hosts a On Thu, 18 Jun 2015 15:24 Ian Lance Taylor notifications@github.com wrote:
|
sorry, doesn't look like it's fixed
|
@davecheney Thanks for testing. That is bizarre. With the CL applied I could certainly expect other problems, but I can't understand how you are seeing the same problems. I don't suppose the system is available for ssh? |
@davecheney I bet you just tried to rebuild os/user rather than running all.bash. I see that too. The patch is to cmd/cgo, though, so you need to "go install cmd/go" before trying to rebuild os/user. |
This was running env CC=clang ./all.bash I'll try to get you SSH to this host. Pls email me a key On Fri, 19 Jun 2015 15:52 Ian Lance Taylor notifications@github.com wrote:
|
Hmmm. bradfitz got me onto an ARM GNU/Linux machine, all.bash failed without CL 11222, passed with it. I only tested the most recent version of the CL, using clang-3.5. I will send you an SSH key. |
Confirmed. Thanks Ian. On Sat, Jun 27, 2015 at 2:18 AM, Ian Lance Taylor notifications@github.com
|
The text was updated successfully, but these errors were encountered: