Wireshark-bugs: [Wireshark-bugs] [Bug 8592] packet-dhcpv6.c, Vendor Specific Information, Cable

Date: Wed, 19 Jun 2013 14:45:58 +0000

Comment # 8 on bug 8592 from
Build Information:
rv 50005, 06/18/13, Win7
Compiled Ubuntu?  NO, BuildBot indicates Ubuntu build is failing due to issue
'check license.'
Fuzz Tested?  On Win7 YES, on Ubuntu NO.
Fuzz tested with both attached captures.


Modifications

A. Issues raised by Wireshark dev team addressed.

       1.  strings in the PKT_CCCV6_KRB_REALM and PKT_CCCV6_IETF_PROV_SRV
cases;
           extract common code into a separate function
       done

       2.  wmem_strbuf rather than a raw buffer
       done

       3.  use of tvb_tmp seems unnecessary
       removed

       4.  create a header field (hf) for each suboption type
       for TLV5, done



B. Additional enhancements or fixes since the initial submission of code.
   A second capture file has been attached that displays address lists as 
   described below.

       1.
       Some CableLabs specific options values are ordered lists of server IPv6 
       addresses, e.g. TFTP Servers.  In the PDML of such a packet, these 
       addresses are difficult to distinguish.
       Fix - The numeric order of addresses in the list is prepended to the 
       proto_item.

       2.
       The Option 23, DNS Recursive Name Server(s) value is an ordered lists of 
       IPv6 addresses.  In the PDML of such a packet, these addresses are 
       difficult to distinguish.
       Fix - The numeric order of addresses in the list is prepended to the 
       proto_item.

       3.
       CableLabs Correlation ID is by specification an unsigned int.
       Fix - changed Correlation ID output format to unsigned int.


You are receiving this mail because:
  • You are watching all bug changes.