ANNOUNCEMENT: Live Wireshark University & Allegro Packets online APAC Wireshark Training Session
April 17th, 2024 | 14:30-16:00 SGT (UTC+8) | Online

Wireshark-bugs: [Wireshark-bugs] [Bug 5531] add support for ANSI C12.22 protocol

Date: Sun, 15 May 2011 08:53:44 -0700 (PDT)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5531

Jeff Morriss <jeff.morriss.ws@xxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jeff.morriss.ws@xxxxxxxxx

--- Comment #3 from Jeff Morriss <jeff.morriss.ws@xxxxxxxxx> 2011-05-15 08:53:43 PDT ---
I'm also quite unsure about this code:

~~
+  /* at this point there are two possibilities:  either the packet
+   * has been dissected already or it has not.  If it has not, then
+   * we already have a tvb full of C12.22 data.  If it has, then we
+   * are actually to levels deep and the data we seek is actually in
+   * the grandparent of the current node.
+   */
+  if ((tree->parent->finfo != NULL) && (tree->parent->parent != NULL))
+    pkt_tree = tree->parent->parent;
~~

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