If the capture child process exits unexpectedly, give more information
[obnox/wireshark/wip.git] / packet-cdp.c
index a5f73463e2e4e245eff60fcd0b3e853d80c99fb7..f33b7424af3eac40e4dffee56ae8de2c4cf5965d 100644 (file)
@@ -2,7 +2,7 @@
  * Routines for the disassembly of the "Cisco Discovery Protocol"
  * (c) Copyright Hannes R. Boehm <hannes@boehm.org>
  *
- * $Id: packet-cdp.c,v 1.18 2000/01/13 00:41:09 guy Exp $
+ * $Id: packet-cdp.c,v 1.27 2000/11/19 08:53:56 guy Exp $
  *
  * Ethereal - Network traffic analyzer
  * By Gerald Combs <gerald@zing.org>
@@ -35,6 +35,7 @@
 
 #include <glib.h>
 #include "packet.h"
+#include "strutil.h"
 #include "nlpid.h"
 
 /*
@@ -101,23 +102,25 @@ dissect_cdp(const u_char *pd, int offset, frame_data *fd, proto_tree *tree)
     guint32 naddresses;
     int addr_length;
 
+    OLD_CHECK_DISPLAY_AS_DATA(proto_cdp, pd, offset, fd, tree);
+
     if (check_col(fd, COL_PROTOCOL))
-        col_add_str(fd, COL_PROTOCOL, "CDP");
+        col_set_str(fd, COL_PROTOCOL, "CDP");
     if (check_col(fd, COL_INFO))
-        col_add_str(fd, COL_INFO, "Cisco Discovery Protocol"); 
+        col_set_str(fd, COL_INFO, "Cisco Discovery Protocol"); 
 
     if(tree){
-        ti = proto_tree_add_item(tree, proto_cdp, offset, END_OF_FRAME, NULL);
+        ti = proto_tree_add_item(tree, proto_cdp, NullTVB, offset, END_OF_FRAME, FALSE);
        cdp_tree = proto_item_add_subtree(ti, ett_cdp);
        
        /* CDP header */
-       proto_tree_add_item(cdp_tree, hf_cdp_version, offset, 1, pd[offset]);
+       proto_tree_add_uint(cdp_tree, hf_cdp_version, NullTVB, offset, 1, pd[offset]);
        offset += 1;
-       proto_tree_add_item_format(cdp_tree, hf_cdp_ttl, offset, 1,
+       proto_tree_add_uint_format(cdp_tree, hf_cdp_ttl, NullTVB, offset, 1,
                                   pntohs(&pd[offset]),
                                   "TTL: %u seconds", pd[offset]);
        offset += 1;
-       proto_tree_add_item_format(cdp_tree, hf_cdp_flags, offset, 2,
+       proto_tree_add_uint_format(cdp_tree, hf_cdp_flags, NullTVB, offset, 2,
                                   pd[offset], 
                                   "Checksum: 0x%04x", pntohs(&pd[offset]));
        offset += 2;
@@ -131,35 +134,34 @@ dissect_cdp(const u_char *pd, int offset, frame_data *fd, proto_tree *tree)
                switch( type ){
                        case TYPE_DEVICE_ID:
                                /* Device ID */
-                               tlvi = proto_tree_add_text(cdp_tree, offset,
+                               tlvi = proto_tree_add_text(cdp_tree, NullTVB, offset,
                                    length, "Device ID: %s",
                                    &pd[offset+4]);
                                tlv_tree = proto_item_add_subtree(tlvi,
                                    ett_cdp_tlv);
-                               proto_tree_add_item(tlv_tree, hf_cdp_tlvtype,
+                               proto_tree_add_uint(tlv_tree, hf_cdp_tlvtype, NullTVB,
                                    offset + TLV_TYPE, 2, type);
-                               proto_tree_add_item(tlv_tree, hf_cdp_tlvlength,
+                               proto_tree_add_uint(tlv_tree, hf_cdp_tlvlength, NullTVB,
                                    offset + TLV_LENGTH, 2, length);
-                               proto_tree_add_text(tlv_tree, offset + 4,
+                               proto_tree_add_text(tlv_tree, NullTVB, offset + 4,
                                    length - 4, "Device ID: %s",
                                    &pd[offset+4]);
                                offset+=length;
                                break;
                        case TYPE_ADDRESS:
                                /* Addresses */
-                               tlvi = proto_tree_add_text(cdp_tree, offset,
-                                   4, "Address",
-                                   type, length);
+                               tlvi = proto_tree_add_text(cdp_tree, NullTVB, offset,
+                                   length, "Addresses");
                                tlv_tree = proto_item_add_subtree(tlvi,
                                    ett_cdp_tlv);
-                               proto_tree_add_item(tlv_tree, hf_cdp_tlvtype,
+                               proto_tree_add_uint(tlv_tree, hf_cdp_tlvtype, NullTVB,
                                    offset + TLV_TYPE, 2, type);
-                               proto_tree_add_item(tlv_tree, hf_cdp_tlvlength,
+                               proto_tree_add_uint(tlv_tree, hf_cdp_tlvlength, NullTVB,
                                    offset + TLV_LENGTH, 2, length);
                                offset += 4;
                                length -= 4;
                                naddresses = pntohl(&pd[offset]);
-                               proto_tree_add_text(tlv_tree, offset, 4,
+                               proto_tree_add_text(tlv_tree, NullTVB, offset, 4,
                                    "Number of addresses: %u", naddresses);
                                offset += 4;
                                length -= 4;
@@ -190,29 +192,29 @@ dissect_cdp(const u_char *pd, int offset, frame_data *fd, proto_tree *tree)
                                       capture work. */
                                    real_length = length + 3;
                                }
-                               tlvi = proto_tree_add_text(cdp_tree, offset,
+                               tlvi = proto_tree_add_text(cdp_tree, NullTVB, offset,
                                    real_length, "Port ID: %s",
                                    &pd[offset+4]);
                                tlv_tree = proto_item_add_subtree(tlvi,
                                    ett_cdp_tlv);
-                               proto_tree_add_item(tlv_tree, hf_cdp_tlvtype,
+                               proto_tree_add_uint(tlv_tree, hf_cdp_tlvtype, NullTVB,
                                    offset + TLV_TYPE, 2, type);
-                               proto_tree_add_item(tlv_tree, hf_cdp_tlvlength,
+                               proto_tree_add_uint(tlv_tree, hf_cdp_tlvlength, NullTVB,
                                    offset + TLV_LENGTH, 2, length);
-                               proto_tree_add_text(tlv_tree, offset + 4,
+                               proto_tree_add_text(tlv_tree, NullTVB, offset + 4,
                                    real_length - 4,
                                    "Sent through Interface: %s",
                                    &pd[offset+4]);
                                offset += real_length;
                                break;
                        case TYPE_CAPABILITIES:
-                               tlvi = proto_tree_add_text(cdp_tree, offset,
+                               tlvi = proto_tree_add_text(cdp_tree, NullTVB, offset,
                                    length, "Capabilities");
                                tlv_tree = proto_item_add_subtree(tlvi,
                                    ett_cdp_tlv);
-                               proto_tree_add_item(tlv_tree, hf_cdp_tlvtype,
+                               proto_tree_add_uint(tlv_tree, hf_cdp_tlvtype, NullTVB,
                                    offset + TLV_TYPE, 2, type);
-                               proto_tree_add_item(tlv_tree, hf_cdp_tlvlength,
+                               proto_tree_add_uint(tlv_tree, hf_cdp_tlvlength, NullTVB,
                                    offset + TLV_LENGTH, 2, length);
                                offset += 4;
                                length -= 4;
@@ -221,13 +223,13 @@ dissect_cdp(const u_char *pd, int offset, frame_data *fd, proto_tree *tree)
                                offset += length;
                                break;
                        case TYPE_IOS_VERSION:
-                               tlvi = proto_tree_add_text(cdp_tree, offset,
+                               tlvi = proto_tree_add_text(cdp_tree, NullTVB, offset,
                                    length, "Software Version");
                                tlv_tree = proto_item_add_subtree(tlvi,
                                    ett_cdp_tlv);
-                               proto_tree_add_item(tlv_tree, hf_cdp_tlvtype,
+                               proto_tree_add_uint(tlv_tree, hf_cdp_tlvtype, NullTVB,
                                    offset + TLV_TYPE, 2, type);
-                               proto_tree_add_item(tlv_tree, hf_cdp_tlvlength,
+                               proto_tree_add_uint(tlv_tree, hf_cdp_tlvlength, NullTVB,
                                    offset + TLV_LENGTH, 2, length);
                                add_multi_line_string_to_tree(tlv_tree,
                                    offset + 4, length - 4, "Software Version: ",
@@ -239,39 +241,39 @@ dissect_cdp(const u_char *pd, int offset, frame_data *fd, proto_tree *tree)
                                stringmem = malloc(length);
                                memset(stringmem, '\0', length);
                                memcpy(stringmem, &pd[offset+4], length - 4 );
-                               tlvi = proto_tree_add_text(cdp_tree,
+                               tlvi = proto_tree_add_text(cdp_tree, NullTVB,
                                    offset, length, "Platform: %s",
                                    stringmem);
                                tlv_tree = proto_item_add_subtree(tlvi,
                                    ett_cdp_tlv);
-                               proto_tree_add_item(tlv_tree, hf_cdp_tlvtype,
+                               proto_tree_add_uint(tlv_tree, hf_cdp_tlvtype, NullTVB,
                                    offset + TLV_TYPE, 2, type);
-                               proto_tree_add_item(tlv_tree, hf_cdp_tlvlength,
+                               proto_tree_add_uint(tlv_tree, hf_cdp_tlvlength, NullTVB,
                                    offset + TLV_LENGTH, 2, length);
-                               proto_tree_add_text(tlv_tree, offset + 4,
+                               proto_tree_add_text(tlv_tree, NullTVB, offset + 4,
                                    length - 4, "Platform: %s", stringmem);
                                free(stringmem);
                                offset+=length;
                                break;
                        default:
-                               tlvi = proto_tree_add_text(cdp_tree, offset,
+                               tlvi = proto_tree_add_text(cdp_tree, NullTVB, offset,
                                    length, "Type: %s, length: %u",
                                    type_str, length);
                                tlv_tree = proto_item_add_subtree(tlvi,
                                    ett_cdp_tlv);
-                               proto_tree_add_item(tlv_tree, hf_cdp_tlvtype,
+                               proto_tree_add_uint(tlv_tree, hf_cdp_tlvtype, NullTVB,
                                    offset + TLV_TYPE, 2, type);
-                               proto_tree_add_item(tlv_tree, hf_cdp_tlvlength,
+                               proto_tree_add_uint(tlv_tree, hf_cdp_tlvlength, NullTVB,
                                    offset + TLV_LENGTH, 2, length);
                                if (length > 4) {
-                                       proto_tree_add_text(tlv_tree,
+                                       proto_tree_add_text(tlv_tree, NullTVB,
                                            offset + 4, length - 4, "Data");
                                } else
                                        return;
                                offset+=length;
                }
        }
-       dissect_data(pd, offset, fd, cdp_tree);
+       old_dissect_data(pd, offset, fd, cdp_tree);
     }
 }
 
@@ -291,6 +293,7 @@ dissect_address_tlv(const u_char *pd, int offset, int length, proto_tree *tree)
     proto_tree *address_tree;
     guint8 protocol_type;
     guint8 protocol_length;
+    int nlpid;
     char *protocol_str;
     guint16 address_length;
     char *address_type_str;
@@ -298,93 +301,63 @@ dissect_address_tlv(const u_char *pd, int offset, int length, proto_tree *tree)
 
     if (length < 1)
         return -1;
+    ti = proto_tree_add_notext(tree, NullTVB, offset, length);
+    address_tree = proto_item_add_subtree(ti, ett_cdp_address);
     protocol_type = pd[offset];
-    if (length < 2) {
-        ti = proto_tree_add_text(tree, offset, length, "Truncated address");
-       address_tree = proto_item_add_subtree(ti, ett_cdp_address);
-        proto_tree_add_text(address_tree, offset, 1, "Protocol type: %s",
-         val_to_str(protocol_type, proto_type_vals, "Unknown (0x%02x)"));
-       return -1;
-    }
-    protocol_length = pd[offset + 1];
-    if (length < 2 + protocol_length) {
-        ti = proto_tree_add_text(tree, offset, length, "Truncated address");
-       address_tree = proto_item_add_subtree(ti, ett_cdp_address);
-        proto_tree_add_text(address_tree, offset, 1, "Protocol type: %s",
-         val_to_str(protocol_type, proto_type_vals, "Unknown (0x%02x)"));
-       offset += 1;
-        proto_tree_add_text(address_tree, offset, 1, "Protocol length: %u",
-          protocol_length);
+    proto_tree_add_text(address_tree, NullTVB, offset, 1, "Protocol type: %s",
+       val_to_str(protocol_type, proto_type_vals, "Unknown (0x%02x)"));
+    offset += 1;
+    length -= 1;
+
+    if (length < 1) {
+        proto_item_set_text(ti, "Truncated address");
        return -1;
     }
+    protocol_length = pd[offset];
+    proto_tree_add_text(address_tree, NullTVB, offset, 1, "Protocol length: %u",
+                       protocol_length);
+    offset += 1;
+    length -= 1;
 
-    if (length < 2 + protocol_length) {
-        ti = proto_tree_add_text(tree, offset, length, "Truncated address");
-       address_tree = proto_item_add_subtree(ti, ett_cdp_address);
-        proto_tree_add_text(address_tree, offset, 1, "Protocol type: %s",
-          val_to_str(protocol_type, proto_type_vals, "Unknown (0x%02x)"));
-        offset += 1;
-        length -= 1;
-        proto_tree_add_text(address_tree, offset, 1, "Protocol length: %u",
-          protocol_length);
-        offset += 1;
-        length -= 1;
+    if (length < protocol_length) {
+        proto_item_set_text(ti, "Truncated address");
         if (length != 0) {
-            proto_tree_add_text(address_tree, offset, length,
+            proto_tree_add_text(address_tree, NullTVB, offset, length,
               "Protocol: %s (truncated)", bytes_to_str(&pd[offset], length));
         }
        return -1;
     }
-
     protocol_str = NULL;
-    if (protocol_type == PROTO_TYPE_NLPID && protocol_length == 1)
-       protocol_str = val_to_str(pd[offset + 2], nlpid_vals, "Unknown (0x%02x)");
+    if (protocol_type == PROTO_TYPE_NLPID && protocol_length == 1) {
+       nlpid = pd[offset];
+       protocol_str = val_to_str(nlpid, nlpid_vals, "Unknown (0x%02x)");
+    } else
+        nlpid = -1;
     if (protocol_str == NULL)
         protocol_str = bytes_to_str(&pd[offset], protocol_length);
+    proto_tree_add_text(address_tree, NullTVB, offset, protocol_length,
+                       "Protocol: %s", protocol_str);
+    offset += protocol_length;
+    length -= protocol_length;
 
-    if (length < 2 + protocol_length + 2) {
-        ti = proto_tree_add_text(tree, offset, length, "Truncated address");
-       address_tree = proto_item_add_subtree(ti, ett_cdp_address);
-        proto_tree_add_text(address_tree, offset, 1, "Protocol type: %s",
-          val_to_str(protocol_type, proto_type_vals, "Unknown (0x%02x)"));
-        offset += 1;
-        length -= 1;
-        proto_tree_add_text(address_tree, offset, 1, "Protocol length: %u",
-          protocol_length);
-        offset += 1;
-        length -= 1;
-        proto_tree_add_text(address_tree, offset, protocol_length,
-          "Protocol: %s", protocol_str);
+    if (length < 2) {
+        proto_item_set_text(ti, "Truncated address");
        return -1;
     }
-    address_length = pntohs(&pd[offset + 2 + protocol_length]);
-
-    if (length < 2 + protocol_length + 2 + address_length) {
-        ti = proto_tree_add_text(tree, offset, length, "Truncated address");
-       address_tree = proto_item_add_subtree(ti, ett_cdp_address);
-        proto_tree_add_text(address_tree, offset, 1, "Protocol type: %s",
-          val_to_str(protocol_type, proto_type_vals, "Unknown (0x%02x)"));
-        offset += 1;
-        length -= 1;
-        proto_tree_add_text(address_tree, offset, 1, "Protocol length: %u",
-          protocol_length);
-        offset += 1;
-        length -= 1;
-        proto_tree_add_text(address_tree, offset, protocol_length,
-          "Protocol: %s", protocol_str);
-        offset += protocol_length;
-        length -= protocol_length;
-        proto_tree_add_text(address_tree, offset, 2, "Address length: %u",
-          address_length);
-        offset += 2;
-        length -= 2;
+    address_length = pntohs(&pd[offset]);
+    proto_tree_add_text(address_tree, NullTVB, offset, 2, "Address length: %u",
+                       address_length);
+    offset += 2;
+    length -= 2;
+
+    if (length < address_length) {
+        proto_item_set_text(ti, "Truncated address");
         if (length != 0) {
-            proto_tree_add_text(address_tree, offset, length,
+            proto_tree_add_text(address_tree, NullTVB, offset, length,
               "Address: %s (truncated)", bytes_to_str(&pd[offset], length));
         }
        return -1;
     }
-
     /* XXX - the Cisco document seems to be saying that, for 802.2-format
        protocol types, 0xAAAA03 0x000000 0x0800 is IPv6, but 0x0800 is
        the Ethernet protocol type for IPv4. */
@@ -392,7 +365,7 @@ dissect_address_tlv(const u_char *pd, int offset, int length, proto_tree *tree)
     address_type_str = NULL;
     address_str = NULL;
     if (protocol_type == PROTO_TYPE_NLPID && protocol_length == 1) {
-        switch (pd[offset + 2]) {
+        switch (nlpid) {
 
         /* XXX - dissect NLPID_ISO8473_CLNP as OSI CLNP address? */
 
@@ -400,7 +373,7 @@ dissect_address_tlv(const u_char *pd, int offset, int length, proto_tree *tree)
             if (address_length == 4) {
                 /* The address is an IP address. */
                 address_type_str = "IP address";
-                address_str = ip_to_str(&pd[offset + 2 + protocol_length + 2]);
+                address_str = ip_to_str(&pd[offset]);
             }
             break;
         }
@@ -408,29 +381,10 @@ dissect_address_tlv(const u_char *pd, int offset, int length, proto_tree *tree)
     if (address_type_str == NULL)
         address_type_str = "Address";
     if (address_str == NULL) {
-        address_str = bytes_to_str(&pd[offset + 2 + protocol_length + 2],
-         address_length);
+        address_str = bytes_to_str(&pd[offset], address_length);
     }
-    ti = proto_tree_add_text(tree, offset, length, "%s: %s",
-      address_type_str, address_str);
-    address_tree = proto_item_add_subtree(ti, ett_cdp_address);
-    proto_tree_add_text(address_tree, offset, 1, "Protocol type: %s",
-      val_to_str(protocol_type, proto_type_vals, "Unknown (0x%02x)"));
-    offset += 1;
-    length -= 1;
-    proto_tree_add_text(address_tree, offset, 1, "Protocol length: %u",
-      protocol_length);
-    offset += 1;
-    length -= 1;
-    proto_tree_add_text(address_tree, offset, protocol_length,
-      "Protocol: %s", protocol_str);
-    offset += protocol_length;
-    length -= protocol_length;
-    proto_tree_add_text(address_tree, offset, 2, "Address length: %u",
-      address_length);
-    offset += 2;
-    length -= 2;
-    proto_tree_add_text(address_tree, offset, address_length, "%s: %s",
+    proto_item_set_text(ti, "%s: %s", address_type_str, address_str);
+    proto_tree_add_text(address_tree, NullTVB, offset, address_length, "%s: %s",
       address_type_str, address_str);
     return 2 + protocol_length + 2 + address_length;
 }
@@ -445,34 +399,34 @@ dissect_capabilities(const u_char *pd, int offset, int length, proto_tree *tree)
     if (length < 4)
         return;
     capabilities = pntohl(&pd[offset]);
-    ti = proto_tree_add_text(tree, offset, length, "Capabilities: 0x%08x",
+    ti = proto_tree_add_text(tree, NullTVB, offset, length, "Capabilities: 0x%08x",
         capabilities);
     capabilities_tree = proto_item_add_subtree(ti, ett_cdp_capabilities);
-    proto_tree_add_text(capabilities_tree, offset, 4,
+    proto_tree_add_text(capabilities_tree, NullTVB, offset, 4,
        decode_boolean_bitfield(capabilities, 0x01, 4*8,
            "Performs level 3 routing",
            "Doesn't perform level 3 routing"));
-    proto_tree_add_text(capabilities_tree, offset, 4,
+    proto_tree_add_text(capabilities_tree, NullTVB, offset, 4,
        decode_boolean_bitfield(capabilities, 0x02, 4*8,
            "Performs level 2 transparent bridging",
            "Doesn't perform level 2 transparent bridging"));
-    proto_tree_add_text(capabilities_tree, offset, 4,
+    proto_tree_add_text(capabilities_tree, NullTVB, offset, 4,
        decode_boolean_bitfield(capabilities, 0x04, 4*8,
            "Performs level 2 source-route bridging",
            "Doesn't perform level 2 source-route bridging"));
-    proto_tree_add_text(capabilities_tree, offset, 4,
+    proto_tree_add_text(capabilities_tree, NullTVB, offset, 4,
        decode_boolean_bitfield(capabilities, 0x08, 4*8,
            "Performs level 2 switching",
            "Doesn't perform level 2 switching"));
-    proto_tree_add_text(capabilities_tree, offset, 4,
+    proto_tree_add_text(capabilities_tree, NullTVB, offset, 4,
        decode_boolean_bitfield(capabilities, 0x10, 4*8,
            "Sends and receives packets for network-layer protocols",
            "Doesn't send or receive packets for network-layer protocols"));
-    proto_tree_add_text(capabilities_tree, offset, 4,
+    proto_tree_add_text(capabilities_tree, NullTVB, offset, 4,
        decode_boolean_bitfield(capabilities, 0x20, 4*8,
            "Doesn't forward IGMP Report packets on nonrouter ports",
            "Forwards IGMP Report packets on nonrouter ports"));
-    proto_tree_add_text(capabilities_tree, offset, 4,
+    proto_tree_add_text(capabilities_tree, NullTVB, offset, 4,
        decode_boolean_bitfield(capabilities, 0x40, 4*8,
            "Provides level 1 functionality",
            "Doesn't provide level 1 functionality"));
@@ -505,7 +459,7 @@ add_multi_line_string_to_tree(proto_tree *tree, gint start, gint len,
            line_len = strlen(p);
            data_len = line_len;
        }
-       proto_tree_add_text(tree, start, data_len, "%s%.*s", prefix,
+       proto_tree_add_text(tree, NullTVB, start, data_len, "%s%.*s", prefix,
           line_len, p);
        if (q == NULL)
            break;