Huge thanks to our Platinum Members Endace and LiveAction,
and our Silver Member Veeam, for supporting the Wireshark Foundation and project.

Wireshark-bugs: [Wireshark-bugs] [Bug 7085] Buildbot crash output: fuzz-2012-04-14-9084.pcap

Date: Mon, 16 Apr 2012 12:05:48 -0700 (PDT)
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.