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 4834] Synchronize bat dissector with batadv dissector

Date: Thu, 3 Jun 2010 08:29:55 -0700 (PDT)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=4834

--- Comment #11 from Jeff Morriss <jeff.morriss.ws@xxxxxxxxx> 2010-06-03 08:29:53 PDT ---
(In reply to comment #8)
> On a related but separate note:
> 
> I wondered why the "endless loop" problem apparently hasn't been seen up to now 
> for the batman dissector (packet-bat.c) (and why you didn't see this).
> 
> It appears that the problem doesn't happen when Wireshark is built with
> "old-packet-list" (which is the default for Wireshark 1.2 and older).
> 
> I found that when I ran the "old-packet-list" Wireshark with the current
> (unrevised) packet-bat thru a debugger that 'tree' was never NULL when the bat
> dissector was called !
> 
> ISTR something about this being the case because of coloring rules or
> something.

That has always been my understanding: things behave differently with or
without coloring rules (or with or without a display filter), though Didier
mentioned once or twice that the new packet list was somehow different here
(but I never understood that).

-- 
Configure bugmail: https://bugs.wireshark.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.