Comment # 8
on bug 8592
from Brian Otte
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.