Wireshark-bugs: [Wireshark-bugs] [Bug 5210] New: Add new time source for Gigamon header

Date: Mon, 13 Sep 2010 14:06:36 -0700 (PDT)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5210

           Summary: Add new time source for Gigamon header
           Product: Wireshark
           Version: SVN
          Platform: Other
        OS/Version: All
            Status: NEW
          Severity: Normal
          Priority: Low
         Component: Wireshark
        AssignedTo: wireshark-bugs@xxxxxxxxxxxxx
        ReportedBy: tieuthunhi@xxxxxxxxxxx


Created an attachment (id=5154)
 --> (https://bugs.wireshark.org/bugzilla/attachment.cgi?id=5154)
Patch for packet-gmhdr.c

Build Information:
Version 1.5.0 (SVN Rev 33467 from /trunk)

Copyright 1998-2010 Gerald Combs <gerald@xxxxxxxxxxxxx> and contributors.
This is free software; see the source for copying conditions. There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

Compiled with GTK+ 2.10.4, (64-bit) with GLib 2.12.3, with libpcap 0.9.4, with
libz 1.2.3, with POSIX capabilities (Linux), with libpcre 6.6, without SMI,
without c-ares, without ADNS, without Lua, without Python, with GnuTLS 1.4.1,
with Gcrypt 1.2.4, with MIT Kerberos, with GeoIP, without PortAudio, without
AirPcap.

Running on Linux 2.6.18-194.3.1.el5, with libpcap version 0.9.4, with libz
1.2.3, GnuTLS 1.4.1, Gcrypt 1.2.4.

Built using gcc 4.1.2 20080704 (Red Hat 4.1.2-46).

Wireshark is Open Source Software released under the GNU General Public
License.

Check the man page and http://www.wireshark.org for more information.
--
Add new time source string for 1588 and GPS for Gigamon header.

Index: epan/dissectors/packet-gmhdr.c
===================================================================
--- epan/dissectors/packet-gmhdr.c    (revision 33467)
+++ epan/dissectors/packet-gmhdr.c    (working copy)
@@ -40,12 +40,14 @@
 #define GMHDR_FTYPE_SRCPORT             2
 #define GMHDR_FTYPE_TIMESTAMP_LOCAL     3
 #define GMHDR_FTYPE_TIMESTAMP_NTP       4
-#define GMHDR_FTYPE_TIMESTAMP_EXT       5
+#define GMHDR_FTYPE_TIMESTAMP_GPS       5
+#define GMHDR_FTYPE_TIMESTAMP_1588      6

 static const value_string gmhdr_ftype_timestamp[] = {
   { GMHDR_FTYPE_TIMESTAMP_LOCAL, "Local" },
   { GMHDR_FTYPE_TIMESTAMP_NTP,   "NTP" },
-  { GMHDR_FTYPE_TIMESTAMP_EXT,   "External" },
+  { GMHDR_FTYPE_TIMESTAMP_GPS,   "GPS" },
+  { GMHDR_FTYPE_TIMESTAMP_1588,  "1588" },
   { 0,                           NULL }
 };

@@ -119,7 +121,8 @@
         break;
       case GMHDR_FTYPE_TIMESTAMP_LOCAL: 
       case GMHDR_FTYPE_TIMESTAMP_NTP:
-      case GMHDR_FTYPE_TIMESTAMP_EXT:
+      case GMHDR_FTYPE_TIMESTAMP_GPS:
+      case GMHDR_FTYPE_TIMESTAMP_1588:
         ti = proto_tree_add_item(gmhdr_tree, hf_gmhdr_timestamp, tvb, offset,
fl, FALSE);
         proto_item_append_text(ti, "; Source: %s", val_to_str(tl>>8,
gmhdr_ftype_timestamp, "Unknown")); 
         break;

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