Revert SVN #39347; There's a better way.
authorwmeier <wmeier@f5534014-38df-0310-8fa8-9805f1628bb7>
Tue, 11 Oct 2011 14:31:01 +0000 (14:31 +0000)
committerwmeier <wmeier@f5534014-38df-0310-8fa8-9805f1628bb7>
Tue, 11 Oct 2011 14:31:01 +0000 (14:31 +0000)
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@39357 f5534014-38df-0310-8fa8-9805f1628bb7

epan/dissectors/packet-mip.c

index 4bb2f021ecab96a982bcfc0730c496fcdb1d0775..c6108cae7c3a7dfab2b67b41d716b50fbc07d563 100644 (file)
@@ -459,7 +459,7 @@ dissect_mip_extensions( tvbuff_t *tvb, int offset, proto_tree *tree)
                             "Unknown Extension %u"));
     ext_tree = proto_item_add_subtree(ti, ett_mip_ext);
 
-    proto_tree_add_item(ext_tree, hf_mip_ext_type, tvb, offset, 1, ENC_BIG_ENDIAN);
+    proto_tree_add_item(ext_tree, hf_mip_ext_type, tvb, offset, 1, ext_type);
     offset++;
     if (ext_type != GEN_AUTH_EXT &&
         ext_type != PMIPv4_NON_SKIP_EXT &&
@@ -475,7 +475,8 @@ dissect_mip_extensions( tvbuff_t *tvb, int offset, proto_tree *tree)
     case FH_AUTH_EXT:
       /* All these extensions look the same.  4 byte SPI followed by a key */
       proto_tree_add_item(ext_tree, hf_mip_aext_spi, tvb, offset, 4, ENC_BIG_ENDIAN);
-      proto_tree_add_item(ext_tree, hf_mip_aext_auth, tvb, offset+4, ext_len-4, ENC_NA);
+      proto_tree_add_item(ext_tree, hf_mip_aext_auth, tvb, offset+4, ext_len-4,
+                          ENC_NA);
       break;
     case MN_NAI_EXT:
       proto_tree_add_item(ext_tree, hf_mip_next_nai, tvb, offset,
@@ -566,10 +567,10 @@ dissect_mip_extensions( tvbuff_t *tvb, int offset, proto_tree *tree)
       break;
     case PMIPv4_NON_SKIP_EXT:   /* draft-leung-mip4-proxy-mode */
       /* sub-type */
-      proto_tree_add_item(ext_tree, hf_mip_pmipv4nonskipext_stype, tvb, offset, 1, ENC_BIG_ENDIAN);
+      proto_tree_add_item(ext_tree, hf_mip_pmipv4nonskipext_stype, tvb, offset, 1, ext_subtype);
       offset++;
           /* len */
-      proto_tree_add_item(ext_tree, hf_mip_ext_len, tvb, offset, 2, ENC_BIG_ENDIAN);
+      proto_tree_add_item(ext_tree, hf_mip_ext_len, tvb, offset, 2, ext_len);
       offset+=2;
       if(ext_subtype == 1){
         /* Sub-type == 1 : PMIPv4 Per-Node Authentication Method */
@@ -583,7 +584,7 @@ dissect_mip_extensions( tvbuff_t *tvb, int offset, proto_tree *tree)
                    "PMIPv4 Sub-Type: %s",
                    val_to_str(ext_subtype, mip_pmipv4skipext_stypes, "Unknown Sub-Type %u"));
       pmipv4_tree = proto_item_add_subtree(tp, ett_mip_pmipv4_ext);
-      proto_tree_add_item(pmipv4_tree, hf_mip_pmipv4skipext_stype, tvb, offset, 1, ENC_NA);
+      proto_tree_add_item(pmipv4_tree, hf_mip_pmipv4skipext_stype, tvb, offset, 1, ext_subtype);
 
       if (ext_subtype == PMIPv4_SKIPEXT_STYPE_INTERFACE_ID) {
         proto_tree_add_item(pmipv4_tree, hf_mip_pmipv4skipext_interfaceid, tvb, offset + 1, ext_len-1, ENC_NA);
@@ -608,7 +609,7 @@ dissect_mip_extensions( tvbuff_t *tvb, int offset, proto_tree *tree)
        */
       proto_tree_add_item(ext_tree, hf_mip_cvse_reserved, tvb, offset, 1, ENC_BIG_ENDIAN);
       offset++;
-      proto_tree_add_uint(ext_tree, hf_mip_ext_len, tvb, offset, 2, ENC_BIG_ENDIAN);
+      proto_tree_add_uint(ext_tree, hf_mip_ext_len, tvb, offset, 2, ext_len);
       offset+=2;
       /* Vendor/Org ID */
       /*Vendor ID & cvse type & cvse value are included in ext_len, so do not increment offset for them here.*/