Home page logo
/

wireshark logo Wireshark mailing list archives

Re: [Wireshark-commits] rev 51337: /trunk/epan/dissectors/ /trunk/epan/dissectors/: packet-dtn.c packet-dtn.h packet-ltp.c
From: Jim Wright <James.R.Wright () Colorado EDU>
Date: Tue, 13 Aug 2013 14:58:09 -0600

It also appears that it removes reporting of Bundle Status Report, Custody Signal and Aggregate Custody Signal so that 
they all are listed as "Administrative Record Type". They are distinct and it would be good to keep them separate.

Jim



On Aug 13, 2013, at 2:10 PM, Jeff Morriss wrote:

On 08/13/13 07:37, mmann () wireshark org wrote:
http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev&revision=51337

User: mmann
Date: 2013/08/13 04:37 AM

Log:
 Some improvements to the bundle protocol:
 1. Cleanly break out Convergence Layer Protocol (TCPCL) into its own (new-style) pure dissector.
 2. Make Bundle dissection into a new style dissector and clean so that it can be called using call_disssector.
 3. Use tcp_dissect_pdus for TCPCL protocol instead of "manual" while loop.  This allows contact header to benefit 
from TCP fragmentation.
 4. Make time fields use time field types so they can be filterable.
 5. A few functions were move to remove need for static declaration at top of file.
 6. Remove pri_hdr_procflags global variable.

 Next up - making more fields filterable.

This change doesn't compile with warnings turned on:

../../../epan/dissectors/packet-dtn.c: In function 'dissect_payload_header':
../../../epan/dissectors/packet-dtn.c:1260:17: error: variable 'header_start' set but not used 
[-Werror=unused-but-set-variable]
../../../epan/dissectors/packet-dtn.c: In function 'dissect_bundle':
../../../epan/dissectors/packet-dtn.c:2259:17: error: variable 'bundle_tree' set but not used 
[-Werror=unused-but-set-variable]
cc1: all warnings being treated as errors

The first one is a simple removal but I can't immediately tell whether 
the now-unused tree should be used or not.

___________________________________________________________________________
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

___________________________________________________________________________
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