Wireshark-bugs: [Wireshark-bugs] [Bug 4626] Dissect IEEE 802.11r and IEEE 802.11w features

Date: Tue, 30 Mar 2010 07:22:51 -0700 (PDT)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=4626

Anders Broman <anders.broman@xxxxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |anders.broman@xxxxxxxxxxxx

--- Comment #2 from Anders Broman <anders.broman@xxxxxxxxxxxx> 2010-03-30 07:22:50 PDT ---
This dissector is far from perfect but at least we can try to follow
the conventions when adding to it :-)

Could you please prefix all header fields with "hf_ieee80211_"
e.g 
 hf_tag_mobility_domain_mdid -> hf_ieee80211_tag_mobility_domain_mdid
:
+    {&hf_tag_mobility_domain_mdid,
+     {"Mobility Domain Identifier", "wlan_mgt.mobility_domain.mdid",
+      FT_UINT16, BASE_HEX, NULL, 0, NULL, HFILL }},

+     {"Mobility Domain Identifier", "ieee80211.wlan_mgt.mobility_domain.mdid",

rsn_cap_mfpr ->  hf_ieee80211_rsn_cap_mfpr

Could
+  proto_tree_add_boolean(cap_tree, rsn_cap_mfpr, tag_tvb, tag_off, 2,
+                         rsn_capab);
:
be replaced by  proto_tree_add_item() ?

Do you have a small capture to verify the changes with?

regards
Anders

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