Skip to content

Ensure claimTimes is initialized for connID#2

Open
laopaoer-wallet wants to merge 1 commit intohhsw2015:mainfrom
laopaoer-wallet:main
Open

Ensure claimTimes is initialized for connID#2
laopaoer-wallet wants to merge 1 commit intohhsw2015:mainfrom
laopaoer-wallet:main

Conversation

@laopaoer-wallet
Copy link
Copy Markdown

Add nil check for claimTimes to prevent concurrent deletion.

panic: assignment to entry in nil map

goroutine 20874 [running]:
main.(*ECHPool).RegisterAndClaim(0xc000183540, {0xc00020b020, 0x24}, {0xc0002ea150, 0x17}, {0x0, 0x0}, {0x85e7f8, 0xc02c4560c0})
./ech-tunnel.go:1171 +0x35f
main.handleSOCKS5Connect({0x85e7f8, 0xc02c4560c0}, {0xc0002ea150, 0x17}, {0xc02a690420, 0x11})
./ech-tunnel.go:1857 +0x207
main.handleSOCKS5Request({0x85e7f8, 0xc02c4560c0}, {0xc02a690420, 0x11}, 0xc0001a1140)
./ech-tunnel.go:1821 +0x7f2
main.handleSOCKS5Protocol({0x85e7f8, 0xc02c4560c0}, 0xc0001a1140, {0xc02a690420, 0x11})
./ech-tunnel.go:1692 +0x425
main.handleProxyConnection({0x85e7f8, 0xc02c4560c0}, 0xc0001a1140)
./ech-tunnel.go:1623 +0x4e5
created by main.runProxyServer in goroutine 1
./ech-tunnel.go:1598 +0x766

Add nil check for claimTimes to prevent concurrent deletion.
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

Successfully merging this pull request may close these issues.

1 participant