Home page logo
/

wireshark logo Wireshark mailing list archives

Re: Filebacked-tvbuffs : GSoC'13
From: Evan Huus <eapache () gmail com>
Date: Thu, 18 Apr 2013 13:28:11 -0400

On Thu, Apr 18, 2013 at 1:11 PM, Guy Harris <guy () alum mit edu> wrote:

On Apr 18, 2013, at 9:28 AM, Evan Huus <eapache () gmail com> wrote:

- Our reassembly code is a bit of a mess anyways, as Guy's recent
commit indicates.

...and that commit doesn't include another thing I started working on (but haven't done much on yet):

The "reassembled" hash table looks up by endpoints and an ID, but there's no guarantee that there won't be *more than 
one* instance of that ID in a given flow between the two endpoints in question, so that lookup needs to take into 
account the frame number as well.

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.

(A bunch of the stuff I've been doing to the reassembly code was to fix issues that showed up when doing some 
regression testing of some changes to other code; there's a DTLS file that's especially good at finding all the 
"funny stuff" in reassembly code, in part because *every fragment* has a "reassembled" length, and hilarity ensues if 
they don't have the same values.)

Sounds like fun :P

Honestly a total from-scratch rewrite is starting to look appealing
from a code-cleanliness perspective, though it would probably still be
an inefficient use of time from an engineering standpoint.

Evan
___________________________________________________________________________
Sent via:    Wireshark-dev mailing list <wireshark-dev () wireshark org>
Archives:    http://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
             mailto:wireshark-dev-request () wireshark org?subject=unsubscribe


  By Date           By Thread  

Current thread:
[ Nmap | Sec Tools | Mailing Lists | Site News | About/Contact | Advertising | Privacy ]
AlienVault