https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=7085
--- Comment #17 from Cristian Constantin <const.crist@xxxxxxxxxxxxxx> 2012-04-16 12:05:47 PDT ---
(In reply to comment #16)
> Cristian, your patch fixes the symptom of this problem, but I'm not sure it
> fixes the root cause. As Jakub pointed out, conversation_set_add2() and
> conversation_set_port2() aren't doing the right thing with respect to the hash
> chains. Even with your patch, I expect there are cases where the code could
> still leak or give incorrect results.
cristian: you are right; there may be some corner cases which are not covered.
see also my comment #13.
>
> Does anyone have any other captures floating around that would also exercise
> this section of the code? I have an idea in mind, but I'm a bit leery of
> touching it with only one case to test against.
cristian: I have tried to crash it with sip traffic and tcp traffic (fuzzy
tests as well). I did not succeed.
--
Configure bugmail: https://bugs.wireshark.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching all bug changes.