Home page logo

wireshark logo Wireshark mailing list archives

Re: Filebacked-tvbuffs : GSoC'13
From: Guy Harris <guy () alum mit edu>
Date: Thu, 18 Apr 2013 10:11:13 -0700

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.

(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.)
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 ]