diff options
-rw-r--r-- | epan/dissectors/packet-ethertype.c | 1 | ||||
-rw-r--r-- | epan/etypes.h | 4 | ||||
-rw-r--r-- | plugins/m2m/Makefile.common | 3 | ||||
-rw-r--r-- | plugins/m2m/packet-m2m.c | 79 | ||||
-rw-r--r-- | plugins/m2m/wmxtypes.h | 36 |
5 files changed, 31 insertions, 92 deletions
diff --git a/epan/dissectors/packet-ethertype.c b/epan/dissectors/packet-ethertype.c index 2d938dfe46..500314381d 100644 --- a/epan/dissectors/packet-ethertype.c +++ b/epan/dissectors/packet-ethertype.c @@ -52,6 +52,7 @@ const value_string etype_vals[] = { {ETHERTYPE_XNS_IDP, "XNS Internet Datagram Protocol" }, {ETHERTYPE_X25L3, "X.25 Layer 3" }, {ETHERTYPE_ARP, "ARP" }, + {ETHERTYPE_WMX_M2M, "WiMax Mac-to-Mac" }, {ETHERTYPE_EPL_V1, "EPL_V1" }, {ETHERTYPE_REVARP, "RARP" }, {ETHERTYPE_DEC_LB, "DEC LanBridge" }, diff --git a/epan/etypes.h b/epan/etypes.h index 7eda1c7634..531d1c85be 100644 --- a/epan/etypes.h +++ b/epan/etypes.h @@ -71,6 +71,10 @@ #define ETHERTYPE_ARP 0x0806 #endif +#ifndef ETHERTYPE_WMX_M2M +#define ETHERTYPE_WMX_M2M 0x08f0 +#endif + #ifndef ETHERTYPE_VINES_IP #define ETHERTYPE_VINES_IP 0x0bad #endif diff --git a/plugins/m2m/Makefile.common b/plugins/m2m/Makefile.common index 2d776844a7..96c1e336f8 100644 --- a/plugins/m2m/Makefile.common +++ b/plugins/m2m/Makefile.common @@ -31,8 +31,7 @@ DISSECTOR_SRC = \ packet-m2m.c # corresponding headers -DISSECTOR_INCLUDES = \ - wmxtypes.h +DISSECTOR_INCLUDES = # Dissector helpers. They're included in the source files in this # directory, but they're not dissectors themselves, i.e. they're not diff --git a/plugins/m2m/packet-m2m.c b/plugins/m2m/packet-m2m.c index df50b7c116..77061ea8f5 100644 --- a/plugins/m2m/packet-m2m.c +++ b/plugins/m2m/packet-m2m.c @@ -33,15 +33,14 @@ #endif #include <string.h> -#include "moduleinfo.h" #include <string.h> #include <glib.h> #include <epan/packet.h> #include <epan/prefs.h> #include <epan/reassemble.h> +#include <epan/etypes.h> #include <plugins/wimax/wimax_tlv.h> -#include "wmxtypes.h" /* forward reference */ static void dissect_m2m(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree); @@ -162,7 +161,7 @@ static hf_register_info hf[] = { "Packet Sequence Number", "m2m.seq_number", FT_UINT16, BASE_DEC, NULL, 0x0, - "", HFILL + NULL, HFILL } }, { @@ -170,7 +169,7 @@ static hf_register_info hf[] = { "Value", "m2m.frame_number", FT_UINT24, BASE_DEC, NULL, 0x0, - "", HFILL + NULL, HFILL } }, { @@ -178,7 +177,7 @@ static hf_register_info hf[] = { "Number of TLVs in the packet", "m2m.tlv_count", FT_UINT16, BASE_DEC, NULL, 0x0, - "", HFILL + NULL, HFILL } } }; @@ -211,7 +210,7 @@ static hf_register_info hf_tlv[] = { "Type", "m2m.tlv_type", FT_UINT8, BASE_DEC, NULL, 0x0, - "", HFILL + NULL, HFILL } }, { @@ -219,7 +218,7 @@ static hf_register_info hf_tlv[] = { "Length", "m2m.tlv_len", FT_UINT8, BASE_DEC, NULL, 0x0, - "", HFILL + NULL, HFILL } }, { @@ -227,7 +226,7 @@ static hf_register_info hf_tlv[] = { "Length Size", "m2m.tlv_len_size", FT_UINT8, BASE_HEX, NULL, 0x0, - "", HFILL + NULL, HFILL } }, { @@ -235,7 +234,7 @@ static hf_register_info hf_tlv[] = { "Value (hex)", "m2m.multibyte_tlv_value", FT_BYTES, BASE_HEX, NULL, 0x0, - "", HFILL + NULL, HFILL } }, { @@ -243,7 +242,7 @@ static hf_register_info hf_tlv[] = { "Value", "m2m.protocol_vers_tlv_value", FT_UINT8, BASE_DEC, NULL, 0x0, - "", HFILL + NULL, HFILL } }, { @@ -251,7 +250,7 @@ static hf_register_info hf_tlv[] = { "Value", "m2m.burst_num_tlv_value", FT_UINT8, BASE_DEC, NULL, 0x0, - "", HFILL + NULL, HFILL } }, { @@ -259,7 +258,7 @@ static hf_register_info hf_tlv[] = { "Value", "m2m.frag_type_tlv_value", FT_UINT8, BASE_DEC, NULL, 0x0, - "", HFILL + NULL, HFILL } }, { @@ -267,7 +266,7 @@ static hf_register_info hf_tlv[] = { "Value", "m2m.frag_num_tlv_value", FT_UINT8, BASE_DEC, NULL, 0x0, - "", HFILL + NULL, HFILL } }, { @@ -275,7 +274,7 @@ static hf_register_info hf_tlv[] = { "Value (hex)", "m2m.pdu_burst_tlv_value", FT_BYTES, BASE_HEX, NULL, 0x0, - "", HFILL + NULL, HFILL } }, { @@ -283,7 +282,7 @@ static hf_register_info hf_tlv[] = { "Value (hex)", "m2m.fast_fb_tlv_value", FT_BYTES, BASE_HEX, NULL, 0x0, - "", HFILL + NULL, HFILL } }, { @@ -291,7 +290,7 @@ static hf_register_info hf_tlv[] = { "Value", "m2m.fch_burst_tlv_value", FT_BYTES, BASE_HEX, NULL, 0x0, - "", HFILL + NULL, HFILL } }, { @@ -299,7 +298,7 @@ static hf_register_info hf_tlv[] = { "Value", "m2m.cdma_code_tlv_value", FT_UINT24, BASE_DEC, NULL, 0x0, - "", HFILL + NULL, HFILL } }, { @@ -307,7 +306,7 @@ static hf_register_info hf_tlv[] = { "Value", "m2m.crc16_status_tlv_value", FT_UINT8, BASE_DEC, NULL, 0x0, - "", HFILL + NULL, HFILL } }, { @@ -315,7 +314,7 @@ static hf_register_info hf_tlv[] = { "Value", "m2m.burst_power_tlv_value", FT_UINT16, BASE_DEC, NULL, 0x0, - "", HFILL + NULL, HFILL } }, { @@ -323,7 +322,7 @@ static hf_register_info hf_tlv[] = { "Value", "m2m.burst_cinr_tlv_value", FT_UINT16, BASE_DEC, NULL, 0x0, - "", HFILL + NULL, HFILL } }, { @@ -331,7 +330,7 @@ static hf_register_info hf_tlv[] = { "Value", "m2m.preamble_tlv_value", FT_UINT16, BASE_DEC, NULL, 0x0, - "", HFILL + NULL, HFILL } }, { @@ -339,7 +338,7 @@ static hf_register_info hf_tlv[] = { "Value (hex)", "m2m.harq_ack_burst_tlv_value", FT_BYTES, BASE_HEX, NULL, 0x0, - "", HFILL + NULL, HFILL } }, { @@ -347,7 +346,7 @@ static hf_register_info hf_tlv[] = { "Value (hex)", "m2m.phy_attributes", FT_BYTES, BASE_HEX, NULL, 0x0, - "", HFILL + NULL, HFILL } }, { @@ -355,7 +354,7 @@ static hf_register_info hf_tlv[] = { "Invalid TLV (hex)", "m2m.invalid_tlv", FT_BYTES, BASE_HEX, NULL, 0x0, - "", HFILL + NULL, HFILL } } }; @@ -375,7 +374,7 @@ void proto_reg_handoff_m2m(void) if (!Initialized) { m2m_handle = create_dissector_handle(dissect_m2m, proto_m2m); - dissector_add("ethertype", WMX_M2M, m2m_handle); + dissector_add("ethertype", ETHERTYPE_WMX_M2M, m2m_handle); } } @@ -418,10 +417,7 @@ static void dissect_m2m(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree) gint tlv_frag_type = 0; gint tlv_frag_number = 0; tlv_info_t m2m_tlv_info; - gint i; gint hf = 0; - static gboolean checked_frame = FALSE; - static gint frame_col = -1; /* display the M2M protocol name */ if (check_col(pinfo->cinfo, COL_PROTOCOL)) @@ -435,7 +431,7 @@ static void dissect_m2m(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree) col_clear(pinfo->cinfo, COL_INFO); } - if (tree) + { /* we are being asked for details */ m2m_item = proto_tree_add_item(tree, proto_m2m, tvb, 0, -1, FALSE); m2m_tree = proto_item_add_subtree(m2m_item, ett_m2m); @@ -538,31 +534,6 @@ static void dissect_m2m(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree) /* add the description */ proto_tree_add_item(tlv_tree, hf_m2m_frame_number, tvb, offset, 3, FALSE); proto_item_append_text(ti, ": %d", g_frame_number); - if (!checked_frame) - { - /* See if a "Frame" column is defined */ - if (pinfo->cinfo != NULL) - { - for (i=0; i < pinfo->cinfo->num_cols; i++ ) - { - if (strcmp(pinfo->cinfo->col_title[i], "Frame") == 0) - { - frame_col = i; - } - } - } - checked_frame = TRUE; - } - if (check_col(pinfo->cinfo, COL_IF_DIR)) - { - /* Check if Frame column is present */ - if (frame_col != -1) - { - /* Display frame number in Frame column */ - - col_append_fstr(pinfo->cinfo, COL_IF_DIR, "%7u", g_frame_number); - } - } break; case TLV_FCH_BURST: diff --git a/plugins/m2m/wmxtypes.h b/plugins/m2m/wmxtypes.h deleted file mode 100644 index 91cf4378be..0000000000 --- a/plugins/m2m/wmxtypes.h +++ /dev/null @@ -1,36 +0,0 @@ -/* wmxtypes.h - * Defines WiMax packet types. - * - * Copyright (c) 2007 by Intel Corporation. - * - * Author: Lu Pan <lu.pan@intel.com> - * - * $Id$ - * - * Wireshark - Network traffic analyzer - * By Gerald Combs <gerald@wireshark.org> - * Copyright 1998 Gerald Combs - * - * - * This program is free software; you can redistribute it and/or - * modify it under the terms of the GNU General Public License - * as published by the Free Software Foundation; either version 2 - * of the License, or (at your option) any later version. - * - * This program is distributed in the hope that it will be useful, - * but WITHOUT ANY WARRANTY; without even the implied warranty of - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the - * GNU General Public License for more details. - * - * You should have received a copy of the GNU General Public License - * along with this program; if not, write to the Free Software - * Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA. - */ - -#ifndef __WMXTYPES_H__ -#define __WMXTYPES_H__ - -/* Protocol types */ -#define WMX_M2M 0x08F0 /* WiMax MAC to MAC protocol */ - -#endif /* wmxtypes.h */ |