Wireshark-bugs: [Wireshark-bugs] [Bug 4834] Synchronize bat dissector with batadv dissector

Date: Wed, 2 Jun 2010 17:40:37 -0700 (PDT)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=4834

--- Comment #8 from Bill Meier <wmeier@xxxxxxxxxxx> 2010-06-02 20:40:36 EDT ---
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.

The "new-packet-list' Wireshark (which is the default for the development 
Wireshark) *does* call the packet-bat dissector with 'tree' == NULL and
immediately crashed when I tried to open a BATMAN file (before making the
changes attached to this bug).

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