Wireshark-bugs: [Wireshark-bugs] [Bug 9941] New: incorrect notion of OSPFv3 AT-bit

Date: Mon, 31 Mar 2014 14:45:44 +0000
Bug ID 9941
Summary incorrect notion of OSPFv3 AT-bit
Classification Unclassified
Product Wireshark
Version Git
Hardware All
URL http://www.iana.org/assignments/ospfv3-parameters/ospfv3-parameters.xhtml#ospfv3-parameters-1
OS All
Status UNCONFIRMED
Severity Minor
Priority Low
Component Dissection engine (libwireshark)
Assignee [email protected]
Reporter [email protected]

Build Information:
Paste the COMPLETE build information from "Help->About Wireshark", "wireshark
-v", or "tshark -v".
--
Wireshark treats bit 0x000400 of OSPFv3 options as bit "I", which does not
exist. 0x000400 is bit "AT" (authentication trailer). The source code requires
a fix along the lines below (I haven't compile-tested it!):

--- a/epan/dissectors/packet-ospf.c
+++ b/epan/dissectors/packet-ospf.c
@@ -122,7 +122,7 @@ static const value_string auth_vals[] = {
 #define OSPF_V3_OPTIONS_DC              0x20
 #define OSPF_V3_OPTIONS_AF              0x0100
 #define OSPF_V3_OPTIONS_L               0x0200
-#define OSPF_V3_OPTIONS_I               0x0400
+#define OSPF_V3_OPTIONS_AT              0x0400
 #define OSPF_V3_OPTIONS_F               0x0800

 /* Bitmask definitions for the informational capabilities bits. */
@@ -3545,8 +3545,8 @@ proto_register_ospf(void)
          { "L", "ospf.v3.options.l", FT_BOOLEAN, 24,
            TFS(&tfs_v3_options_l), OSPF_V3_OPTIONS_L, NULL, HFILL }},
         {&hf_ospf_filter[OSPFF_V3_OPTIONS_I],
-         { "I", "ospf.v3.options.i", FT_BOOLEAN, 24,
-           TFS(&tfs_v3_options_i), OSPF_V3_OPTIONS_I, NULL, HFILL }},
+         { "AT", "ospf.v3.options.at", FT_BOOLEAN, 24,
+           TFS(&tfs_v3_options_at), OSPF_V3_OPTIONS_AT, NULL, HFILL }},
         {&hf_ospf_filter[OSPFF_V3_OPTIONS_F],
          { "F", "ospf.v3.options.f", FT_BOOLEAN, 24,
            TFS(&tfs_v3_options_f), OSPF_V3_OPTIONS_F, NULL, HFILL }},

Thank you.


You are receiving this mail because:
  • You are watching all bug changes.