On Apr 18, 2013, at 10:28 AM, Evan Huus <[email protected]> wrote:

> That hadn't even occurred to me; I was thinking more of the fact that
> we don't have a separate 'head' structure for reassembly chains and
> just assume certain fields are set/unset based on whether the
> structure is first in the list or not.

I just made an attempt to clean that up.

In the process, I found at least one dissector that assumes that the head 
structure and the fragment structure look the same, and even some of the 
freeing code in reassemble.c assumes that.

I dropped the attempt in question for now.

A from-scratch rewrite might called for here.

___________________________________________________________________________
Sent via:    Wireshark-dev mailing list <[email protected]>
Archives:    http://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
             mailto:[email protected]?subject=unsubscribe

Reply via email to