More updates from Hannes Gredler.
[obnox/wireshark/wip.git] / packet-atalk.c
index 0f2d6bbf69cf37130c55fe725a20192759197a99..9a2bb39acdee59710d3602080df97c95f285a749 100644 (file)
@@ -1,7 +1,7 @@
 /* packet-atalk.c
  * Routines for Appletalk packet disassembly (DDP, currently).
  *
- * $Id: packet-atalk.c,v 1.21 1999/11/04 07:06:50 guy Exp $
+ * $Id: packet-atalk.c,v 1.54 2001/04/27 01:27:36 guy Exp $
  *
  * Simon Wilkinson <sxw@dcs.ed.ac.uk>
  *
 #endif
 
 #include <glib.h>
-#include "globals.h"
 #include "packet.h"
-#include "packet-atalk.h"
+#include "etypes.h"
+#include "ppptypes.h"
+#include "aftypes.h"
+#include "atalk-utils.h"
 
 static int proto_ddp = -1;
 static int hf_ddp_hopcount = -1;
@@ -49,16 +51,49 @@ static int hf_ddp_dst_socket = -1;
 static int hf_ddp_src_socket = -1;
 static int hf_ddp_type = -1;
 
-/* P = Padding, H = Hops, L = Len */
-#if BYTE_ORDER == BIG_ENDIAN
- /* PPHHHHLL LLLLLLLL */
-# define ddp_hops(x)   ( ( x >> 10) & 0x3C )
-# define ddp_len(x)            ( x & 0x03ff )
-#else
- /* LLLLLLLL PPHHHHLL*/
-# define ddp_hops(x)   ( x & 0x3C )
-# define ddp_len(x)            ( ntohs(x) & 0x03ff )
-#endif
+static int proto_nbp = -1;
+static int hf_nbp_op = -1;
+static int hf_nbp_info = -1;
+static int hf_nbp_count = -1;
+static int hf_nbp_tid = -1;
+
+static int hf_nbp_node_net = -1;
+static int hf_nbp_node_port = -1;
+static int hf_nbp_node_node = -1;
+static int hf_nbp_node_enum = -1;
+static int hf_nbp_node_object = -1;
+static int hf_nbp_node_type = -1;
+static int hf_nbp_node_zone = -1;
+
+static int proto_rtmp = -1;
+static int hf_rtmp_net = -1;
+static int hf_rtmp_node_len = -1;
+static int hf_rtmp_node = -1;
+static int hf_rtmp_tuple_net = -1;
+static int hf_rtmp_tuple_range_start = -1;
+static int hf_rtmp_tuple_range_end = -1;
+static int hf_rtmp_tuple_dist = -1;
+static int hf_rtmp_function = -1;
+
+static gint ett_nbp = -1;
+static gint ett_nbp_info = -1;
+static gint ett_nbp_node = -1;
+static gint ett_rtmp = -1;
+static gint ett_rtmp_tuple = -1;
+static gint ett_ddp = -1;
+static gint ett_pstring = -1;
+
+static dissector_table_t ddp_dissector_table;
+
+/*
+ * P = Padding, H = Hops, L = Len
+ *
+ * PPHHHHLL LLLLLLLL
+ *
+ * Assumes the argument is in host byte order.
+ */
+#define ddp_hops(x)    ( ( x >> 10) & 0x3C )
+#define ddp_len(x)             ( x & 0x03ff )
 typedef struct _e_ddp {
   guint16      hops_len; /* combines pad, hops, and len */
   guint16      sum,dnet,snet;
@@ -67,32 +102,8 @@ typedef struct _e_ddp {
   guint8       type;
 } e_ddp;
 
-#define DDP_RTMPDATA   0x01
-#define DDP_NBP                0x02
-#define DDP_ATP                0x03
-#define DDP_AEP                0x04
-#define DDP_RTMPREQ    0x05
-#define DDP_ZIP                0x06
-#define DDP_ADSP       0x07
 #define DDP_HEADER_SIZE 13
 
-gchar *
-atalk_addr_to_str(const struct atalk_ddp_addr *addrp)
-{
-  static gchar str[3][14];
-  static gchar *cur;
-
-  if (cur == &str[0][0]) {
-    cur = &str[1][0];
-  } else if (cur == &str[1][0]) {
-    cur = &str[2][0];
-  } else {
-    cur = &str[0][0];
-  }
-
-  sprintf(cur, "%u.%u:%u", addrp->net, addrp->node, addrp->port);
-  return cur;
-}
 
 static const value_string op_vals[] = {
   {DDP_RTMPDATA, "AppleTalk Routing Table response or data" },
@@ -102,25 +113,266 @@ static const value_string op_vals[] = {
   {DDP_RTMPREQ, "AppleTalk Routing Table request"},
   {DDP_ZIP, "AppleTalk Zone Information Protocol packet"},
   {DDP_ADSP, "AppleTalk Data Stream Protocol"},
+  {DDP_EIGRP, "Cisco EIGRP for AppleTalk"},
   {0, NULL}
 };
 
-void
-dissect_ddp(const u_char *pd, int offset, frame_data *fd, proto_tree *tree) {
+static const value_string rtmp_function_vals[] = {
+  {1, "Request"},
+  {2, "Route Data Request (split horizon processed)"},
+  {3, "Route Data Request (no split horizon processing)"},
+  {0, NULL}
+};
+
+#define NBP_LOOKUP 2
+#define NBP_FORWARD 4
+#define NBP_REPLY 3
+
+static const value_string nbp_op_vals[] = {
+  {NBP_LOOKUP, "lookup"},
+  {NBP_FORWARD, "forward request"},
+  {NBP_REPLY, "reply"},
+  {0, NULL}
+};
+
+/*
+ * XXX - do this with an FT_UINT_STRING?
+ * Unfortunately, you can't extract from an FT_UINT_STRING the string,
+ * which we'd want to do in order to put it into the "Data:" portion.
+ */
+int dissect_pascal_string(tvbuff_t *tvb, int offset, proto_tree *tree,
+       int hf_index)
+{
+       int len;
+       
+       len = tvb_get_guint8(tvb, offset);
+       offset++;
+
+       if ( tree )
+       {
+               char *tmp;
+               proto_tree *item;
+               proto_tree *subtree;
+               
+               /*
+                * XXX - if we could do this inside the protocol tree
+                * code, we could perhaps avoid allocating and freeing
+                * this string buffer.
+                */
+               tmp = g_malloc( len+1 );
+               tvb_memcpy(tvb, tmp, offset, len);
+               tmp[len] = 0;
+               item = proto_tree_add_string(tree, hf_index, tvb, offset-1, len+1, tmp);
+
+               subtree = proto_item_add_subtree(item, ett_pstring);
+               proto_tree_add_text(subtree, tvb, offset-1, 1, "Length: %d", len);
+               proto_tree_add_text(subtree, tvb, offset, len, "Data: %s", tmp);
+               
+               g_free(tmp);
+       }
+       offset += len;
+       
+       return offset;  
+}
+
+static void
+dissect_rtmp_request(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree) {
+  proto_tree *rtmp_tree;
+  proto_item *ti;
+  guint8 function;
+
+  if (check_col(pinfo->fd, COL_PROTOCOL))
+    col_set_str(pinfo->fd, COL_PROTOCOL, "RTMP");
+  if (check_col(pinfo->fd, COL_INFO))
+    col_clear(pinfo->fd, COL_INFO);
+
+  function = tvb_get_guint8(tvb, 0);
+
+  if (check_col(pinfo->fd, COL_INFO))
+    col_add_fstr(pinfo->fd, COL_INFO, "%s",
+       val_to_str(function, rtmp_function_vals, "Unknown function (%02)"));
+  
+  if (tree) {
+    ti = proto_tree_add_item(tree, proto_rtmp, tvb, 0, 1, FALSE);
+    rtmp_tree = proto_item_add_subtree(ti, ett_rtmp);
+
+    proto_tree_add_uint(rtmp_tree, hf_rtmp_function, tvb, 0, 1, function);
+  }
+}
+
+static void
+dissect_rtmp_data(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree) {
+  proto_tree *rtmp_tree;
+  proto_item *ti;
+  int offset = 0;
+  guint16 net;
+  guint8 nodelen,nodelen_bits;
+  guint16 node; /* might be more than 8 bits */
+  int i;
+
+  if (check_col(pinfo->fd, COL_PROTOCOL))
+    col_set_str(pinfo->fd, COL_PROTOCOL, "RTMP");
+  if (check_col(pinfo->fd, COL_INFO))
+    col_clear(pinfo->fd, COL_INFO);
+
+  net = tvb_get_ntohs(tvb, offset);
+  nodelen_bits = tvb_get_guint8(tvb, offset+2);
+  if ( nodelen_bits <= 8 ) {
+    node = tvb_get_guint8(tvb, offset)+1;
+    nodelen = 1;
+  } else {
+    node = tvb_get_ntohs(tvb, offset);
+    nodelen = 2;
+  }
+  
+  if (check_col(pinfo->fd, COL_INFO))
+    col_add_fstr(pinfo->fd, COL_INFO, "Net: %u  Node Len: %u  Node: %u",
+               net, nodelen_bits, node);
+  
+  if (tree) {
+    ti = proto_tree_add_item(tree, proto_rtmp, tvb, offset,
+                            tvb_length_remaining(tvb, offset), FALSE);
+    rtmp_tree = proto_item_add_subtree(ti, ett_rtmp);
+
+    proto_tree_add_uint(rtmp_tree, hf_rtmp_net, tvb, offset, 2, net);
+    proto_tree_add_uint(rtmp_tree, hf_rtmp_node_len, tvb, offset+2, 1,
+                       nodelen_bits);
+    proto_tree_add_uint(rtmp_tree, hf_rtmp_node, tvb, offset+3, nodelen,
+                       node);
+    offset += 3 + nodelen;
+
+    i = 1;
+    while (tvb_offset_exists(tvb, offset)) {
+      proto_tree *tuple_item, *tuple_tree;
+      guint16 tuple_net;
+      guint8 tuple_dist;
+      guint16 tuple_range_end;
+
+      tuple_net = tvb_get_ntohs(tvb, offset);
+      tuple_dist = tvb_get_guint8(tvb, offset+2);
+
+      if (tuple_dist & 0x80) {
+        tuple_range_end = tvb_get_ntohs(tvb, offset+3);
+        tuple_item = proto_tree_add_text(rtmp_tree, tvb, offset, 6,
+                       "Tuple %d:  Range Start: %u  Dist: %u  Range End: %u",
+                       i, tuple_net, tuple_dist&0x7F, tuple_range_end);
+      } else {
+        tuple_item = proto_tree_add_text(rtmp_tree, tvb, offset, 3,
+                       "Tuple %d:  Net: %u  Dist: %u",
+                       i, tuple_net, tuple_dist);
+      }
+      tuple_tree = proto_item_add_subtree(tuple_item, ett_rtmp_tuple);
+
+      if (tuple_dist & 0x80) {
+        proto_tree_add_uint(tuple_tree, hf_rtmp_tuple_range_start, tvb, offset, 2, 
+                       tuple_net);
+      } else {
+        proto_tree_add_uint(tuple_tree, hf_rtmp_tuple_net, tvb, offset, 2, 
+                       tuple_net);
+      }
+      proto_tree_add_uint(tuple_tree, hf_rtmp_tuple_dist, tvb, offset+2, 1,
+                       tuple_dist & 0x7F);
+
+      if (tuple_dist & 0x80) {
+        /*
+         * Extended network tuple.
+         */
+        proto_tree_add_item(tuple_tree, hf_rtmp_tuple_range_end, tvb, offset+3, 2, 
+                               FALSE);
+       offset += 6;
+      } else
+        offset += 3;
+
+      i++;
+    }
+  }
+}
+
+static void
+dissect_nbp(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree) {
+  proto_tree *nbp_tree;
+  proto_tree *nbp_info_tree;
+  proto_item *ti, *info_item;
+  int offset = 0;
+  guint8 info;
+  guint op, count;
+  unsigned int i;
+
+  if (check_col(pinfo->fd, COL_PROTOCOL))
+    col_set_str(pinfo->fd, COL_PROTOCOL, "NBP");
+  if (check_col(pinfo->fd, COL_INFO))
+    col_clear(pinfo->fd, COL_INFO);
+
+  info = tvb_get_guint8(tvb, offset);
+  op = info >> 4;
+  count = info & 0x0F;
+
+  if (check_col(pinfo->fd, COL_INFO))
+    col_add_fstr(pinfo->fd, COL_INFO, "Op: %s  Count: %u",
+      val_to_str(op, nbp_op_vals, "Unknown (0x%01x)"), count);
+  
+  if (tree) {
+    ti = proto_tree_add_item(tree, proto_nbp, tvb, offset,
+               tvb_length_remaining(tvb, offset), FALSE);
+    nbp_tree = proto_item_add_subtree(ti, ett_nbp);
+
+    info_item = proto_tree_add_uint_format(nbp_tree, hf_nbp_info, tvb, offset, 1,
+               info,
+               "Info: 0x%01X  Operation: %s  Count: %u", info,
+               val_to_str(op, nbp_op_vals, "Unknown (0x%01X)"),
+               count);
+    nbp_info_tree = proto_item_add_subtree(info_item, ett_nbp_info);
+    proto_tree_add_uint(nbp_info_tree, hf_nbp_op, tvb, offset, 1, info);
+    proto_tree_add_uint(nbp_info_tree, hf_nbp_count, tvb, offset, 1, info);
+    proto_tree_add_item(nbp_tree, hf_nbp_tid, tvb, offset+1, 1, FALSE);
+    offset += 2;
+
+    for (i=0; i<count; i++) {
+      proto_tree *node_item,*node_tree;
+      int soffset = offset;
+
+      node_item = proto_tree_add_text(nbp_tree, tvb, offset, 4, 
+                       "Node %d", i+1);
+      node_tree = proto_item_add_subtree(node_item, ett_nbp_node);
+
+      proto_tree_add_item(node_tree, hf_nbp_node_net, tvb, offset, 2, FALSE);
+      offset += 2;
+      proto_tree_add_item(node_tree, hf_nbp_node_node, tvb, offset, 1, FALSE);
+      offset++;
+      proto_tree_add_item(node_tree, hf_nbp_node_port, tvb, offset, 1, FALSE);
+      offset++;
+      proto_tree_add_item(node_tree, hf_nbp_node_enum, tvb, offset, 1, FALSE);
+      offset++;
+
+      offset = dissect_pascal_string(tvb, offset, node_tree, hf_nbp_node_object);
+      offset = dissect_pascal_string(tvb, offset, node_tree, hf_nbp_node_type);
+      offset = dissect_pascal_string(tvb, offset, node_tree, hf_nbp_node_zone);
+
+      proto_item_set_len(node_item, offset-soffset);
+    }
+  }
+
+  return;
+}
+
+static void
+dissect_ddp(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree) {
   e_ddp       ddp;
   proto_tree *ddp_tree;
   proto_item *ti;
   static struct atalk_ddp_addr src, dst;
+  tvbuff_t   *new_tvb;
 
-  if (!BYTES_ARE_IN_FRAME(offset, DDP_HEADER_SIZE)) {
-    dissect_data(pd, offset, fd, tree);
-    return;
-  }
+  if (check_col(pinfo->fd, COL_PROTOCOL))
+    col_set_str(pinfo->fd, COL_PROTOCOL, "DDP");
+  if (check_col(pinfo->fd, COL_INFO))
+    col_clear(pinfo->fd, COL_INFO);
 
-  memcpy(&ddp, &pd[offset], sizeof(e_ddp));
+  tvb_memcpy(tvb, (guint8 *)&ddp, 0, sizeof(e_ddp));
   ddp.dnet=ntohs(ddp.dnet);
   ddp.snet=ntohs(ddp.snet);
   ddp.sum=ntohs(ddp.sum);
+  ddp.hops_len=ntohs(ddp.hops_len);
   
   src.net = ddp.snet;
   src.node = ddp.snode;
@@ -128,43 +380,51 @@ dissect_ddp(const u_char *pd, int offset, frame_data *fd, proto_tree *tree) {
   dst.net = ddp.dnet;
   dst.node = ddp.dnode;
   dst.port = ddp.dport;
-  SET_ADDRESS(&pi.net_src, AT_ATALK, sizeof src, (guint8 *)&src);
-  SET_ADDRESS(&pi.src, AT_ATALK, sizeof src, (guint8 *)&src);
-  SET_ADDRESS(&pi.net_dst, AT_ATALK, sizeof dst, (guint8 *)&dst);
-  SET_ADDRESS(&pi.dst, AT_ATALK, sizeof dst, (guint8 *)&dst);
-
-  if (check_col(fd, COL_PROTOCOL))
-    col_add_str(fd, COL_PROTOCOL, "DDP");
-  if (check_col(fd, COL_INFO))
-    col_add_str(fd, COL_INFO,
+  SET_ADDRESS(&pinfo->net_src, AT_ATALK, sizeof src, (guint8 *)&src);
+  SET_ADDRESS(&pinfo->src, AT_ATALK, sizeof src, (guint8 *)&src);
+  SET_ADDRESS(&pinfo->net_dst, AT_ATALK, sizeof dst, (guint8 *)&dst);
+  SET_ADDRESS(&pinfo->dst, AT_ATALK, sizeof dst, (guint8 *)&dst);
+
+  if (check_col(pinfo->fd, COL_INFO))
+    col_add_str(pinfo->fd, COL_INFO,
       val_to_str(ddp.type, op_vals, "Unknown DDP protocol (%02x)"));
   
   if (tree) {
-    ti = proto_tree_add_item(tree, proto_ddp, offset, DDP_HEADER_SIZE, NULL);
-    ddp_tree = proto_item_add_subtree(ti, ETT_DDP);
-    proto_tree_add_item(ddp_tree, hf_ddp_hopcount, offset,      1, 
+    ti = proto_tree_add_item(tree, proto_ddp, tvb, 0, DDP_HEADER_SIZE,
+                            FALSE);
+    ddp_tree = proto_item_add_subtree(ti, ett_ddp);
+    proto_tree_add_uint(ddp_tree, hf_ddp_hopcount,   tvb, 0, 1,
                        ddp_hops(ddp.hops_len));
-    proto_tree_add_item(ddp_tree, hf_ddp_len, offset,      2, 
+    proto_tree_add_uint(ddp_tree, hf_ddp_len,        tvb, 0, 2, 
                        ddp_len(ddp.hops_len));
-    proto_tree_add_item(ddp_tree, hf_ddp_checksum, offset + 2,  2, ddp.sum);
-    proto_tree_add_item(ddp_tree, hf_ddp_dst_net, offset + 4,  2, ddp.dnet);
-    proto_tree_add_item(ddp_tree, hf_ddp_src_net,  offset + 6,  2, ddp.snet);
-    proto_tree_add_item(ddp_tree, hf_ddp_dst_node, offset + 8,  1, ddp.dnode);
-    proto_tree_add_item(ddp_tree, hf_ddp_src_node, offset + 9,  1, ddp.snode);
-    proto_tree_add_item(ddp_tree, hf_ddp_dst_socket, offset + 10, 1, ddp.dport);
-    proto_tree_add_item(ddp_tree, hf_ddp_src_socket, offset + 11, 1, ddp.sport);
-    proto_tree_add_item(ddp_tree, hf_ddp_type, offset + 12, 1, ddp.type);  
+    proto_tree_add_uint(ddp_tree, hf_ddp_checksum,   tvb, 2,  2,
+                       ddp.sum);
+    proto_tree_add_uint(ddp_tree, hf_ddp_dst_net,    tvb, 4,  2,
+                       ddp.dnet);
+    proto_tree_add_uint(ddp_tree, hf_ddp_src_net,    tvb, 6,  2,
+                       ddp.snet);
+    proto_tree_add_uint(ddp_tree, hf_ddp_dst_node,   tvb, 8,  1,
+                       ddp.dnode);
+    proto_tree_add_uint(ddp_tree, hf_ddp_src_node,   tvb, 9,  1,
+                       ddp.snode);
+    proto_tree_add_uint(ddp_tree, hf_ddp_dst_socket, tvb, 10, 1,
+                       ddp.dport);
+    proto_tree_add_uint(ddp_tree, hf_ddp_src_socket, tvb, 11, 1,
+                       ddp.sport);
+    proto_tree_add_uint(ddp_tree, hf_ddp_type,       tvb, 12, 1,
+                       ddp.type);  
   }
 
-  offset += DDP_HEADER_SIZE;
+  new_tvb = tvb_new_subset(tvb, DDP_HEADER_SIZE, -1, -1);
 
-  dissect_data(pd, offset, fd, tree);
+  if (!dissector_try_port(ddp_dissector_table, ddp.type, new_tvb, pinfo, tree))
+    dissect_data(new_tvb, 0, pinfo, tree);
 }
 
 void
 proto_register_atalk(void)
 {
-  static hf_register_info hf[] = {
+  static hf_register_info hf_ddp[] = {
     { &hf_ddp_hopcount,
       { "Hop count",           "ddp.hopcount", FT_UINT8,  BASE_DEC, NULL, 0x0,
        "" }},
@@ -206,6 +466,104 @@ proto_register_atalk(void)
        "" }},
   };
 
-  proto_ddp = proto_register_protocol("Datagram Delivery Protocol", "ddp");
-  proto_register_field_array(proto_ddp, hf, array_length(hf));
+  static hf_register_info hf_nbp[] = {
+    { &hf_nbp_op,
+      { "Operation",           "nbp.op",       FT_UINT8,  BASE_DEC, 
+               VALS(nbp_op_vals), 0xF0, "Operation" }},
+    { &hf_nbp_info,
+      { "Info",                "nbp.info",     FT_UINT8,  BASE_HEX, 
+               NULL, 0x0, "Info" }},
+    { &hf_nbp_count,
+      { "Count",               "nbp.count",    FT_UINT8,  BASE_DEC, 
+               NULL, 0x0F, "Count" }},
+    { &hf_nbp_node_net,
+      { "Network",             "nbp.net",      FT_UINT16,  BASE_DEC, 
+               NULL, 0x0, "Network" }},
+    { &hf_nbp_node_node,
+      { "Node",                "nbp.node",     FT_UINT8,  BASE_DEC, 
+               NULL, 0x0, "Node" }},
+    { &hf_nbp_node_port,
+      { "Port",                "nbp.port",     FT_UINT8,  BASE_DEC, 
+               NULL, 0x0, "Port" }},
+    { &hf_nbp_node_enum,
+      { "Enumerator",          "nbp.enum",     FT_UINT8,  BASE_DEC, 
+               NULL, 0x0, "Enumerator" }},
+    { &hf_nbp_node_object,
+      { "Object",              "nbp.object",   FT_STRING,  BASE_DEC, 
+               NULL, 0x0, "Object" }},
+    { &hf_nbp_node_type,
+      { "Type",                "nbp.type",     FT_STRING,  BASE_DEC, 
+               NULL, 0x0, "Type" }},
+    { &hf_nbp_node_zone,
+      { "Zone",                "nbp.zone",     FT_STRING,  BASE_DEC, 
+               NULL, 0x0, "Zone" }},
+    { &hf_nbp_tid,
+      { "Transaction ID",              "nbp.tid",      FT_UINT8,  BASE_DEC, 
+               NULL, 0x0, "Transaction ID" }}
+  };
+
+  static hf_register_info hf_rtmp[] = {
+    { &hf_rtmp_net,
+      { "Net",         "rtmp.net",     FT_UINT16,  BASE_DEC, 
+               NULL, 0x0, "Net" }},
+    { &hf_rtmp_node,
+      { "Node",                "nbp.nodeid",   FT_UINT8,  BASE_DEC, 
+               NULL, 0x0, "Node" }},
+    { &hf_rtmp_node_len,
+      { "Node Length",         "nbp.nodeid.length",    FT_UINT8,  BASE_DEC, 
+               NULL, 0x0, "Node Length" }},
+    { &hf_rtmp_tuple_net,
+      { "Net",         "rtmp.tuple.net",       FT_UINT16,  BASE_DEC, 
+               NULL, 0x0, "Net" }},
+    { &hf_rtmp_tuple_range_start,
+      { "Range Start",         "rtmp.tuple.range_start",       FT_UINT16,  BASE_DEC, 
+               NULL, 0x0, "Range Start" }},
+    { &hf_rtmp_tuple_range_end,
+      { "Range End",           "rtmp.tuple.range_end", FT_UINT16,  BASE_DEC, 
+               NULL, 0x0, "Range End" }},
+    { &hf_rtmp_tuple_dist,
+      { "Distance",            "rtmp.tuple.dist",      FT_UINT16,  BASE_DEC, 
+               NULL, 0x0, "Distance" }},
+    { &hf_rtmp_function,
+      { "Function",            "rtmp.function",        FT_UINT8,  BASE_DEC, 
+               VALS(rtmp_function_vals), 0x0, "Request Function" }}
+  };
+
+
+  static gint *ett[] = {
+       &ett_ddp,
+       &ett_nbp,
+       &ett_nbp_info,
+       &ett_nbp_node,
+       &ett_pstring,
+       &ett_rtmp,
+       &ett_rtmp_tuple
+  };
+
+  proto_ddp = proto_register_protocol("Datagram Delivery Protocol", "DDP", "ddp");
+  proto_register_field_array(proto_ddp, hf_ddp, array_length(hf_ddp));
+
+  proto_nbp = proto_register_protocol("Name Binding Protocol", "NBP", "nbp");
+  proto_register_field_array(proto_nbp, hf_nbp, array_length(hf_nbp));
+
+  proto_rtmp = proto_register_protocol("Routing Table Maintenance Protocol",
+                                      "RTMP", "rtmp");
+  proto_register_field_array(proto_rtmp, hf_rtmp, array_length(hf_rtmp));
+
+  proto_register_subtree_array(ett, array_length(ett));
+
+  /* subdissector code */
+  ddp_dissector_table = register_dissector_table("ddp.type");
+}
+
+void
+proto_reg_handoff_atalk(void)
+{
+  dissector_add("ethertype", ETHERTYPE_ATALK, dissect_ddp, proto_ddp);
+  dissector_add("chdlctype", ETHERTYPE_ATALK, dissect_ddp, proto_ddp);
+  dissector_add("ppp.protocol", PPP_AT, dissect_ddp, proto_ddp);
+  dissector_add("null.type", BSD_AF_APPLETALK, dissect_ddp, proto_ddp);
+  dissector_add("ddp.type", DDP_NBP, dissect_nbp, proto_nbp);
+  dissector_add("ddp.type", DDP_RTMPREQ, dissect_rtmp_request, proto_rtmp);
+  dissector_add("ddp.type", DDP_RTMPDATA, dissect_rtmp_data, proto_rtmp);
 }