Huge thanks to our Platinum Members Endace and LiveAction,
and our Silver Member Veeam, for supporting the Wireshark Foundation and project.

Wireshark-bugs: [Wireshark-bugs] [Bug 1755] New: PATCH : SNDCP reassembly fails / overlaps when

Date: Mon, 20 Aug 2007 17:00:43 +0000 (GMT)
http://bugs.wireshark.org/bugzilla/show_bug.cgi?id=1755

           Summary: PATCH : SNDCP reassembly fails / overlaps when same NPDU
                    number in another call
           Product: Wireshark
           Version: SVN
          Platform: PC
        OS/Version: Windows Server 2003
            Status: NEW
          Severity: Major
          Priority: Medium
         Component: Wireshark
        AssignedTo: wireshark-bugs@xxxxxxxxxxxxx
        ReportedBy: ccolin@xxxxxxxxxx


Build Information:
Build Information:
wireshark 0.99.6A3S

Copyright 1998-2007 Gerald Combs <gerald@xxxxxxxxxxxxx> and contributors.
This is free software; see the source for copying conditions. There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

Compiled with GTK+ 2.10.12, with GLib 2.12.12, with WinPcap (version unknown),
with libz 1.2.3, with libpcre 6.4, with Net-SNMP 5.4, with ADNS, with Lua 5.1,
with GnuTLS 1.6.1, with Gcrypt 1.2.3, with MIT Kerberos, with PortAudio
PortAudio V19-devel, with AirPcap.

Running on Windows Server 2003 Service Pack 2, build 3790, with WinPcap version
4.0 (packet.dll version 4.0.0.755), based on libpcap version 0.9.5, without
AirPcap.

Built using Microsoft Visual C++ 8.0 build 50727

--
*msg_reassembled_table hashtable is not used in packet-sndcp.c reassembly code.
Hence my understanding is that reassembly is not "closed" when the last segment
is seen, and if another call comes with the same N-PDU number, it is added
wrongly to the first frame. 

modified simply by replacing fragment_add_seq() with fragment_add_seq_check()
containing above mentionned hashtable as argument.


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