Fix using mbuf after returning it to mbuf free list for redis mset.#461
Fix using mbuf after returning it to mbuf free list for redis mset.#461huachaohuang wants to merge 2 commits intotwitter:masterfrom huachaohuang:master
Conversation
|
I merge the bugfix in #486 , but using mbuf_length on a mbuf is not a good way after putting the mbuf to the free list. So can you create a new pr based on the master, I want to merge your code. |
| language: c | ||
| script: bash ./travis.sh | ||
|
|
||
| sudo: required |
There was a problem hiding this comment.
What is the intention of this line?
|
|
My understanding of this: twemproxy is single threaded so it doesn't cause issues in practice, but would agree in principle (e.g. if mbuf_free were to be replaced with an actuall free call as a debug compile option to debug memory leaks or use-after free) |
No description provided.