Dissection of SSAPs is a bit weird. 0x1 would be displayed as ">Unknown (0x01)"
authorJoerg Mayer <jmayer@loplof.de>
Sat, 13 Feb 2016 13:56:08 +0000 (14:56 +0100)
committerJörg Mayer <jmayer@loplof.de>
Sat, 13 Feb 2016 14:04:17 +0000 (14:04 +0000)
on my particular trace. This duplicates the value with the C/R-bit set. Needs
a proper fix eventually.
Small indentation fix.

Change-Id: I6bf7c560b5161994b8d90d7ae70724c03c6df73b
Reviewed-on: https://code.wireshark.org/review/13926
Reviewed-by: Jörg Mayer <jmayer@loplof.de>
epan/dissectors/packet-llc.c

index dfd2b1786358e310092cfb7d0bb3a4f9726dcde4..4d6a8a69bbe50b093ea94993e516cbc2f7c11eee 100644 (file)
@@ -137,6 +137,7 @@ static dissector_handle_t data_handle;
  */
 const value_string sap_vals[] = {
        { SAP_NULL,           "NULL LSAP" },
+       { SAP_NULL+1,         "NULL LSAP" },
        { SAP_LLC_SLMGMT,     "LLC Sub-Layer Management" },
        { SAP_SNA_PATHCTRL,   "SNA Path Control" },
        { SAP_IP,             "TCP/IP" },
@@ -821,7 +822,7 @@ proto_register_llc(void)
          "LLC SAP", FT_UINT8, BASE_HEX, DISSECTOR_TABLE_NOT_ALLOW_DUPLICATE);
        xid_subdissector_table = register_dissector_table("llc.xid_dsap",
          "LLC XID SAP", FT_UINT8, BASE_HEX, DISSECTOR_TABLE_NOT_ALLOW_DUPLICATE);
-    register_capture_dissector_table("llc.dsap", "LLC");
+       register_capture_dissector_table("llc.dsap", "LLC");
 
        register_dissector("llc", dissect_llc, proto_llc);
 }