Wireshark-dev: [Wireshark-dev] broken OAM cell detection for juniper dissector

Date Prev · Date Next · Thread Prev · Thread Next
Date Index · Thread Index · Other Months · All Mailing Lists
From: Hannes Gredler <hannes@xxxxxxxxxxx>
Date: Wed, 4 Apr 2007 12:07:47 +0200
hi wireshark developers,

attached a small bugfix that uses the correct bits
for determining if the ATM payload is a OAM cell.

asking for inclusion.

tx,

/hannes

Index: packet-juniper.c
===================================================================
--- packet-juniper.c    (revision 21334)
+++ packet-juniper.c    (working copy)
@@ -1100,7 +1100,7 @@
   case JUNIPER_PIC_ATM2:
       tisub = proto_tree_add_uint64(juniper_subtree, hf_juniper_atm2_cookie, tvb, offset, 8, cookie2);
       offset += atm2_header_len;
-      if (cookie2 & 0x7000) /* OAM cell ? */
+      if (cookie2 & 0x70) /* OAM cell ? */
           next_proto = PROTO_OAM;
       break;
   default: /* should not happen */