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 double free for hdrs (pass ownership to message) #4628
base: master
Are you sure you want to change the base?
fix double free for hdrs (pass ownership to message) #4628
Conversation
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.
I'm fine with the changes to the comments.
Co-authored-by: Arthur O'Dwyer <arthur.j.odwyer@gmail.com>
@Quuxplusone, @arches-andy just to double check: should someone tap the merge button or will it done at some point automatically?
|
Someone should tap the merge button; but I'm not a maintainer, just an interested bystander, and I think @arches-andy is too. So... @emasab, ping! |
Fixes double free for hdrs variable, more info here: #4627
Rare case when try to send message for non-existing topic/server etc.