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
Fix #812 #826
Fix #812 #826
Conversation
Signed-off-by: jaylin <jaylin@emqx.io>
dd6e274
to
0378aad
Compare
Can confirm seg fault no longer appears when connecting via paho-mqtt. aarch64 with latest nanomq head, this nng branch, and built with |
thanks, will fix ci and have a release this month! |
Signed-off-by: jaylin <jaylin@emqx.io>
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #826 +/- ##
==========================================
- Coverage 65.76% 65.64% -0.13%
==========================================
Files 129 129
Lines 37297 37302 +5
==========================================
- Hits 24530 24488 -42
- Misses 12767 12814 +47 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And should we also change the nego cb part?
src/sp/transport/mqtt/broker_tcp.c
Outdated
// duplicated with fixed_header_adaptor | ||
nni_msg_set_remaining_len(msg, len); | ||
// fixed_header_adaptor(p->rxlen, msg); | ||
nni_msg_set_remaining_len(msg, nni_msg_len(msg)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Already set at 722line
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oh sure
Signed-off-by: jaylin <jaylin@emqx.io>
Signed-off-by: jaylin <jaylin@emqx.io>
Signed-off-by: jaylin <jaylin@emqx.io>
…f get_var_integar Signed-off-by: jaylin <jaylin@emqx.io>
No description provided.