Get rid of redundant include of <stdio.h> - one is enough.
[obnox/wireshark/wip.git] / packet-ip.c
index 0f955baf0c4d0881a033d15a2ee424cfb485a889..d2ae6a4b5dcbc8090b7347d54d0217fcc633436e 100644 (file)
@@ -1,7 +1,7 @@
 /* packet-ip.c
  * Routines for IP and miscellaneous IP protocol packet disassembly
  *
- * $Id: packet-ip.c,v 1.60 1999/11/02 05:38:51 guy Exp $
+ * $Id: packet-ip.c,v 1.73 2000/02/09 19:17:51 gram Exp $
  *
  * Ethereal - Network traffic analyzer
  * By Gerald Combs <gerald@zing.org>
@@ -40,7 +40,6 @@
 #include <glib.h>
 #include "packet.h"
 #include "resolv.h"
-#include "util.h"
 
 #ifdef NEED_SNPRINTF_H
 # ifdef HAVE_STDARG_H
 #include "packet-ip.h"
 #endif
 
+/* Decode the old IPv4 TOS field as the DiffServ DS Field */
+gboolean g_ip_dscp_actif = TRUE;
+
 static int proto_ip = -1;
 static int hf_ip_version = -1;
 static int hf_ip_hdr_len = -1;
+static int hf_ip_dsfield = -1;
+static int hf_ip_dsfield_dscp = -1;
+static int hf_ip_dsfield_cu = -1;
 static int hf_ip_tos = -1;
 static int hf_ip_tos_precedence = -1;
 static int hf_ip_tos_delay = -1;
@@ -77,6 +82,15 @@ static int hf_ip_ttl = -1;
 static int hf_ip_proto = -1;
 static int hf_ip_checksum = -1;
 
+static gint ett_ip = -1;
+static gint ett_ip_dsfield = -1;
+static gint ett_ip_tos = -1;
+static gint ett_ip_off = -1;
+static gint ett_ip_options = -1;
+static gint ett_ip_option_sec = -1;
+static gint ett_ip_option_route = -1;
+static gint ett_ip_option_timestamp = -1;
+
 static int proto_igmp = -1;
 static int hf_igmp_version = -1;
 static int hf_igmp_type = -1;
@@ -84,11 +98,15 @@ static int hf_igmp_unused = -1;
 static int hf_igmp_checksum = -1;
 static int hf_igmp_group = -1;
 
+static gint ett_igmp = -1;
+
 static int proto_icmp = -1;
 static int hf_icmp_type = -1;
 static int hf_icmp_code = -1;
 static int hf_icmp_checksum = -1;
 
+static gint ett_icmp = -1;
+
 /* ICMP structs and definitions */
 typedef struct _e_icmp {
   guint8  icmp_type;
@@ -231,6 +249,34 @@ typedef struct _e_ip
 #define IP_MF          0x2000          /* Flag: "More Fragments"       */
 #define IP_OFFSET      0x1FFF          /* "Fragment Offset" part       */
 
+/* Differentiated Services Field. See RFCs 2474, 2597 and 2598. */
+#define IPDSFIELD_DSCP_MASK     0xFC
+#define IPDSFIELD_DSCP_SHIFT   2
+#define IPDSFIELD_DSCP(dsfield)        (((dsfield)&IPDSFIELD_DSCP_MASK)>>IPDSFIELD_DSCP_SHIFT)
+#define IPDSFIELD_DSCP_DEFAULT  0x00
+#define IPDSFIELD_DSCP_CS1      0x08
+#define IPDSFIELD_DSCP_CS2      0x10
+#define IPDSFIELD_DSCP_CS3      0x18
+#define IPDSFIELD_DSCP_CS4      0x20
+#define IPDSFIELD_DSCP_CS5      0x28
+#define IPDSFIELD_DSCP_CS6      0x30
+#define IPDSFIELD_DSCP_CS7      0x38
+#define IPDSFIELD_DSCP_AF11     0x0A
+#define IPDSFIELD_DSCP_AF12     0x0C
+#define IPDSFIELD_DSCP_AF13     0x0E
+#define IPDSFIELD_DSCP_AF21     0x12
+#define IPDSFIELD_DSCP_AF22     0x14
+#define IPDSFIELD_DSCP_AF23     0x16
+#define IPDSFIELD_DSCP_AF31     0x1A
+#define IPDSFIELD_DSCP_AF32     0x1C
+#define IPDSFIELD_DSCP_AF33     0x1E
+#define IPDSFIELD_DSCP_AF41     0x22
+#define IPDSFIELD_DSCP_AF42     0x24
+#define IPDSFIELD_DSCP_AF43     0x26
+#define IPDSFIELD_DSCP_EF       0x2E
+#define IPDSFIELD_CU_MASK      0x03
+
+/* IP TOS, superseded by the DS Field, RFC 2474. */
 #define IPTOS_TOS_MASK    0x1E
 #define IPTOS_TOS(tos)    ((tos) & IPTOS_TOS_MASK)
 #define IPTOS_NONE        0x00
@@ -300,7 +346,11 @@ typedef struct _e_ip
 
 
 void
-capture_ip(const u_char *pd, int offset, guint32 cap_len, packet_counts *ld) {
+capture_ip(const u_char *pd, int offset, packet_counts *ld) {
+  if (!BYTES_ARE_IN_FRAME(offset, IPH_MIN_LEN)) {
+    ld->other++;
+    return;
+  }
   switch (pd[offset + 9]) {
     case IP_PROTO_TCP:
       ld->tcp++;
@@ -317,6 +367,9 @@ capture_ip(const u_char *pd, int offset, guint32 cap_len, packet_counts *ld) {
     case IP_PROTO_GRE:
       ld->gre++;
       break;
+    case IP_PROTO_VINES:
+      ld->gre++;
+      break;
     default:
       ld->other++;
   }
@@ -348,7 +401,7 @@ dissect_ipopt_security(const ip_tcp_opt *optp, const u_char *opd, int offset,
     {0,                  NULL          } };
 
   tf = proto_tree_add_text(opt_tree, offset,      optlen, "%s:", optp->name);
-  field_tree = proto_item_add_subtree(tf, optp->subtree_index);
+  field_tree = proto_item_add_subtree(tf, *optp->subtree_index);
   offset += 2;
 
   val = pntohs(opd);
@@ -384,7 +437,7 @@ dissect_ipopt_route(const ip_tcp_opt *optp, const u_char *opd, int offset,
 
   tf = proto_tree_add_text(opt_tree, offset,      optlen, "%s (%d bytes)",
                                optp->name, optlen);
-  field_tree = proto_item_add_subtree(tf, optp->subtree_index);
+  field_tree = proto_item_add_subtree(tf, *optp->subtree_index);
 
   optoffset += 2;      /* skip past type and length */
   optlen -= 2;         /* subtract size of type and length */
@@ -446,7 +499,7 @@ dissect_ipopt_timestamp(const ip_tcp_opt *optp, const u_char *opd,
   guint ts;
 
   tf = proto_tree_add_text(opt_tree, offset,      optlen, "%s:", optp->name);
-  field_tree = proto_item_add_subtree(tf, optp->subtree_index);
+  field_tree = proto_item_add_subtree(tf, *optp->subtree_index);
 
   optoffset += 2;      /* skip past type and length */
   optlen -= 2;         /* subtract size of type and length */
@@ -473,25 +526,18 @@ dissect_ipopt_timestamp(const ip_tcp_opt *optp, const u_char *opd,
 
   while (optlen > 0) {
     if (flg == IPOPT_TS_TSANDADDR) {
-      if (optlen < 4) {
-        proto_tree_add_text(field_tree, offset + optoffset, optlen,
-          "(suboption would go past end of option)");
-        break;
-      }
       /* XXX - check whether it goes past end of packet */
-      ts = pntohl(opd);
-      opd += 4;
-      optlen -= 4;
-      if (optlen < 4) {
+      if (optlen < 8) {
         proto_tree_add_text(field_tree, offset + optoffset, optlen,
           "(suboption would go past end of option)");
         break;
       }
-      /* XXX - check whether it goes past end of packet */
       memcpy((char *)&addr, (char *)opd, sizeof(addr));
       opd += 4;
-      optlen -= 4;
-      proto_tree_add_text(field_tree, offset,      8,
+      ts = pntohl(opd);
+      opd += 4;
+      optlen -= 8;
+      proto_tree_add_text(field_tree, offset + optoffset,      8,
           "Address = %s, time stamp = %u",
           ((addr.s_addr == 0) ? "-" :  (char *)get_hostname(addr.s_addr)),
           ts);
@@ -517,7 +563,7 @@ static const ip_tcp_opt ipopts[] = {
   {
     IPOPT_END,
     "EOL",
-    -1,
+    NULL,
     NO_LENGTH,
     0,
     NULL,
@@ -525,7 +571,7 @@ static const ip_tcp_opt ipopts[] = {
   {
     IPOPT_NOOP,
     "NOP",
-    -1,
+    NULL,
     NO_LENGTH,
     0,
     NULL,
@@ -533,7 +579,7 @@ static const ip_tcp_opt ipopts[] = {
   {
     IPOPT_SEC,
     "Security",
-    ETT_IP_OPTION_SEC,
+    &ett_ip_option_sec,
     FIXED_LENGTH,
     IPOLEN_SEC,
     dissect_ipopt_security
@@ -541,7 +587,7 @@ static const ip_tcp_opt ipopts[] = {
   {
     IPOPT_SSRR,
     "Strict source route",
-    ETT_IP_OPTION_ROUTE,
+    &ett_ip_option_route,
     VARIABLE_LENGTH,
     IPOLEN_SSRR_MIN,
     dissect_ipopt_route
@@ -549,7 +595,7 @@ static const ip_tcp_opt ipopts[] = {
   {
     IPOPT_LSRR,
     "Loose source route",
-    ETT_IP_OPTION_ROUTE,
+    &ett_ip_option_route,
     VARIABLE_LENGTH,
     IPOLEN_LSRR_MIN,
     dissect_ipopt_route
@@ -557,7 +603,7 @@ static const ip_tcp_opt ipopts[] = {
   {
     IPOPT_RR,
     "Record route",
-    ETT_IP_OPTION_ROUTE,
+    &ett_ip_option_route,
     VARIABLE_LENGTH,
     IPOLEN_RR_MIN,
     dissect_ipopt_route
@@ -565,7 +611,7 @@ static const ip_tcp_opt ipopts[] = {
   {
     IPOPT_SID,
     "Stream identifier",
-    -1,
+    NULL,
     FIXED_LENGTH,
     IPOLEN_SID,
     dissect_ipopt_sid
@@ -573,7 +619,7 @@ static const ip_tcp_opt ipopts[] = {
   {
     IPOPT_TIMESTAMP,
     "Time stamp",
-    ETT_IP_OPTION_TIMESTAMP,
+    &ett_ip_option_timestamp,
     VARIABLE_LENGTH,
     IPOLEN_TIMESTAMP_MIN,
     dissect_ipopt_timestamp
@@ -638,7 +684,7 @@ dissect_ip_tcp_options(const u_char *opd, int offset, guint length,
            option length. */
         proto_tree_add_text(opt_tree, offset,      2,
               "%s (with too-short option length = %u byte%s)", name,
-              plurality(len, "", "s"));
+              len, plurality(len, "", "s"));
         return;
       } else if (len - 2 > length) {
         /* Bogus - option goes past the end of the header. */
@@ -699,8 +745,33 @@ static const value_string proto_vals[] = { {IP_PROTO_ICMP, "ICMP"},
                                            {IP_PROTO_ESP,  "ESP" },
                                            {IP_PROTO_IPV6, "IPv6"},
                                            {IP_PROTO_PIM,  "PIM" },
+                                           {IP_PROTO_VINES,"VINES"},
                                            {0,             NULL  } };
 
+static const value_string dscp_vals[] = {
+                 { IPDSFIELD_DSCP_DEFAULT, "Default"               },
+                 { IPDSFIELD_DSCP_CS1,     "Class Selector 1"      },
+                 { IPDSFIELD_DSCP_CS2,     "Class Selector 2"      },
+                 { IPDSFIELD_DSCP_CS3,     "Class Selector 3"      },
+                 { IPDSFIELD_DSCP_CS4,     "Class Selector 4"      },
+                 { IPDSFIELD_DSCP_CS5,     "Class Selector 5"      },
+                 { IPDSFIELD_DSCP_CS6,     "Class Selector 6"      },
+                 { IPDSFIELD_DSCP_CS7,     "Class Selector 7"      },
+                 { IPDSFIELD_DSCP_AF11,    "Assured Forwarding 11" },
+                 { IPDSFIELD_DSCP_AF12,    "Assured Forwarding 12" },
+                 { IPDSFIELD_DSCP_AF13,    "Assured Forwarding 13" },
+                 { IPDSFIELD_DSCP_AF21,    "Assured Forwarding 21" },
+                 { IPDSFIELD_DSCP_AF22,    "Assured Forwarding 22" },
+                 { IPDSFIELD_DSCP_AF23,    "Assured Forwarding 23" },
+                 { IPDSFIELD_DSCP_AF31,    "Assured Forwarding 31" },
+                 { IPDSFIELD_DSCP_AF32,    "Assured Forwarding 32" },
+                 { IPDSFIELD_DSCP_AF33,    "Assured Forwarding 33" },
+                 { IPDSFIELD_DSCP_AF41,    "Assured Forwarding 41" },
+                 { IPDSFIELD_DSCP_AF42,    "Assured Forwarding 42" },
+                 { IPDSFIELD_DSCP_AF43,    "Assured Forwarding 43" },
+                 { IPDSFIELD_DSCP_EF,      "Expedited Forwarding"  },
+                 { 0,                      NULL                    } };
+
 static const value_string precedence_vals[] = {
                  { IPTOS_PREC_ROUTINE,         "routine"              },
                  { IPTOS_PREC_PRIORITY,        "priority"             },
@@ -737,6 +808,28 @@ static const true_false_string flags_set_truth = {
   "Not set"
 };
 
+static char *ip_checksum_state(e_ip *iph)
+{
+    unsigned long Sum;
+    unsigned char *Ptr, *PtrEnd;
+    unsigned short word;
+
+    Sum    = 0;
+    PtrEnd = (lo_nibble(iph->ip_v_hl) * 4 + (char *)iph);
+    for (Ptr = (unsigned char *) iph; Ptr < PtrEnd; Ptr += 2) {
+       memcpy(&word, Ptr, sizeof word);
+        Sum += word;
+    }
+
+    Sum = (Sum & 0xFFFF) + (Sum >> 16);
+    Sum = (Sum & 0xFFFF) + (Sum >> 16);
+
+    if (Sum != 0xffff)
+        return "incorrect";
+
+    return "correct";
+}
+
 void
 dissect_ip(const u_char *pd, int offset, frame_data *fd, proto_tree *tree) {
   e_ip       iph;
@@ -786,6 +879,7 @@ dissect_ip(const u_char *pd, int offset, frame_data *fd, proto_tree *tree) {
     case IP_PROTO_AH:
     case IP_PROTO_IPV6:
     case IP_PROTO_PIM:
+    case IP_PROTO_VINES:
       /* Names are set in the associated dissect_* routines */
       break;
     default:
@@ -823,27 +917,39 @@ dissect_ip(const u_char *pd, int offset, frame_data *fd, proto_tree *tree) {
     }
 
     ti = proto_tree_add_item(tree, proto_ip, offset, hlen, NULL);
-    ip_tree = proto_item_add_subtree(ti, ETT_IP);
+    ip_tree = proto_item_add_subtree(ti, ett_ip);
 
     proto_tree_add_item(ip_tree, hf_ip_version, offset, 1, hi_nibble(iph.ip_v_hl));
     proto_tree_add_item_format(ip_tree, hf_ip_hdr_len, offset, 1, hlen,
-       "Header length: %d bytes", hlen); 
-    tf = proto_tree_add_item_format(ip_tree, hf_ip_tos, offset + 1, 1, iph.ip_tos,
-       "Type of service: 0x%02x (%s)", iph.ip_tos,
-       val_to_str( IPTOS_TOS(iph.ip_tos), iptos_vals, "Unknown") );
-
-    field_tree = proto_item_add_subtree(tf, ETT_IP_TOS);
-    proto_tree_add_item(field_tree, hf_ip_tos_precedence, offset + 1, 1, iph.ip_tos);
-    proto_tree_add_item(field_tree, hf_ip_tos_delay, offset + 1, 1, iph.ip_tos);
-    proto_tree_add_item(field_tree, hf_ip_tos_throughput, offset + 1, 1, iph.ip_tos);
-    proto_tree_add_item(field_tree, hf_ip_tos_reliability, offset + 1, 1, iph.ip_tos);
-    proto_tree_add_item(field_tree, hf_ip_tos_cost, offset + 1, 1, iph.ip_tos);
+       "Header length: %u bytes", hlen);
+
+    if (g_ip_dscp_actif) {
+      tf = proto_tree_add_item_format(ip_tree, hf_ip_dsfield, offset + 1, 1, iph.ip_tos,
+          "Differentiated Services Field: 0x%02x (DSCP 0x%02x: %s)", iph.ip_tos,
+          IPDSFIELD_DSCP(iph.ip_tos), val_to_str(IPDSFIELD_DSCP(iph.ip_tos), dscp_vals,
+          "Unknown DSCP"));
+
+      field_tree = proto_item_add_subtree(tf, ett_ip_dsfield);
+      proto_tree_add_item(field_tree, hf_ip_dsfield_dscp, offset + 1, 1, iph.ip_tos);
+      proto_tree_add_item(field_tree, hf_ip_dsfield_cu, offset + 1, 1, iph.ip_tos);
+    } else {
+      tf = proto_tree_add_item_format(ip_tree, hf_ip_tos, offset + 1, 1, iph.ip_tos,
+         "Type of service: 0x%02x (%s)", iph.ip_tos,
+         val_to_str( IPTOS_TOS(iph.ip_tos), iptos_vals, "Unknown") );
+
+      field_tree = proto_item_add_subtree(tf, ett_ip_tos);
+      proto_tree_add_item(field_tree, hf_ip_tos_precedence, offset + 1, 1, iph.ip_tos);
+      proto_tree_add_item(field_tree, hf_ip_tos_delay, offset + 1, 1, iph.ip_tos);
+      proto_tree_add_item(field_tree, hf_ip_tos_throughput, offset + 1, 1, iph.ip_tos);
+      proto_tree_add_item(field_tree, hf_ip_tos_reliability, offset + 1, 1, iph.ip_tos);
+      proto_tree_add_item(field_tree, hf_ip_tos_cost, offset + 1, 1, iph.ip_tos);
+    }
     proto_tree_add_item(ip_tree, hf_ip_len, offset +  2, 2, iph.ip_len);
     proto_tree_add_item(ip_tree, hf_ip_id, offset +  4, 2, iph.ip_id);
 
     flags = (iph.ip_off & (IP_DF|IP_MF)) >> 12;
     tf = proto_tree_add_item(ip_tree, hf_ip_flags, offset +  6, 1, flags);
-    field_tree = proto_item_add_subtree(tf, ETT_IP_OFF);
+    field_tree = proto_item_add_subtree(tf, ett_ip_off);
     proto_tree_add_item(field_tree, hf_ip_flags_df, offset + 6, 1, flags),
     proto_tree_add_item(field_tree, hf_ip_flags_mf, offset + 6, 1, flags),
 
@@ -852,7 +958,8 @@ dissect_ip(const u_char *pd, int offset, frame_data *fd, proto_tree *tree) {
     proto_tree_add_item(ip_tree, hf_ip_ttl, offset +  8, 1, iph.ip_ttl);
     proto_tree_add_item_format(ip_tree, hf_ip_proto, offset +  9, 1, iph.ip_p,
        "Protocol: %s (0x%02x)", ipprotostr(iph.ip_p), iph.ip_p);
-    proto_tree_add_item(ip_tree, hf_ip_checksum, offset + 10, 2, iph.ip_sum);
+    proto_tree_add_item_format(ip_tree, hf_ip_checksum, offset + 10, 2, iph.ip_sum,
+        "Header checksum: 0x%04x (%s)", iph.ip_sum, ip_checksum_state((e_ip*) &pd[offset]));
     proto_tree_add_item(ip_tree, hf_ip_src, offset + 12, 4, iph.ip_src);
     proto_tree_add_item(ip_tree, hf_ip_dst, offset + 16, 4, iph.ip_dst);
     proto_tree_add_item_hidden(ip_tree, hf_ip_addr, offset + 12, 4, iph.ip_src);
@@ -865,7 +972,7 @@ dissect_ip(const u_char *pd, int offset, frame_data *fd, proto_tree *tree) {
       optlen = hlen - sizeof (e_ip);   /* length of options, in bytes */
       tf = proto_tree_add_text(ip_tree, offset +  20, optlen,
         "Options: (%d bytes)", optlen);
-      field_tree = proto_item_add_subtree(tf, ETT_IP_OPTIONS);
+      field_tree = proto_item_add_subtree(tf, ett_ip_options);
       dissect_ip_tcp_options(&pd[offset + 20], offset + 20, optlen,
          ipopts, N_IP_OPTS, IPOPT_END, field_tree);
     }
@@ -913,6 +1020,9 @@ again:
     case IP_PROTO_OSPF:
       dissect_ospf(pd, offset, fd, tree);
       break;
+    case IP_PROTO_VINES:
+      dissect_vines_frp(pd, offset, fd, tree);
+      break;
     case IP_PROTO_RSVP:
       dissect_rsvp(pd, offset, fd, tree);
       break;
@@ -930,12 +1040,18 @@ again:
     case IP_PROTO_IPV6:
       dissect_ipv6(pd, offset, fd, tree);
       break;
+    case IP_PROTO_IPV4:
+      dissect_ip(pd, offset, fd, tree);
+      break;
     case IP_PROTO_PIM:
       dissect_pim(pd, offset, fd, tree);
       break;
     case IP_PROTO_IPCOMP:
       dissect_ipcomp(pd, offset, fd, tree);
       break;
+    case IP_PROTO_VRRP:
+      dissect_vrrp(pd, offset, fd, tree);
+      break;
     default:
       dissect_data(pd, offset, fd, tree);
       break;
@@ -1071,7 +1187,7 @@ dissect_icmp(const u_char *pd, int offset, frame_data *fd, proto_tree *tree) {
 
   if (tree) {
     ti = proto_tree_add_item(tree, proto_icmp, offset, 4, NULL);
-    icmp_tree = proto_item_add_subtree(ti, ETT_ICMP);
+    icmp_tree = proto_item_add_subtree(ti, ett_icmp);
     proto_tree_add_item_format(icmp_tree, hf_icmp_type, offset,      1, 
                               ih.icmp_type,
                               "Type: %d (%s)",
@@ -1212,7 +1328,7 @@ dissect_igmp(const u_char *pd, int offset, frame_data *fd, proto_tree *tree) {
       strcpy(type_str, "PIM");
       break;
     case IGMP_V2_M_RPT:
-      strcpy(type_str, "Host reponse (v2)");
+      strcpy(type_str, "Host response (v2)");
       break;
     case IGMP_MTRC_RESP:
       strcpy(type_str, "Traceroute response");
@@ -1230,7 +1346,7 @@ dissect_igmp(const u_char *pd, int offset, frame_data *fd, proto_tree *tree) {
     col_add_str(fd, COL_INFO, type_str);
   if (tree) {
     ti = proto_tree_add_item(tree, proto_igmp, offset, 8, NULL);
-    igmp_tree = proto_item_add_subtree(ti, ETT_IGMP);
+    igmp_tree = proto_item_add_subtree(ti, ett_igmp);
     proto_tree_add_item(igmp_tree, hf_igmp_version, offset,     1, 
                        hi_nibble(ih.igmp_v_t));
     proto_tree_add_item_format(igmp_tree, hf_igmp_type,  offset    , 1, 
@@ -1273,9 +1389,13 @@ proto_register_igmp(void)
                { "Group address",      "igmp.group", FT_IPv4, BASE_NONE, NULL, 0x0,
                        "" }},
        };
+       static gint *ett[] = {
+               &ett_igmp,
+       };
 
        proto_igmp = proto_register_protocol ("Internet Group Management Protocol", "igmp");
        proto_register_field_array(proto_igmp, hf, array_length(hf));
+       proto_register_subtree_array(ett, array_length(ett));
 }
 
 void
@@ -1291,6 +1411,20 @@ proto_register_ip(void)
                { "Header Length",      "ip.hdr_len", FT_UINT8, BASE_DEC, NULL, 0x0,
                        "" }},
 
+               { &hf_ip_dsfield,
+               { "Differentiated Services field",      "ip.dsfield", FT_UINT8, BASE_DEC, NULL, 0x0,
+                       "" }},
+
+               { &hf_ip_dsfield_dscp,
+               { "Differentiated Services Codepoint",  "ip.dsfield.dscp", FT_UINT8, BASE_HEX,
+                       VALS(dscp_vals), IPDSFIELD_DSCP_MASK,
+                       "" }},
+
+               { &hf_ip_dsfield_cu,
+               { "Currently Unused",   "ip.dsfield.cu", FT_UINT8, BASE_DEC, NULL,
+                       IPDSFIELD_CU_MASK,
+                       "" }},
+
                { &hf_ip_tos,
                { "Type of Service",    "ip.tos", FT_UINT8, BASE_DEC, NULL, 0x0,
                        "" }},
@@ -1368,9 +1502,20 @@ proto_register_ip(void)
                { "Header checksum",    "ip.checksum", FT_UINT16, BASE_HEX, NULL, 0x0,
                        "" }},
        };
+       static gint *ett[] = {
+               &ett_ip,
+               &ett_ip_dsfield,
+               &ett_ip_tos,
+               &ett_ip_off,
+               &ett_ip_options,
+               &ett_ip_option_sec,
+               &ett_ip_option_route,
+               &ett_ip_option_timestamp,
+       };
 
        proto_ip = proto_register_protocol ("Internet Protocol", "ip");
        proto_register_field_array(proto_ip, hf, array_length(hf));
+       proto_register_subtree_array(ett, array_length(ett));
 }
 
 void
@@ -1390,14 +1535,20 @@ proto_register_icmp(void)
       { "Checksum",    "icmp.checksum",        FT_UINT16, BASE_HEX,    NULL, 0x0,
        "" }},
   };
+  static gint *ett[] = {
+    &ett_icmp,
+  };
   
   proto_icmp = proto_register_protocol ("Internet Control Message Protocol", 
                                        "icmp");
   proto_register_field_array(proto_icmp, hf, array_length(hf));
+  proto_register_subtree_array(ett, array_length(ett));
 }
 
 static int proto_eigrp = -1;
 
+static gint ett_eigrp = -1;
+
 void
 dissect_eigrp(const u_char *pd, int offset, frame_data *fd, proto_tree *tree) {
   e_eigrp     ih;
@@ -1427,7 +1578,7 @@ static const value_string eigrp_opcode_vals[] = {
   if (tree) {
 
      ti = proto_tree_add_item(tree, proto_eigrp, offset, END_OF_FRAME, NULL);
-     eigrp_tree = proto_item_add_subtree(ti, ETT_RIP);
+     eigrp_tree = proto_item_add_subtree(ti, ett_eigrp);
   
      proto_tree_add_text(eigrp_tree, offset, 1, "Version: %d", ih.eigrp_version); 
      proto_tree_add_text(eigrp_tree, offset + 1, 1, "Opcode: %d (%s)", ih.eigrp_opcode,
@@ -1444,5 +1595,9 @@ static const value_string eigrp_opcode_vals[] = {
 void
 proto_register_eigrp(void)
    {
+      static gint *ett[] = {
+        &ett_eigrp,
+      };
    proto_eigrp = proto_register_protocol("Enhanced Interior Gateway Routing Protocol", "eigrp");
+   proto_register_subtree_array(ett, array_length(ett));
    }