checkAPIs.pl: support for new-style dissectors in check_hf_entries
[metze/wireshark/wip.git] / epan / dissectors / packet-ismp.c
index e9a78db9b241dae577156cd17d90aa676d504e04..4b91b26ef8d66f9ed19b97570913b099902893cf 100644 (file)
@@ -3,41 +3,23 @@
  * Enterasys Networks Home: http://www.enterasys.com/
  * Copyright 2003, Joshua Craig Douglas <jdouglas@enterasys.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.
+ * SPDX-License-Identifier: GPL-2.0-or-later
  */
 
-#ifdef HAVE_CONFIG_H
-# include "config.h"
-#endif
-
-#include <stdio.h>
-#include <stdlib.h>
-#include <string.h>
-
-#include <glib.h>
+#include "config.h"
 
 #include <epan/packet.h>
-#include <epan/strutil.h>
+#include <epan/expert.h>
+#include <epan/to_str.h>
 #include <epan/etypes.h>
+#include <epan/addr_resolv.h>
 
+void proto_register_ismp(void);
+void proto_reg_handoff_ismp(void);
 
 /* Initialize the protocol and registered fields */
 static int proto_ismp = -1;
@@ -48,6 +30,7 @@ static int hf_ismp_code_length = -1;
 static int hf_ismp_auth_data = -1;
 
 /* Enterasys/Cabletron Dicovery Protocol fields*/
+static int hf_ismp_edp = -1;
 static int hf_ismp_edp_version = -1;
 static int hf_ismp_edp_module_ip = -1;
 static int hf_ismp_edp_module_mac = -1;
@@ -99,6 +82,16 @@ static int hf_ismp_edp_num_neighbors = -1;
 static int hf_ismp_edp_neighbors = -1;
 static int hf_ismp_edp_num_tuples = -1;
 static int hf_ismp_edp_tuples = -1;
+/* Generated from convert_proto_tree_add_text.pl */
+static int hf_ismp_assigned_neighbor_state = -1;
+static int hf_ismp_hold_time = -1;
+static int hf_ismp_interface_name = -1;
+static int hf_ismp_tuple_length = -1;
+static int hf_ismp_neighborhood_mac_address = -1;
+static int hf_ismp_unknown_tuple_data = -1;
+static int hf_ismp_tuple_type = -1;
+static int hf_ismp_system_description = -1;
+static int hf_ismp_interface_ipx_address = -1;
 
 
 /* Initialize the subtree pointers */
@@ -110,6 +103,8 @@ static gint ett_ismp_edp_neighbors_leaf = -1;
 static gint ett_ismp_edp_tuples = -1;
 static gint ett_ismp_edp_tuples_leaf = -1;
 
+static expert_field ei_ismp_malformed = EI_INIT;
+
 /* ISMP TYPES */
 #define ISMPTYPE_EDP   2
 
@@ -130,19 +125,19 @@ static gint ett_ismp_edp_tuples_leaf = -1;
 
 
 static const value_string edp_device_types[] = {
-        { EDP_DEVICE_TYPE_SFS17,       "Network Switch running SecureFast version 1.7 or lower" },
-        { EDP_DEVICE_TYPE_SFS18,         "Network Switch running SecureFast version 1.8 or greater" },
-        { EDP_DEVICE_TYPE_ROUTER,         "Router" },
-        { EDP_DEVICE_TYPE_BRIDGE,         "Bridge" },
-        { EDP_DEVICE_TYPE_VLANMAN,         "Cabletron VLAN Manager" },
-        { EDP_DEVICE_TYPE_NTSERVER,         "Network Server (NT)" },
-        { EDP_DEVICE_TYPE_NTCLIENT,    "Network Workstation (NT)" },
-        { EDP_DEVICE_TYPE_WIN95,     "Windows95" },
-        { EDP_DEVICE_TYPE_WIN98,        "Windows98" },
-        { EDP_DEVICE_TYPE_UNIXSERVER,       "UNIX Server" },
-        { EDP_DEVICE_TYPE_UNIXCLIENT, "UNIX Workstation" },
-        { EDP_DEVICE_TYPE_ACCESSPOINT,     "Roamabout wireless access point" },
-        { 0,                    NULL },
+       { EDP_DEVICE_TYPE_SFS17,       "Network Switch running SecureFast version 1.7 or lower" },
+       { EDP_DEVICE_TYPE_SFS18,       "Network Switch running SecureFast version 1.8 or greater" },
+       { EDP_DEVICE_TYPE_ROUTER,      "Router" },
+       { EDP_DEVICE_TYPE_BRIDGE,      "Bridge" },
+       { EDP_DEVICE_TYPE_VLANMAN,     "Cabletron VLAN Manager" },
+       { EDP_DEVICE_TYPE_NTSERVER,    "Network Server (NT)" },
+       { EDP_DEVICE_TYPE_NTCLIENT,    "Network Workstation (NT)" },
+       { EDP_DEVICE_TYPE_WIN95,       "Windows95" },
+       { EDP_DEVICE_TYPE_WIN98,       "Windows98" },
+       { EDP_DEVICE_TYPE_UNIXSERVER,  "UNIX Server" },
+       { EDP_DEVICE_TYPE_UNIXCLIENT,  "UNIX Workstation" },
+       { EDP_DEVICE_TYPE_ACCESSPOINT, "Roamabout wireless access point" },
+       { 0,                    NULL },
 };
 
 
@@ -209,131 +204,151 @@ static const value_string edp_tuple_types[] =
        { 0,NULL }
 };
 
-/* Is value set? */
-static const true_false_string is_set = {
-        "set",
-        "not set"
-};
+static gchar*
+ipx_addr_to_str(const guint32 net, const guint8 *ad)
+{
+       gchar       *buf;
+       const gchar *name;
 
+       name = get_ether_name_if_known(ad);
+
+       if (name) {
+               buf = wmem_strdup_printf(wmem_packet_scope(), "%s.%s",
+                               get_ipxnet_name(wmem_packet_scope(), net),
+                               name);
+       }
+       else {
+               buf = wmem_strdup_printf(wmem_packet_scope(), "%s.%s",
+                               get_ipxnet_name(wmem_packet_scope(), net),
+                               bytestring_to_str(wmem_packet_scope(), ad, 6, '\0'));
+       }
+       return buf;
+}
 
 /* Function to dissect EDP portion of ISMP message */
 static void
 dissect_ismp_edp(tvbuff_t *tvb, packet_info *pinfo, int offset, proto_tree *ismp_tree)
 {
-       /* local variables used for EDP dissection */ 
+       /* local variables used for EDP dissection */
        int neighbors_count = 0;
        int tuples_count = 0;
-       guint16 device_type = 0; 
-       guint32 options = 0;
+       guint16 device_type = 0;
        guint16 num_neighbors = 0;
        guint16 num_tuples = 0;
        guint16 tuple_type = 0;
-       guint16 tuple_length = 0;
-       const guint8 *neighbors_ptr;
-       const guint8 *tuples_ptr;
+       guint32 tuple_length = 0;
+       gchar* ipx_addr_str;
 
        /* Set up structures needed to add the protocol subtree and manage it */
        proto_item *edp_ti;
-       proto_tree *ismp_edp_tree;
-       
-       proto_item *edp_options_ti;
-       proto_tree *edp_options_tree;
+       proto_tree *edp_tree;
 
        proto_item *edp_neighbors_ti;
        proto_tree *edp_neighbors_tree;
 
-       proto_item *edp_neighbors_leaf_ti;
        proto_tree *edp_neighbors_leaf_tree;
 
        proto_item *edp_tuples_ti;
        proto_tree *edp_tuples_tree;
 
-       proto_item *edp_tuples_leaf_ti;
        proto_tree *edp_tuples_leaf_tree;
 
-       /* add column iformation marking this as EDP (Enterasys Discover Protocol */
-       if (check_col(pinfo->cinfo, COL_PROTOCOL))
-               col_set_str(pinfo->cinfo, COL_PROTOCOL, "EDP Message");
-       if (check_col(pinfo->cinfo, COL_INFO)) 
-               col_clear(pinfo->cinfo, COL_INFO);
+       /* add column information marking this as EDP (Enterasys Discover Protocol */
+       col_set_str(pinfo->cinfo, COL_PROTOCOL, "ISMP.EDP");
+       col_clear(pinfo->cinfo, COL_INFO);
 
        /* create display subtree for EDP */
-       if (ismp_tree) {
-               edp_ti  = proto_tree_add_text(ismp_tree, tvb, offset, 0, "Enterasys Discovery Protocol");
-               ismp_edp_tree = proto_item_add_subtree(edp_ti, ett_ismp_edp);
+               edp_ti  = proto_tree_add_item(ismp_tree, hf_ismp_edp, tvb, offset, -1, ENC_NA);
+               edp_tree = proto_item_add_subtree(edp_ti, ett_ismp_edp);
 
-               
-               proto_tree_add_item(ismp_edp_tree, hf_ismp_edp_version, tvb, offset, 2, FALSE);
+               col_add_fstr(pinfo->cinfo, COL_INFO, "MIP %s, MMAC %s, ifIdx %d",
+                       tvb_ip_to_str(tvb, offset+2),
+                       tvb_ether_to_str(tvb, offset+6),
+                       tvb_get_ntohl(tvb, offset+12));
+
+               proto_tree_add_item(edp_tree, hf_ismp_edp_version, tvb, offset, 2, ENC_BIG_ENDIAN);
                offset += 2;
-               proto_tree_add_item(ismp_edp_tree, hf_ismp_edp_module_ip, tvb, offset, 4, FALSE);
+               proto_tree_add_item(edp_tree, hf_ismp_edp_module_ip, tvb, offset, 4, ENC_BIG_ENDIAN);
                offset += 4;
-               proto_tree_add_item(ismp_edp_tree, hf_ismp_edp_module_mac, tvb, offset, 6, FALSE);
+               proto_tree_add_item(edp_tree, hf_ismp_edp_module_mac, tvb, offset, 6, ENC_NA);
                offset += 6;
-               proto_tree_add_item(ismp_edp_tree, hf_ismp_edp_module_port, tvb, offset, 4, FALSE);
+               proto_tree_add_item(edp_tree, hf_ismp_edp_module_port, tvb, offset, 4, ENC_BIG_ENDIAN);
                offset += 4;
-               proto_tree_add_item(ismp_edp_tree, hf_ismp_edp_chassis_mac, tvb, offset, 6, FALSE);
+               proto_tree_add_item(edp_tree, hf_ismp_edp_chassis_mac, tvb, offset, 6, ENC_NA);
                offset += 6;
-               proto_tree_add_item(ismp_edp_tree, hf_ismp_edp_chassis_ip, tvb, offset, 4, FALSE);
+               proto_tree_add_item(edp_tree, hf_ismp_edp_chassis_ip, tvb, offset, 4, ENC_BIG_ENDIAN);
                offset += 4;
                device_type = tvb_get_ntohs(tvb, offset);
-               proto_tree_add_item(ismp_edp_tree, hf_ismp_edp_device_type, tvb, offset, 2, FALSE);
+               proto_tree_add_item(edp_tree, hf_ismp_edp_device_type, tvb, offset, 2, ENC_BIG_ENDIAN);
                offset += 2;
-               proto_tree_add_uint_format(ismp_edp_tree, hf_ismp_edp_module_rev, tvb, offset, 4, tvb_get_ntohl(tvb, offset),
-                       "Module Firmware Revision: %02x.%02x.%02x.%02x", tvb_get_guint8(tvb, offset),
+               proto_tree_add_uint_format_value(edp_tree, hf_ismp_edp_module_rev, tvb, offset, 4, tvb_get_ntohl(tvb, offset),
+                       "%02x.%02x.%02x.%02x", tvb_get_guint8(tvb, offset),
                        tvb_get_guint8(tvb, offset+1), tvb_get_guint8(tvb, offset+2), tvb_get_guint8(tvb, offset+3));
                offset += 4;
 
-               /* create display subtree for EDP options */
-               options = tvb_get_ntohl(tvb, offset);
-               edp_options_ti = proto_tree_add_uint_format(ismp_edp_tree, hf_ismp_edp_options, tvb, offset, 4,
-                       options,"Options: 0x%08x",options);
-               edp_options_tree = proto_item_add_subtree(edp_options_ti, ett_ismp_edp_options);
-
                /* depending on device_type, show the appropriate options */
                switch (device_type) {
                        case EDP_DEVICE_TYPE_SFS17:
                        case EDP_DEVICE_TYPE_SFS18:
-                               proto_tree_add_item(edp_options_tree, hf_ismp_edp_sfs_option_uplink_flood, tvb, offset, 4, FALSE);
-                               proto_tree_add_item(edp_options_tree, hf_ismp_edp_sfs_option_uplink_port, tvb, offset, 4, FALSE);
-                               proto_tree_add_item(edp_options_tree, hf_ismp_edp_sfs_option_uplink_core, tvb, offset, 4, FALSE);
-                               proto_tree_add_item(edp_options_tree, hf_ismp_edp_sfs_option_uplink_switch, tvb, offset, 4, FALSE);
-                               proto_tree_add_item(edp_options_tree, hf_ismp_edp_sfs_option_isolated, tvb, offset, 4, FALSE);
-                               proto_tree_add_item(edp_options_tree, hf_ismp_edp_sfs_option_redun, tvb, offset, 4, FALSE);
-                               proto_tree_add_item(edp_options_tree, hf_ismp_edp_sfs_option_conmsg, tvb, offset, 4, FALSE);
-                               proto_tree_add_item(edp_options_tree, hf_ismp_edp_sfs_option_calltap, tvb, offset, 4, FALSE);
-                               proto_tree_add_item(edp_options_tree, hf_ismp_edp_sfs_option_tagflood, tvb, offset, 4, FALSE);
-                               proto_tree_add_item(edp_options_tree, hf_ismp_edp_sfs_option_unused2, tvb, offset, 4, FALSE);
-                               proto_tree_add_item(edp_options_tree, hf_ismp_edp_sfs_option_resolve, tvb, offset, 4, FALSE);
-                               proto_tree_add_item(edp_options_tree, hf_ismp_edp_sfs_option_flood, tvb, offset, 4, FALSE);
-                               proto_tree_add_item(edp_options_tree, hf_ismp_edp_sfs_option_lsp, tvb, offset, 4, FALSE);
-                               proto_tree_add_item(edp_options_tree, hf_ismp_edp_sfs_option_sfssup, tvb, offset, 4, FALSE);
-                               proto_tree_add_item(edp_options_tree, hf_ismp_edp_sfs_option_unused1, tvb, offset, 4, FALSE);
+                {
+                       static const gint *options[] = {
+                               &hf_ismp_edp_sfs_option_uplink_flood,
+                               &hf_ismp_edp_sfs_option_uplink_port,
+                               &hf_ismp_edp_sfs_option_uplink_core,
+                               &hf_ismp_edp_sfs_option_uplink_switch,
+                               &hf_ismp_edp_sfs_option_isolated,
+                               &hf_ismp_edp_sfs_option_redun,
+                               &hf_ismp_edp_sfs_option_conmsg,
+                               &hf_ismp_edp_sfs_option_calltap,
+                               &hf_ismp_edp_sfs_option_tagflood,
+                               &hf_ismp_edp_sfs_option_unused2,
+                               &hf_ismp_edp_sfs_option_resolve,
+                               &hf_ismp_edp_sfs_option_flood,
+                               &hf_ismp_edp_sfs_option_lsp,
+                               &hf_ismp_edp_sfs_option_sfssup,
+                               &hf_ismp_edp_sfs_option_unused1,
+                           NULL
+                       };
+                       proto_tree_add_bitmask(edp_tree, tvb, offset, hf_ismp_edp_options, ett_ismp_edp_options, options, ENC_BIG_ENDIAN);
+                }
                                break;
                        case EDP_DEVICE_TYPE_ROUTER:
-                               proto_tree_add_item(edp_options_tree, hf_ismp_edp_rtr_option_level1, tvb, offset, 4, FALSE);
-                               proto_tree_add_item(edp_options_tree, hf_ismp_edp_rtr_option_trans, tvb, offset, 4, FALSE);
-                               proto_tree_add_item(edp_options_tree, hf_ismp_edp_rtr_option_route, tvb, offset, 4, FALSE);
-                               proto_tree_add_item(edp_options_tree, hf_ismp_edp_rtr_option_igmp_snoop, tvb, offset, 4, FALSE);
-                               proto_tree_add_item(edp_options_tree, hf_ismp_edp_rtr_option_gmrp, tvb, offset, 4, FALSE);
-                               proto_tree_add_item(edp_options_tree, hf_ismp_edp_rtr_option_gvrp, tvb, offset, 4, FALSE);
-                               proto_tree_add_item(edp_options_tree, hf_ismp_edp_rtr_option_8021q, tvb, offset, 4, FALSE);
-                               proto_tree_add_item(edp_options_tree, hf_ismp_edp_rtr_option_dvmrp, tvb, offset, 4, FALSE);
-                               proto_tree_add_item(edp_options_tree, hf_ismp_edp_rtr_option_ospf, tvb, offset, 4, FALSE);
-                               proto_tree_add_item(edp_options_tree, hf_ismp_edp_rtr_option_bgp, tvb, offset, 4, FALSE);
-                               proto_tree_add_item(edp_options_tree, hf_ismp_edp_rtr_option_rip, tvb, offset, 4, FALSE);
-                               proto_tree_add_item(edp_options_tree, hf_ismp_edp_rtr_option_igmp, tvb, offset, 4, FALSE);
-                               proto_tree_add_item(edp_options_tree, hf_ismp_edp_rtr_option_ssr, tvb, offset, 4, FALSE);
+                {
+                       static const gint *options[] = {
+                               &hf_ismp_edp_rtr_option_level1,
+                               &hf_ismp_edp_rtr_option_trans,
+                               &hf_ismp_edp_rtr_option_route,
+                               &hf_ismp_edp_rtr_option_igmp_snoop,
+                               &hf_ismp_edp_rtr_option_gmrp,
+                               &hf_ismp_edp_rtr_option_gvrp,
+                               &hf_ismp_edp_rtr_option_8021q,
+                               &hf_ismp_edp_rtr_option_dvmrp,
+                               &hf_ismp_edp_rtr_option_ospf,
+                               &hf_ismp_edp_rtr_option_bgp,
+                               &hf_ismp_edp_rtr_option_rip,
+                               &hf_ismp_edp_rtr_option_igmp,
+                               &hf_ismp_edp_rtr_option_ssr,
+                           NULL
+                       };
+
+                       proto_tree_add_bitmask(edp_tree, tvb, offset, hf_ismp_edp_options, ett_ismp_edp_options, options, ENC_BIG_ENDIAN);
+                }
                                break;
                        case EDP_DEVICE_TYPE_BRIDGE:
-                               proto_tree_add_item(edp_options_tree, hf_ismp_edp_switch_option_level1, tvb, offset, 4, FALSE);
-                               proto_tree_add_item(edp_options_tree, hf_ismp_edp_switch_option_trans, tvb, offset, 4, FALSE);
-                               proto_tree_add_item(edp_options_tree, hf_ismp_edp_switch_option_route, tvb, offset, 4, FALSE);
-                               proto_tree_add_item(edp_options_tree, hf_ismp_edp_switch_option_igmp, tvb, offset, 4, FALSE);
-                               proto_tree_add_item(edp_options_tree, hf_ismp_edp_switch_option_gmrp, tvb, offset, 4, FALSE);
-                               proto_tree_add_item(edp_options_tree, hf_ismp_edp_switch_option_gvrp, tvb, offset, 4, FALSE);
-                               proto_tree_add_item(edp_options_tree, hf_ismp_edp_switch_option_8021q, tvb, offset, 4, FALSE);
-                               break;
-                       case EDP_DEVICE_TYPE_VLANMAN:
+                {
+                       static const gint *options[] = {
+                               &hf_ismp_edp_switch_option_level1,
+                               &hf_ismp_edp_switch_option_trans,
+                               &hf_ismp_edp_switch_option_route,
+                               &hf_ismp_edp_switch_option_igmp,
+                               &hf_ismp_edp_switch_option_gmrp,
+                               &hf_ismp_edp_switch_option_gvrp,
+                               &hf_ismp_edp_switch_option_8021q,
+                           NULL
+                       };
+
+                       proto_tree_add_bitmask(edp_tree, tvb, offset, hf_ismp_edp_options, ett_ismp_edp_options, options, ENC_BIG_ENDIAN);
+                }
                                break;
                        case EDP_DEVICE_TYPE_NTSERVER:
                        case EDP_DEVICE_TYPE_NTCLIENT:
@@ -341,69 +356,62 @@ dissect_ismp_edp(tvbuff_t *tvb, packet_info *pinfo, int offset, proto_tree *ismp
                        case EDP_DEVICE_TYPE_WIN98:
                        case EDP_DEVICE_TYPE_UNIXSERVER:
                        case EDP_DEVICE_TYPE_UNIXCLIENT:
-                               proto_tree_add_item(edp_options_tree, hf_ismp_edp_end_station_option_ad, tvb, offset, 4, FALSE);
-                               proto_tree_add_item(edp_options_tree, hf_ismp_edp_end_station_option_dns, tvb, offset, 4, FALSE);
-                               proto_tree_add_item(edp_options_tree, hf_ismp_edp_end_station_option_dhcp, tvb, offset, 4, FALSE);
+                {
+                       static const gint *options[] = {
+                               &hf_ismp_edp_end_station_option_ad,
+                               &hf_ismp_edp_end_station_option_dns,
+                               &hf_ismp_edp_end_station_option_dhcp,
+                           NULL
+                       };
+
+                       proto_tree_add_bitmask(edp_tree, tvb, offset, hf_ismp_edp_options, ett_ismp_edp_options, options, ENC_BIG_ENDIAN);
+                }
                                break;
+                       case EDP_DEVICE_TYPE_VLANMAN:
                        case EDP_DEVICE_TYPE_ACCESSPOINT:
                        default:
+                       proto_tree_add_item(edp_tree, hf_ismp_edp_options, tvb, offset, 4, ENC_BIG_ENDIAN);
                                break;
                }
                offset += 4;
-               
+
                /* determine the number of neighbors and create EDP neighbors subtree */
                num_neighbors = tvb_get_ntohs(tvb, offset);
-               proto_tree_add_item(ismp_edp_tree, hf_ismp_edp_num_neighbors, tvb, offset, 2, FALSE);
+               proto_tree_add_item(edp_tree, hf_ismp_edp_num_neighbors, tvb, offset, 2, ENC_BIG_ENDIAN);
                offset += 2;
                if (num_neighbors > 0)
                {
-                       tvb_ensure_bytes_exist(tvb, offset, num_neighbors*10);
-                       if (tvb_reported_length_remaining(tvb, offset) >= (num_neighbors *10))
-                       {
-                               neighbors_ptr = tvb_get_ptr( tvb, offset, (num_neighbors*10) );
-                               edp_neighbors_ti = proto_tree_add_bytes_format(ismp_edp_tree, hf_ismp_edp_neighbors, tvb, 
-                                       offset, num_neighbors*10, "Neighbors:");
-                       }
-                       else
-                       {
-                               neighbors_ptr = tvb_get_ptr( tvb, offset, tvb_reported_length_remaining(tvb, offset) );
-                               edp_neighbors_ti = proto_tree_add_bytes_format(ismp_edp_tree, hf_ismp_edp_neighbors, tvb, 
-                                       offset, num_neighbors *10, "Neighbors:");
-                       }
+                       edp_neighbors_ti = proto_tree_add_item(edp_tree, hf_ismp_edp_neighbors, tvb,
+                                                                               offset, num_neighbors*10, ENC_NA);
                        edp_neighbors_tree = proto_item_add_subtree(edp_neighbors_ti, ett_ismp_edp_neighbors);
                        while ( neighbors_count < num_neighbors && tvb_reported_length_remaining(tvb, offset) >= 10)
                        {
-                               edp_neighbors_leaf_ti = proto_tree_add_text(edp_neighbors_tree, tvb, offset, 10,
-                                               "Neighbor%d", (neighbors_count+1));
-                               edp_neighbors_leaf_tree = proto_item_add_subtree(edp_neighbors_leaf_ti, ett_ismp_edp_neighbors_leaf);
-
-                               proto_tree_add_text(edp_neighbors_leaf_tree, tvb, offset, 6,
-                                       "MAC Address: %s", ether_to_str(tvb_get_ptr(tvb, offset, 6)));
-                               proto_tree_add_text(edp_neighbors_leaf_tree, tvb, offset, 4,
-                                       "Assigned Neighbor State 0x%04x",tvb_get_ntohl(tvb, offset));
+                               edp_neighbors_leaf_tree = proto_tree_add_subtree_format(edp_neighbors_tree, tvb, offset, 10,
+                                                                               ett_ismp_edp_neighbors_leaf, NULL, "Neighbor%d", (neighbors_count+1));
+
+                               proto_tree_add_item(edp_neighbors_leaf_tree, hf_ismp_neighborhood_mac_address, tvb, offset, 6, ENC_NA);
+                               proto_tree_add_item(edp_neighbors_leaf_tree, hf_ismp_assigned_neighbor_state, tvb, offset, 4, ENC_BIG_ENDIAN);
                                offset += 10;
                                neighbors_count++;
                        }
                        if (neighbors_count != num_neighbors)
                        {
-                               proto_tree_add_text(ismp_edp_tree, tvb, offset, tvb_reported_length_remaining(tvb, offset),
-                               "MALFORMED PACKET");
+                               proto_tree_add_expert(edp_tree, pinfo, &ei_ismp_malformed, tvb, offset, -1);
                                return;
                        }
                }
-               
+
                /* determine data remains, if so, count tuples
                   and create EDP tuples subtree */
-               if (tvb_reported_length_remaining(tvb, offset) != 0 && 
-                       tvb_reported_length_remaining(tvb, offset) >= 2) 
+               if (tvb_reported_length_remaining(tvb, offset) != 0 &&
+                       tvb_reported_length_remaining(tvb, offset) >= 2)
                {
                        num_tuples = tvb_get_ntohs(tvb, offset);
-                       proto_tree_add_item(ismp_edp_tree, hf_ismp_edp_num_tuples, tvb, offset, 2, FALSE);
+                       proto_tree_add_item(edp_tree, hf_ismp_edp_num_tuples, tvb, offset, 2, ENC_BIG_ENDIAN);
                        offset += 2;
                }
                else if (tvb_reported_length_remaining(tvb, offset) > 0) {
-                       proto_tree_add_text(ismp_edp_tree, tvb, offset, 
-                               tvb_reported_length_remaining(tvb, offset), "MALFORMED PACKET");
+                       proto_tree_add_expert(edp_tree, pinfo, &ei_ismp_malformed, tvb, offset, -1);
                        return;
                }
                else
@@ -414,86 +422,72 @@ dissect_ismp_edp(tvbuff_t *tvb, packet_info *pinfo, int offset, proto_tree *ismp
                /* start populating tuple information */
                if (num_tuples && tvb_reported_length_remaining(tvb, offset) >= 4)
                {
-                       tuples_ptr = tvb_get_ptr(tvb, offset, tvb_reported_length_remaining(tvb, offset));
-                       edp_tuples_ti = proto_tree_add_bytes_format(ismp_edp_tree, hf_ismp_edp_tuples, tvb,
-                               offset, tvb_reported_length_remaining(tvb, offset), "Tuples:");
+                       edp_tuples_ti = proto_tree_add_bytes_format(edp_tree, hf_ismp_edp_tuples, tvb,
+                               offset, -1, NULL, "Tuples");
                        edp_tuples_tree = proto_item_add_subtree(edp_tuples_ti, ett_ismp_edp_tuples);
 
                        while ( (tuples_count < num_tuples) && (tvb_reported_length_remaining(tvb, offset) >= 4) )
                        {
 
-                               tuple_length = tvb_get_ntohs(tvb, offset+2);
-                               edp_tuples_leaf_ti = proto_tree_add_text(edp_tuples_tree, tvb, offset, tuple_length,
-                                       "Tuple%d", tuples_count+1);
-
-                               edp_tuples_leaf_tree = proto_item_add_subtree(edp_tuples_leaf_ti, ett_ismp_edp_tuples_leaf);
+                               edp_tuples_leaf_tree = proto_tree_add_subtree_format(edp_tuples_tree, tvb, offset, 4,
+                                       ett_ismp_edp_tuples_leaf, NULL, "Tuple%d", tuples_count+1);
 
                                tuple_type = tvb_get_ntohs(tvb, offset);
-                               proto_tree_add_text(edp_tuples_leaf_tree, tvb, offset, 2,
-                                       "Tuple Type: %s(%d)", val_to_str( tuple_type, edp_tuple_types, "Unknown"), tuple_type );
+                               proto_tree_add_item(edp_tuples_leaf_tree, hf_ismp_tuple_type, tvb, offset, 2, ENC_BIG_ENDIAN);
                                offset += 2;
-                               proto_tree_add_text(edp_tuples_leaf_tree, tvb, offset, 2,
-                                       "Tuple Length: %d", tuple_length);
-                               tuple_length -= 4;
+                               proto_tree_add_item_ret_uint(edp_tuples_leaf_tree, hf_ismp_tuple_length, tvb, offset, 2, ENC_BIG_ENDIAN, &tuple_length);
+                               if (tuple_length < 4) {
+                                       proto_tree_add_expert(edp_tree, pinfo, &ei_ismp_malformed, tvb, offset, 2);
+                                       return;
+                               }
                                offset += 2;
-                               
-                               if (tvb_reported_length_remaining(tvb, offset) >= tuple_length)
+                               proto_item_set_len(edp_tuples_leaf_tree, tuple_length);
+                               tuple_length -= 4;
+
+                               if ((guint)tvb_reported_length_remaining(tvb, offset) >= tuple_length)
                                {
-                                       tvb_ensure_bytes_exist(tvb, offset, tuple_length);
                                        switch (tuple_type)
-                                       {                                               
+                                       {
                                                case EDP_TUPLE_HOLD:
-                                                       proto_tree_add_text(edp_tuples_leaf_tree, tvb, offset, tuple_length,
-                                                               "Hold Time = %d", tvb_get_ntohs(tvb, offset));
+                                                       proto_tree_add_item(edp_tuples_leaf_tree, hf_ismp_hold_time, tvb, offset, tuple_length, ENC_BIG_ENDIAN);
                                                        break;
                                                case EDP_TUPLE_INT_NAME:
-                                                       proto_tree_add_text(edp_tuples_leaf_tree, tvb, offset, tuple_length,
-                                                               "Interface Name = %s", tvb_format_text(tvb, offset, tuple_length));
+                                                       proto_tree_add_item(edp_tuples_leaf_tree, hf_ismp_interface_name, tvb, offset, tuple_length, ENC_NA|ENC_ASCII);
+                                                       col_append_fstr(pinfo->cinfo, COL_INFO, ", ifName %s",
+                                                               tvb_format_text(tvb, offset, tuple_length));
                                                        break;
                                                case EDP_TUPLE_SYS_DESCRIPT:
-                                                       proto_tree_add_text(edp_tuples_leaf_tree, tvb, offset, tuple_length,
-                                                               "System Description = %s", tvb_format_text(tvb, offset, tuple_length));
+                                                       proto_tree_add_item(edp_tuples_leaf_tree, hf_ismp_system_description, tvb, offset, tuple_length, ENC_NA|ENC_ASCII);
                                                        break;
                                                case EDP_TUPLE_IPX_ADDR:
-                                                       proto_tree_add_text(edp_tuples_leaf_tree, tvb, offset, tuple_length,
-                                                               "Interface IPX_address = %s", 
-                                                               ipx_addr_to_str(tvb_get_ntohl(tvb, offset), 
-                                                               tvb_get_ptr(tvb, offset+4, tuple_length-4)));
+                                                       if (tuple_length != 4+6) {
+                                                               proto_tree_add_expert(edp_tree, pinfo, &ei_ismp_malformed, tvb, offset, tuple_length);
+                                                               return;
+                                                       }
+                                                       ipx_addr_str = ipx_addr_to_str(tvb_get_ntohl(tvb, offset), tvb_get_ptr(tvb, offset+4, tuple_length-4));
+                                                       proto_tree_add_string(edp_tuples_leaf_tree, hf_ismp_interface_ipx_address ,tvb, offset, tuple_length, ipx_addr_str);
                                                        break;
                                                case EDP_TUPLE_UNKNOWN:
                                                default:
-                                                       proto_tree_add_text(edp_tuples_leaf_tree, tvb, offset, tuple_length,
-                                                               "Unknown Tuple Data %s", tvb_format_text(tvb, offset, tuple_length));
+                                                       proto_tree_add_item(edp_tuples_leaf_tree, hf_ismp_unknown_tuple_data, tvb, offset, tuple_length, ENC_NA|ENC_ASCII);
                                                        break;
-                                       }  
+                                       }
                                }
                                offset += tuple_length;
 
                                tuples_count++;
-                               tuple_type = 0;
-                               tuple_length = 0;
-                       } 
-                       if (tuples_count != num_tuples)
-                       {
-                               proto_tree_add_text(ismp_edp_tree, tvb, offset, 
-                                       tvb_reported_length_remaining(tvb, offset), "MALFORMED PACKET");
-                               return;
                        }
-                       else
-                       {
-                               return;
-                       }
-               }
-                       
-       }
-               
+                       if (tuples_count != num_tuples)
+                               proto_tree_add_expert(edp_tree, pinfo, &ei_ismp_malformed, tvb, offset, -1);
 
+                       return;
+               }
 }
 
 
 /* Code to actually dissect the packets */
-static void
-dissect_ismp(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree)
+static int
+dissect_ismp(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, void* data _U_)
 {
        int offset = 0;
        guint16 message_type = 0;
@@ -505,10 +499,8 @@ dissect_ismp(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree)
        proto_tree *ismp_tree;
 
 /* Make entries in Protocol column and Info column on summary display */
-       if (check_col(pinfo->cinfo, COL_PROTOCOL)) 
-               col_set_str(pinfo->cinfo, COL_PROTOCOL, "ISMP");
-       if (check_col(pinfo->cinfo, COL_INFO)) 
-               col_clear(pinfo->cinfo, COL_INFO);
+       col_set_str(pinfo->cinfo, COL_PROTOCOL, "ISMP");
+       col_clear(pinfo->cinfo, COL_INFO);
 
        /*
         * XXX - I've seen captures with packets that have the ISMP
@@ -519,321 +511,336 @@ dissect_ismp(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree)
         */
        if (tvb_memeql(tvb, offset, weird_stuff, sizeof weird_stuff) == 0)
                offset += 3;    /* skip the weird stuff, for now */
-       if (tree) {
-               /* create display subtree for ismp */
-               ti = proto_tree_add_item(tree, proto_ismp, tvb, offset, -1, FALSE);
-
-               ismp_tree = proto_item_add_subtree(ti, ett_ismp);
-
-               /* add an items to the subtree */
-               proto_tree_add_item(ismp_tree, hf_ismp_version, tvb, offset, 2, FALSE);
-               offset += 2;
-               message_type = tvb_get_ntohs(tvb, offset);
-               proto_tree_add_item(ismp_tree, hf_ismp_message_type, tvb, offset, 2, FALSE);
-               offset += 2;
-               proto_tree_add_item(ismp_tree, hf_ismp_seq_num, tvb, offset, 2, FALSE);
-               offset += 2;
-               code_length = tvb_get_guint8(tvb, offset);
-               proto_tree_add_item(ismp_tree, hf_ismp_code_length, tvb, offset, 1, FALSE);
-               offset += 1;
-               proto_tree_add_item(ismp_tree, hf_ismp_auth_data, tvb, offset, code_length, FALSE);
-               offset += code_length;
-               
-               /* if Enterasys Discover Protocol, dissect it */
-               if(message_type == ISMPTYPE_EDP)
-                       dissect_ismp_edp(tvb, pinfo, offset, ismp_tree);
-       }
 
+       /* create display subtree for ismp */
+       ti = proto_tree_add_item(tree, proto_ismp, tvb, offset, -1, ENC_NA);
+       ismp_tree = proto_item_add_subtree(ti, ett_ismp);
+
+       /* add an items to the subtree */
+       proto_tree_add_item(ismp_tree, hf_ismp_version, tvb, offset, 2, ENC_BIG_ENDIAN);
+       offset += 2;
+       message_type = tvb_get_ntohs(tvb, offset);
+       proto_tree_add_item(ismp_tree, hf_ismp_message_type, tvb, offset, 2, ENC_BIG_ENDIAN);
+       offset += 2;
+       proto_tree_add_item(ismp_tree, hf_ismp_seq_num, tvb, offset, 2, ENC_BIG_ENDIAN);
+       offset += 2;
+       code_length = tvb_get_guint8(tvb, offset);
+       proto_tree_add_item(ismp_tree, hf_ismp_code_length, tvb, offset, 1, ENC_BIG_ENDIAN);
+       offset += 1;
+       proto_tree_add_item(ismp_tree, hf_ismp_auth_data, tvb, offset, code_length, ENC_NA);
+       offset += code_length;
+
+       /* if Enterasys Discover Protocol, dissect it */
+       if(message_type == ISMPTYPE_EDP)
+               dissect_ismp_edp(tvb, pinfo, offset, tree);
+
+       return tvb_captured_length(tvb);
 }
 
 
 /* Register this protocol with Wireshark */
 void
 proto_register_ismp(void)
-{                 
+{
 
 /* Setup list of header fields  See Section 1.6.1 for details*/
        static hf_register_info hf[] = {
                { &hf_ismp_version,
                        { "Version",           "ismp.version",
-                       FT_UINT16, BASE_DEC, NULL, 0x0,          
-                       "", HFILL }
+                       FT_UINT16, BASE_DEC, NULL, 0x0,
+                       NULL, HFILL }
                },
                { &hf_ismp_message_type,
                        { "Message Type", "ismp.msgtype",
                        FT_UINT16, BASE_DEC, NULL, 0x0,
-                       "", HFILL }
+                       NULL, HFILL }
                },
                { &hf_ismp_seq_num,
                        { "Sequence Number", "ismp.seqnum",
                        FT_UINT16, BASE_DEC, NULL, 0x0,
-                       "", HFILL }
+                       NULL, HFILL }
                },
                { &hf_ismp_code_length,
                        { "Auth Code Length", "ismp.codelen",
                        FT_UINT8, BASE_DEC, NULL, 0x0,
-                       "", HFILL }
-               }, 
+                       NULL, HFILL }
+               },
                { &hf_ismp_auth_data,
                        { "Auth Data", "ismp.authdata",
-                       FT_BYTES, BASE_HEX, NULL, 0x0,
-                       "", HFILL }
+                       FT_BYTES, BASE_NONE, NULL, 0x0,
+                       NULL, HFILL }
+               },
+               { &hf_ismp_edp,
+                       { "EDP", "ismp.edp",
+                       FT_PROTOCOL, BASE_NONE, NULL, 0x0,
+                       "Enterasys Discovery Protocol", HFILL }
                },
                { &hf_ismp_edp_version,
                        { "Version", "ismp.edp.version",
                        FT_UINT16, BASE_DEC, NULL, 0x0,
-                       "", HFILL }
+                       NULL, HFILL }
                },
                { &hf_ismp_edp_module_ip,
                        { "Module IP Address", "ismp.edp.modip",
                        FT_IPv4, BASE_NONE, NULL, 0x0,
-                       "", HFILL }
+                       NULL, HFILL }
                },
                { &hf_ismp_edp_module_mac,
                        { "Module MAC Address", "ismp.edp.modmac",
                        FT_ETHER, BASE_NONE, NULL, 0x0,
-                       "", HFILL }
+                       NULL, HFILL }
                },
                { &hf_ismp_edp_module_port,
                        { "Module Port (ifIndex num)", "ismp.edp.modport",
                        FT_UINT32, BASE_DEC, NULL, 0x0,
-                       "", HFILL }
+                       NULL, HFILL }
                },
                { &hf_ismp_edp_chassis_mac,
                        { "Chassis MAC Address", "ismp.edp.chassismac",
                        FT_ETHER, BASE_NONE, NULL, 0x0,
-                       "", HFILL }
+                       NULL, HFILL }
                },
                { &hf_ismp_edp_chassis_ip,
                        { "Chassis IP Address", "ismp.edp.chassisip",
                        FT_IPv4, BASE_NONE, NULL, 0x0,
-                       "", HFILL }
+                       NULL, HFILL }
                },
                { &hf_ismp_edp_device_type,
                        { "Device Type", "ismp.edp.devtype",
-                       FT_UINT16, BASE_DEC, edp_device_types, 0x0,
-                       "", HFILL }
+                       FT_UINT16, BASE_DEC, VALS(edp_device_types), 0x0,
+                       NULL, HFILL }
                },
                { &hf_ismp_edp_module_rev,
                        { "Module Firmware Revision", "ismp.edp.rev",
                        FT_UINT32, BASE_DEC, NULL, 0x0,
-                       "", HFILL }
+                       NULL, HFILL }
                },
                { &hf_ismp_edp_options,
-                       { "Device Options", "ismp.edp.opts",
-                       FT_UINT32, BASE_DEC, NULL, 0x0,
-                       "", HFILL }
+                       { "Device Options", "ismp.edp.options",
+                       FT_UINT32, BASE_HEX, NULL, 0x0,
+                       NULL, HFILL }
                },
                { &hf_ismp_edp_sfs_option_unused1,
-                       { "Unused", "ismp.edp.opts",
-                       FT_BOOLEAN, 32, TFS(&is_set), EDP_SFS_OPTION_UNUSED1,
-                       "", HFILL }
+                       { "Unused", "ismp.edp.sfs_option_unused1",
+                       FT_BOOLEAN, 32, TFS(&tfs_set_notset), EDP_SFS_OPTION_UNUSED1,
+                       NULL, HFILL }
                },
                { &hf_ismp_edp_sfs_option_sfssup,
-                       { "SFS Support", "ismp.edp.opts",
-                       FT_BOOLEAN, 32, TFS(&is_set), EDP_SFS_OPTION_SFSSUP,
-                       "", HFILL }
+                       { "SFS Support", "ismp.edp.sfs_option_sfssup",
+                       FT_BOOLEAN, 32, TFS(&tfs_set_notset), EDP_SFS_OPTION_SFSSUP,
+                       NULL, HFILL }
                },
                { &hf_ismp_edp_sfs_option_lsp,
-                       { "LSP Support", "ismp.edp.opts",
-                       FT_BOOLEAN, 32, TFS(&is_set), EDP_SFS_OPTION_LSP,
-                       "", HFILL }
+                       { "LSP Support", "ismp.edp.sfs_option_lsp",
+                       FT_BOOLEAN, 32, TFS(&tfs_set_notset), EDP_SFS_OPTION_LSP,
+                       NULL, HFILL }
                },
                { &hf_ismp_edp_sfs_option_flood,
-                       { "Flood Path Support", "ismp.edp.opts",
-                       FT_BOOLEAN, 32, TFS(&is_set), EDP_SFS_OPTION_FLOOD,
-                       "", HFILL }
+                       { "Flood Path Support", "ismp.edp.sfs_option_flood",
+                       FT_BOOLEAN, 32, TFS(&tfs_set_notset), EDP_SFS_OPTION_FLOOD,
+                       NULL, HFILL }
                },
                { &hf_ismp_edp_sfs_option_resolve,
-                       { "Resolve Support", "ismp.edp.opts",
-                       FT_BOOLEAN, 32, TFS(&is_set), EDP_SFS_OPTION_RESOLVE,
-                       "", HFILL }
+                       { "Resolve Support", "ismp.edp.sfs_option_resolve",
+                       FT_BOOLEAN, 32, TFS(&tfs_set_notset), EDP_SFS_OPTION_RESOLVE,
+                       NULL, HFILL }
                },
                { &hf_ismp_edp_sfs_option_unused2,
-                       { "Unused", "ismp.edp.opts",
-                       FT_BOOLEAN, 32, TFS(&is_set), EDP_SFS_OPTION_UNUSED2,
-                       "", HFILL }
+                       { "Unused", "ismp.edp.sfs_option_unused2",
+                       FT_BOOLEAN, 32, TFS(&tfs_set_notset), EDP_SFS_OPTION_UNUSED2,
+                       NULL, HFILL }
                },
                { &hf_ismp_edp_sfs_option_tagflood,
-                       { "Tagged Flood Support", "ismp.edp.opts",
-                       FT_BOOLEAN, 32, TFS(&is_set), EDP_SFS_OPTION_TAGFLOOD,
-                       "", HFILL }
+                       { "Tagged Flood Support", "ismp.edp.sfs_option_tagflood",
+                       FT_BOOLEAN, 32, TFS(&tfs_set_notset), EDP_SFS_OPTION_TAGFLOOD,
+                       NULL, HFILL }
                },
                { &hf_ismp_edp_sfs_option_calltap,
-                       { "Call Tap Support", "ismp.edp.opts",
-                       FT_BOOLEAN, 32, TFS(&is_set), EDP_SFS_OPTION_CALLTAP,
-                       "", HFILL }
+                       { "Call Tap Support", "ismp.edp.sfs_option_calltap",
+                       FT_BOOLEAN, 32, TFS(&tfs_set_notset), EDP_SFS_OPTION_CALLTAP,
+                       NULL, HFILL }
                },
                { &hf_ismp_edp_sfs_option_conmsg,
-                       { "Connection Message Support", "ismp.edp.opts",
-                       FT_BOOLEAN, 32, TFS(&is_set), EDP_SFS_OPTION_CONMSG,
-                       "", HFILL }
+                       { "Connection Message Support", "ismp.edp.sfs_option_conmsg",
+                       FT_BOOLEAN, 32, TFS(&tfs_set_notset), EDP_SFS_OPTION_CONMSG,
+                       NULL, HFILL }
                },
                { &hf_ismp_edp_sfs_option_redun,
-                       { "Redundant Access Support", "ismp.edp.opts",
-                       FT_BOOLEAN, 32, TFS(&is_set), EDP_SFS_OPTION_REDUN,
-                       "", HFILL }
+                       { "Redundant Access Support", "ismp.edp.sfs_option_redun",
+                       FT_BOOLEAN, 32, TFS(&tfs_set_notset), EDP_SFS_OPTION_REDUN,
+                       NULL, HFILL }
                },
                { &hf_ismp_edp_sfs_option_isolated,
-                       { "Isolated Switch", "ismp.edp.opts",
-                       FT_BOOLEAN, 32, TFS(&is_set), EDP_SFS_OPTION_ISOLATED,
-                       "", HFILL }
+                       { "Isolated Switch", "ismp.edp.sfs_option_isolated",
+                       FT_BOOLEAN, 32, TFS(&tfs_set_notset), EDP_SFS_OPTION_ISOLATED,
+                       NULL, HFILL }
                },
                { &hf_ismp_edp_sfs_option_uplink_switch,
-                       { "Uplink Switch", "ismp.edp.opts",
-                       FT_BOOLEAN, 32, TFS(&is_set), EDP_SFS_OPTION_UPLINK_SWITCH,
-                       "", HFILL }
+                       { "Uplink Switch", "ismp.edp.sfs_option_uplink_switch",
+                       FT_BOOLEAN, 32, TFS(&tfs_set_notset), EDP_SFS_OPTION_UPLINK_SWITCH,
+                       NULL, HFILL }
                },
                { &hf_ismp_edp_sfs_option_uplink_core,
-                       { "Uplink Core", "ismp.edp.opts",
-                       FT_BOOLEAN, 32, TFS(&is_set), EDP_SFS_OPTION_UPLINK_CORE,
-                       "", HFILL }
+                       { "Uplink Core", "ismp.edp.sfs_option_uplink_core",
+                       FT_BOOLEAN, 32, TFS(&tfs_set_notset), EDP_SFS_OPTION_UPLINK_CORE,
+                       NULL, HFILL }
                },
                { &hf_ismp_edp_sfs_option_uplink_port,
-                       { "Uplink Port", "ismp.edp.opts",
-                       FT_BOOLEAN, 32, TFS(&is_set), EDP_SFS_OPTION_UPLINK_PORT,
-                       "", HFILL }
+                       { "Uplink Port", "ismp.edp.sfs_option_uplink_port",
+                       FT_BOOLEAN, 32, TFS(&tfs_set_notset), EDP_SFS_OPTION_UPLINK_PORT,
+                       NULL, HFILL }
                },
                { &hf_ismp_edp_sfs_option_uplink_flood,
-                       { "Uplink Flood Support", "ismp.edp.opts",
-                       FT_BOOLEAN, 32, TFS(&is_set), EDP_SFS_OPTION_UPLINK_FLOOD,
-                       "", HFILL }
+                       { "Uplink Flood Support", "ismp.edp.sfs_option_uplink_flood",
+                       FT_BOOLEAN, 32, TFS(&tfs_set_notset), EDP_SFS_OPTION_UPLINK_FLOOD,
+                       NULL, HFILL }
                },
                { &hf_ismp_edp_rtr_option_ssr,
-                       { "SSR Type Device", "ismp.edp.opts",
-                       FT_BOOLEAN, 32, TFS(&is_set), EDP_RTR_OPTION_SSR,
-                       "", HFILL }
+                       { "SSR Type Device", "ismp.edp.rtr_option_ssr",
+                       FT_BOOLEAN, 32, TFS(&tfs_set_notset), EDP_RTR_OPTION_SSR,
+                       NULL, HFILL }
                },
                { &hf_ismp_edp_rtr_option_igmp,
-                       { "IGMP Active", "ismp.edp.opts",
-                       FT_BOOLEAN, 32, TFS(&is_set), EDP_RTR_OPTION_IGMP,
-                       "", HFILL }
+                       { "IGMP Active", "ismp.edp.rtr_option_igmp",
+                       FT_BOOLEAN, 32, TFS(&tfs_set_notset), EDP_RTR_OPTION_IGMP,
+                       NULL, HFILL }
                },
                { &hf_ismp_edp_rtr_option_rip,
-                       { "RIP Active", "ismp.edp.opts",
-                       FT_BOOLEAN, 32, TFS(&is_set), EDP_RTR_OPTION_RIP,
-                       "", HFILL }
+                       { "RIP Active", "ismp.edp.rtr_option_rip",
+                       FT_BOOLEAN, 32, TFS(&tfs_set_notset), EDP_RTR_OPTION_RIP,
+                       NULL, HFILL }
                },
                { &hf_ismp_edp_rtr_option_bgp,
-                       { "BGP Active", "ismp.edp.opts",
-                       FT_BOOLEAN, 32, TFS(&is_set), EDP_RTR_OPTION_BGP,
-                       "", HFILL }
+                       { "BGP Active", "ismp.edp.rtr_option_bgp",
+                       FT_BOOLEAN, 32, TFS(&tfs_set_notset), EDP_RTR_OPTION_BGP,
+                       NULL, HFILL }
                },
                { &hf_ismp_edp_rtr_option_ospf,
-                       { "OSPF Active", "ismp.edp.opts",
-                       FT_BOOLEAN, 32, TFS(&is_set), EDP_RTR_OPTION_OSPF,
-                       "", HFILL }
+                       { "OSPF Active", "ismp.edp.rtr_option_ospf",
+                       FT_BOOLEAN, 32, TFS(&tfs_set_notset), EDP_RTR_OPTION_OSPF,
+                       NULL, HFILL }
                },
                { &hf_ismp_edp_rtr_option_dvmrp,
-                       { "DVMRP Active", "ismp.edp.opts",
-                       FT_BOOLEAN, 32, TFS(&is_set), EDP_RTR_OPTION_DVMRP,
-                       "", HFILL }
+                       { "DVMRP Active", "ismp.edp.rtr_option_dvmrp",
+                       FT_BOOLEAN, 32, TFS(&tfs_set_notset), EDP_RTR_OPTION_DVMRP,
+                       NULL, HFILL }
                },
                { &hf_ismp_edp_rtr_option_8021q,
-                       { "802.1Q Support", "ismp.edp.opts",
-                       FT_BOOLEAN, 32, TFS(&is_set), EDP_RTR_OPTION_8021Q,
-                       "", HFILL }
+                       { "802.1Q Support", "ismp.edp.rtr_option_8021q",
+                       FT_BOOLEAN, 32, TFS(&tfs_set_notset), EDP_RTR_OPTION_8021Q,
+                       NULL, HFILL }
                },
                { &hf_ismp_edp_rtr_option_gvrp,
-                       { "GVRP Support", "ismp.edp.opts",
-                       FT_BOOLEAN, 32, TFS(&is_set), EDP_RTR_OPTION_GVRP,
-                       "", HFILL }
+                       { "GVRP Support", "ismp.edp.rtr_option_gvrp",
+                       FT_BOOLEAN, 32, TFS(&tfs_set_notset), EDP_RTR_OPTION_GVRP,
+                       NULL, HFILL }
                },
                { &hf_ismp_edp_rtr_option_gmrp,
-                       { "GMRP Support", "ismp.edp.opts",
-                       FT_BOOLEAN, 32, TFS(&is_set), EDP_RTR_OPTION_GMRP,
-                       "", HFILL }
+                       { "GMRP Support", "ismp.edp.rtr_option_gmrp",
+                       FT_BOOLEAN, 32, TFS(&tfs_set_notset), EDP_RTR_OPTION_GMRP,
+                       NULL, HFILL }
                },
                { &hf_ismp_edp_rtr_option_igmp_snoop,
-                       { "IGMP Snooping Support", "ismp.edp.opts",
-                       FT_BOOLEAN, 32, TFS(&is_set), EDP_RTR_OPTION_IGMP_SNOOP,
-                       "", HFILL }
+                       { "IGMP Snooping Support", "ismp.edp.rtr_option_igmp_snoop",
+                       FT_BOOLEAN, 32, TFS(&tfs_set_notset), EDP_RTR_OPTION_IGMP_SNOOP,
+                       NULL, HFILL }
                },
                { &hf_ismp_edp_rtr_option_route,
-                       { "Route Bridging", "ismp.edp.opts",
-                       FT_BOOLEAN, 32, TFS(&is_set), EDP_RTR_OPTION_ROUTE,
-                       "", HFILL }
+                       { "Route Bridging", "ismp.edp.rtr_option_route",
+                       FT_BOOLEAN, 32, TFS(&tfs_set_notset), EDP_RTR_OPTION_ROUTE,
+                       NULL, HFILL }
                },
                { &hf_ismp_edp_rtr_option_trans,
-                       { "Transparent Bridging", "ismp.edp.opts",
-                       FT_BOOLEAN, 32, TFS(&is_set), EDP_RTR_OPTION_TRANS,
-                       "", HFILL }
+                       { "Transparent Bridging", "ismp.edp.rtr_option_trans",
+                       FT_BOOLEAN, 32, TFS(&tfs_set_notset), EDP_RTR_OPTION_TRANS,
+                       NULL, HFILL }
                },
                { &hf_ismp_edp_rtr_option_level1,
-                       { "Level 1 Functionality", "ismp.edp.opts",
-                       FT_BOOLEAN, 32, TFS(&is_set), EDP_RTR_OPTION_LEVEL1,
-                       "", HFILL }
+                       { "Level 1 Functionality", "ismp.edp.rtr_option_level1",
+                       FT_BOOLEAN, 32, TFS(&tfs_set_notset), EDP_RTR_OPTION_LEVEL1,
+                       NULL, HFILL }
                },
                { &hf_ismp_edp_switch_option_8021q,
-                       { "802.1Q Support", "ismp.edp.opts",
-                       FT_BOOLEAN, 32, TFS(&is_set), EDP_SWITCH_OPTION_8021Q,
-                       "", HFILL }
+                       { "802.1Q Support", "ismp.edp.switch_option_8021q",
+                       FT_BOOLEAN, 32, TFS(&tfs_set_notset), EDP_SWITCH_OPTION_8021Q,
+                       NULL, HFILL }
                },
                { &hf_ismp_edp_switch_option_gvrp,
-                       { "GVRP Support", "ismp.edp.opts",
-                       FT_BOOLEAN, 32, TFS(&is_set), EDP_SWITCH_OPTION_GVRP,
-                       "", HFILL }
+                       { "GVRP Support", "ismp.edp.switch_option_gvrp",
+                       FT_BOOLEAN, 32, TFS(&tfs_set_notset), EDP_SWITCH_OPTION_GVRP,
+                       NULL, HFILL }
                },
                { &hf_ismp_edp_switch_option_gmrp,
-                       { "GMRP Support", "ismp.edp.opts",
-                       FT_BOOLEAN, 32, TFS(&is_set), EDP_SWITCH_OPTION_GMRP,
-                       "", HFILL }
+                       { "GMRP Support", "ismp.edp.switch_option_gmrp",
+                       FT_BOOLEAN, 32, TFS(&tfs_set_notset), EDP_SWITCH_OPTION_GMRP,
+                       NULL, HFILL }
                },
                { &hf_ismp_edp_switch_option_igmp,
-                       { "IGMP Snooping Support", "ismp.edp.opts",
-                       FT_BOOLEAN, 32, TFS(&is_set), EDP_SWITCH_OPTION_IGMP,
-                       "", HFILL }
+                       { "IGMP Snooping Support", "ismp.edp.switch_option_igmp",
+                       FT_BOOLEAN, 32, TFS(&tfs_set_notset), EDP_SWITCH_OPTION_IGMP,
+                       NULL, HFILL }
                },
                { &hf_ismp_edp_switch_option_route,
-                       { "Route Bridging", "ismp.edp.opts",
-                       FT_BOOLEAN, 32, TFS(&is_set), EDP_SWITCH_OPTION_ROUTE,
-                       "", HFILL }
+                       { "Route Bridging", "ismp.edp.switch_option_route",
+                       FT_BOOLEAN, 32, TFS(&tfs_set_notset), EDP_SWITCH_OPTION_ROUTE,
+                       NULL, HFILL }
                },
                { &hf_ismp_edp_switch_option_trans,
-                       { "Transparent Bridging", "ismp.edp.opts",
-                       FT_BOOLEAN, 32, TFS(&is_set), EDP_SWITCH_OPTION_TRANS,
-                       "", HFILL }
+                       { "Transparent Bridging", "ismp.edp.switch_option_trans",
+                       FT_BOOLEAN, 32, TFS(&tfs_set_notset), EDP_SWITCH_OPTION_TRANS,
+                       NULL, HFILL }
                },
                { &hf_ismp_edp_switch_option_level1,
-                       { "Level 1 Functionality", "ismp.edp.opts",
-                       FT_BOOLEAN, 32, TFS(&is_set), EDP_SWITCH_OPTION_LEVEL1,
-                       "", HFILL }
+                       { "Level 1 Functionality", "ismp.edp.switch_option_level1",
+                       FT_BOOLEAN, 32, TFS(&tfs_set_notset), EDP_SWITCH_OPTION_LEVEL1,
+                       NULL, HFILL }
                },
                { &hf_ismp_edp_end_station_option_dhcp,
-                       { "DHCP Enabled", "ismp.edp.opts",
-                       FT_BOOLEAN, 32, TFS(&is_set), EDP_END_STATION_OPTION_DHCP,
-                       "", HFILL }
+                       { "DHCP Enabled", "ismp.edp.end_station_option_dhcp",
+                       FT_BOOLEAN, 32, TFS(&tfs_set_notset), EDP_END_STATION_OPTION_DHCP,
+                       NULL, HFILL }
                },
                { &hf_ismp_edp_end_station_option_dns,
-                       { "DNS Enabled", "ismp.edp.opts",
-                       FT_BOOLEAN, 32, TFS(&is_set), EDP_END_STATION_OPTION_DNS,
-                       "", HFILL }
+                       { "DNS Enabled", "ismp.edp.end_station_option_dns",
+                       FT_BOOLEAN, 32, TFS(&tfs_set_notset), EDP_END_STATION_OPTION_DNS,
+                       NULL, HFILL }
                },
                { &hf_ismp_edp_end_station_option_ad,
-                       { "Active Directory Enabled", "ismp.edp.opts",
-                       FT_BOOLEAN, 32, TFS(&is_set), EDP_END_STATION_OPTION_AD,
-                       "", HFILL }
+                       { "Active Directory Enabled", "ismp.edp.end_station_option_ad",
+                       FT_BOOLEAN, 32, TFS(&tfs_set_notset), EDP_END_STATION_OPTION_AD,
+                       NULL, HFILL }
                },
                { &hf_ismp_edp_num_neighbors,
                        { "Number of Known Neighbors", "ismp.edp.maccount",
                        FT_UINT16, BASE_DEC, NULL, 0x0,
-                       "", HFILL }
+                       NULL, HFILL }
                },
                { &hf_ismp_edp_neighbors,
                        { "Neighbors", "ismp.edp.nbrs",
                        FT_BYTES, BASE_NONE, NULL, 0x0,
-                       "", HFILL }
+                       NULL, HFILL }
                },
                { &hf_ismp_edp_num_tuples,
                        { "Number of Tuples", "ismp.edp.numtups",
                        FT_UINT16, BASE_DEC, NULL, 0x0,
-                       "", HFILL }
+                       NULL, HFILL }
                },
                { &hf_ismp_edp_tuples,
                        { "Number of Tuples", "ismp.edp.tups",
                        FT_BYTES, BASE_NONE, NULL, 0x0,
-                       "", HFILL }
-               },
+                       NULL, HFILL }
+               },
+
+               /* Generated from convert_proto_tree_add_text.pl */
+               { &hf_ismp_neighborhood_mac_address, { "MAC Address", "ismp.neighborhood_mac_address", FT_ETHER, BASE_NONE, NULL, 0x0, NULL, HFILL }},
+               { &hf_ismp_assigned_neighbor_state, { "Assigned Neighbor State", "ismp.assigned_neighbor_state", FT_UINT32, BASE_HEX, NULL, 0x0, NULL, HFILL }},
+               { &hf_ismp_tuple_type, { "Tuple Type", "ismp.tuple_type", FT_UINT16, BASE_DEC, VALS(edp_tuple_types), 0x0, NULL, HFILL }},
+               { &hf_ismp_tuple_length, { "Tuple Length", "ismp.tuple_length", FT_UINT16, BASE_DEC, NULL, 0x0, NULL, HFILL }},
+               { &hf_ismp_hold_time, { "Hold Time", "ismp.hold_time", FT_UINT16, BASE_DEC, NULL, 0x0, NULL, HFILL }},
+               { &hf_ismp_interface_name, { "Interface Name", "ismp.interface_name", FT_STRING, BASE_NONE, NULL, 0x0, NULL, HFILL }},
+               { &hf_ismp_system_description, { "System Description", "ismp.system_description", FT_STRING, BASE_NONE, NULL, 0x0, NULL, HFILL }},
+               { &hf_ismp_interface_ipx_address, { "Interface IPX_address", "ismp.interface_ipx_address", FT_STRING, BASE_NONE, NULL, 0x0, NULL, HFILL }},
+               { &hf_ismp_unknown_tuple_data, { "Unknown Tuple Data", "ismp.unknown_tuple_data", FT_STRING, BASE_NONE, NULL, 0x0, NULL, HFILL }},
        };
 
 /* Setup protocol subtree array */
@@ -847,6 +854,12 @@ proto_register_ismp(void)
                &ett_ismp_edp_tuples_leaf,
        };
 
+       static ei_register_info ei[] = {
+               { &ei_ismp_malformed, { "ismp.malformed", PI_MALFORMED, PI_ERROR, "Malformed packet", EXPFILL }},
+       };
+
+       expert_module_t* expert_ismp;
+
 /* Register the protocol name and description */
        proto_ismp = proto_register_protocol("InterSwitch Message Protocol",
            "ISMP", "ismp");
@@ -854,6 +867,8 @@ proto_register_ismp(void)
 /* Required function calls to register the header fields and subtrees used */
        proto_register_field_array(proto_ismp, hf, array_length(hf));
        proto_register_subtree_array(ett, array_length(ett));
+       expert_ismp = expert_register_protocol(proto_ismp);
+       expert_register_field_array(expert_ismp, ei, array_length(ei));
 }
 
 
@@ -868,5 +883,18 @@ proto_reg_handoff_ismp(void)
 
        ismp_handle = create_dissector_handle(dissect_ismp,
            proto_ismp);
-       dissector_add("ethertype", ETHERTYPE_ISMP, ismp_handle);
+       dissector_add_uint("ethertype", ETHERTYPE_ISMP, ismp_handle);
 }
+
+/*
+ * Editor modelines  -  http://www.wireshark.org/tools/modelines.html
+ *
+ * Local variables:
+ * c-basic-offset: 8
+ * tab-width: 8
+ * indent-tabs-mode: t
+ * End:
+ *
+ * vi: set shiftwidth=8 tabstop=8 noexpandtab:
+ * :indentSize=8:tabSize=8:noTabs=false:
+ */