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

Concurrent map write causes panic #3

Closed
knusbaum opened this issue Jan 27, 2021 · 1 comment
Closed

Concurrent map write causes panic #3

knusbaum opened this issue Jan 27, 2021 · 1 comment

Comments

@knusbaum
Copy link

fatal error: concurrent map writes

https://github.com/fhs/mux9p

goroutine 148 [running]:
runtime.throw(0x43855ed, 0x15)
	/Users/kyle.nusbaum/sdk/go1.14.11/src/runtime/panic.go:1116 +0x72 fp=0xc000387d78 sp=0xc000387d48 pc=0x4034892
runtime.mapassign(0x431c160, 0xc000390030, 0xc000296598, 0x464b260)
	/Users/kyle.nusbaum/sdk/go1.14.11/src/runtime/map.go:585 +0x59d fp=0xc000387df8 sp=0xc000387d78 pc=0x4010c5d
github.com/fhs/mux9p.(*Config).conninthread(0xc000076000, 0xc0003a4000)
	/Users/kyle.nusbaum/go/pkg/mod/github.com/fhs/[email protected]/mux9p.go:240 +0x20f fp=0xc000387fd0 sp=0xc000387df8 pc=0x418597f
runtime.goexit()
	/Users/kyle.nusbaum/sdk/go1.14.11/src/runtime/asm_amd64.s:1373 +0x1 fp=0xc000387fd8 sp=0xc000387fd0 pc=0x4063b81
created by github.com/fhs/mux9p.(*Config).listenthread
	/Users/kyle.nusbaum/go/pkg/mod/github.com/fhs/[email protected]/mux9p.go:203 +0x4a
@fhs fhs closed this as completed in 3ad6ed5 Feb 13, 2021
@fhs
Copy link
Owner

fhs commented Feb 13, 2021

Thanks for reporting. I think all the races are fixed now -- I had to redesign the guts of the library to fix them. I may have introduced other issues in the process.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants