https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=7355
--- Comment #3 from Chris Maynard <christopher.maynard@xxxxxxxxx> 2012-06-11 11:02:16 PDT ---
(In reply to comment #2)
> Created attachment 8583 [details]
> Patch decoding the Hdr Ext Length field as specified in the IPv6 RFC (v2)
>
> Patch decoding the Hdr Ext Length field as specified in the IPv6 RFC.
Well, personally I don't particularly like this patch. I think Wireshark's job
is to help translate raw data into meaningful data. The raw data doesn't tell
you the actual length, and if the description of the length field is, "Length
of the XXX header in 8-octet units, not including the first 8 octets.", then in
order to compute the actual length, Wireshark adds in the 8 bytes unaccounted
for in the raw data for you so that you don't have to remember to do it
yourself or have to reference RFC2460 to see what the displayed value of the
length really means.
Maybe others have a different opinion though?
(I do like the proto_tree_add_text() -> proto_tree_add_item() change though so
the field can be filterable.)
--
Configure bugmail: https://bugs.wireshark.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching all bug changes.
