Wireshark-bugs: [Wireshark-bugs] [Bug 2540] New: Wrongly use hf_snp_pkm_version_support for PKM_

Date: Wed, 14 May 2008 01:27:32 -0700 (PDT)
http://bugs.wireshark.org/bugzilla/show_bug.cgi?id=2540

           Summary: Wrongly use hf_snp_pkm_version_support for
                    PKM_ATTR_SECURITY_NEGOTIATION_PARAMETER_SUB_AUTHORIZATIO
                    N_POLICY_SUPPORT
           Product: Wireshark
           Version: SVN
          Platform: PC
        OS/Version: Windows XP
            Status: ASSIGNED
          Severity: Major
          Priority: Medium
         Component: Wireshark
        AssignedTo: wireshark-bugs@xxxxxxxxxxxxx
        ReportedBy: cfreeer@xxxxxxxxxxxx
                CC: cfreeer@xxxxxxxxxxxx


Build Information:
SVN 25292
--
Wrongly use hf_snp_pkm_version_support for
PKM_ATTR_SECURITY_NEGOTIATION_PARAMETER_SUB_AUTHORIZATION_POLICY_SUPPORT.
Should use hf_snp_auth_policy_support instead.



My patch:
--- wireshark/plugins/wimax/wimax_utils.c       2008-05-14 16:18:01.000000000
+0800
+++ ./wimax_utils.c     2008-05-14 16:21:45.000000000 +0800
@@ -2672,7 +2672,7 @@
                break;
                case
PKM_ATTR_SECURITY_NEGOTIATION_PARAMETER_SUB_AUTHORIZATION_POLICY_SUPPORT:
                        /* add TLV subtree */
-                       tlv_tree = add_tlv_subtree(&tlv_info,
ett_security_negotiation_parameters, tree, hf_snp_pkm_version_support, tvb,
offset, tlv_len, FALSE);
+                       tlv_tree = add_tlv_subtree(&tlv_info,
ett_security_negotiation_parameters, tree, hf_snp_auth_policy_support, tvb,
offset, tlv_len, FALSE);
                        proto_tree_add_item(tlv_tree,
hf_snp_auth_policy_support_bit0, tvb, offset, 1, FALSE);
                        proto_tree_add_item(tlv_tree,
hf_snp_auth_policy_support_bit1, tvb, offset, 1, FALSE);
                        proto_tree_add_item(tlv_tree,
hf_snp_auth_policy_support_bit2, tvb, offset, 1, FALSE);
[gli@spimaxcn1 ~]$


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