On Apr 18, 2013, at 10:28 AM, Evan Huus <eapache@xxxxxxxxx> 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.