Wireshark-bugs: [Wireshark-bugs] [Bug 7911] fix dissection of some GSM RR IEs which include leng
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: bugzilla-daemon@xxxxxxxxxxxxx
Date: Wed, 24 Oct 2012 11:46:09 -0700 (PDT)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=7911

--- Comment #3 from Mike Morrin <wireshark@xxxxxxxxxxxxxxx> 2012-10-24 11:46:09 PDT ---
(In reply to comment #2)
> Hi,
> 
> instead of changing the IE definition from TLV to TV, why not use the len
> parameter computed by the ELEM_OPT_TLV macro? IMHO, it would look like less
> hacky.
> If the hf giving the length of the field is really needed (I do not really see
> the added value as it is already given by the ELEM_OPT_TLB macro), we could use
> a proto_tree_add_uint function call. For example:
> 
I chose this method so that the dissector matches 44.018, which includes the
length octet in the CSN.1 description.

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