-
Notifications
You must be signed in to change notification settings - Fork 97
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
After a frame error, call to Subscribe results in a SIGSEGV #98
Comments
We already check for closed connection status on subscribe https://github.com/go-stomp/stomp/blob/master/conn.go#L597 |
Yeah I'm not sure how it's getting it yet. I see you already check, but something doesn't seem right. |
connect use ssl and network is not good can got this panic wait a moment afert subscribe
msg in nil, I think i need reconnect.
|
How are you connecting?
Also remember that subscription.C may return nil messages so it's the
consumer responsibility to make sure you're operating on a valid message
and it can be acked before trying to do so
…On Mon, 6 Feb 2023, 10:25 niuhuan, ***@***.***> wrote:
connect use ssl and network is not good can got this panic
wait a moment afert subscribe
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x2 addr=0x40 pc=0x1021bf13c]
goroutine 1 [running]:github.com/go-stomp/stomp/v3.(*Conn).createAckNackFrame(0x140001666c0?, 0x14000119f60?, 0x1?)
***@***.***/conn.go:723 +0x7cgithub.com/go-stomp/stomp/v3.(*Conn).Ack(0x0?, 0x1021c5e4f?)
***@***.***/conn.go:674 +0x28
main.main()
/Volumes/DATA/Projects/club-server/club-server/afdian_mq_insert/rev/main.go:26 +0xf8
—
Reply to this email directly, view it on GitHub
<#98 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAFE4G6ZUAIUZ3HSYANWNN3WWEQXRANCNFSM43TCA5KA>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
It'd be nice if it just returned an error to say the connection has been closed
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x80 pc=0x677a41]
goroutine 1 [running]:
github.com/go-stomp/stomp/v3.(*Conn).Subscribe(0x0, 0xc000025320, 0x90, 0x0, 0xc000cd1bc0, 0x4, 0x4, 0x0, 0x0, 0x0)
github.com/go-stomp/stomp/[email protected]/conn.go:595 +0x61
The text was updated successfully, but these errors were encountered: