ANNOUNCEMENT: Live Wireshark University & Allegro Packets online APAC Wireshark Training Session
April 17th, 2024 | 14:30-16:00 SGT (UTC+8) | Online

Wireshark-bugs: [Wireshark-bugs] [Bug 3914] Decoding Timestamp Sequence Field in Bundle Protocol

Date: Wed, 2 Dec 2009 08:25:07 -0800 (PST)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=3914

--- Comment #6 from Bill Meier <wmeier@xxxxxxxxxxx> 2009-12-02 08:25:03 PST ---
A question re decoding SDNV values in packet-dtn.c:

Looking at the Bundle Protocol Specification 
(draft-irtf-dtnrg-bundle-spec-10.txt)
[http://tools.ietf.org/html/draft-irtf-dtnrg-bundle-spec-10#page-14]

I see the following about the SDNV encoding scheme:

   "...an 8-octet value (a 64-bit quantity with no leading zeroes) would be
   encoded in a 10-octet SDNV (one octet containing the high-order bit
   of the value padded with six leading zero bits, followed by nine
   octets containing the remaining 63 bits of the value)."

Looking at the code in evaluate_sdnv_64() in packet-dtn.c it appears to me that
the the code actually considers only 1 - 8 octet SDNVs as valid for a 64 bit
quantity  when it should allow up to 10 octets.

There appears to be a similar issue for evaluate_sdnv().

Is my conclusion correct ?

Thanks

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