Removed trailing whitespaces from .h and .c files using the
[obnox/wireshark/wip.git] / packet-fddi.c
index feb0cfd13c343d851836197dc090235655dd4fc4..e3ec6a5d9e8125ffaf4afeb49bb220573e862cc5 100644 (file)
@@ -3,12 +3,11 @@
  *
  * Laurent Deniel <deniel@worldnet.fr>
  *
- * $Id: packet-fddi.c,v 1.13 1999/07/08 03:18:19 gram Exp $
+ * $Id: packet-fddi.c,v 1.57 2002/08/02 23:35:49 jmayer Exp $
  *
  * Ethereal - Network traffic analyzer
- * By Gerald Combs <gerald@zing.org>
+ * By Gerald Combs <gerald@ethereal.com>
  * 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
 # include "config.h"
 #endif
 
-#ifdef HAVE_SYS_TYPES_H
-# include <sys/types.h>
-#endif
-
+#include <stdio.h>
+#include <string.h>
 #include <glib.h>
-#include "packet.h"
-#include "resolv.h"
-
-int proto_fddi = -1;
-int hf_fddi_fc = -1;
-int hf_fddi_dst = -1;
-int hf_fddi_dst_vendor = -1;
-int hf_fddi_src = -1;
-int hf_fddi_src_vendor = -1;
+#include <epan/bitswap.h>
+#include <epan/packet.h>
+#include "packet-fddi.h"
+#include "packet-llc.h"
+#include <epan/resolv.h>
+
+static int proto_fddi = -1;
+static int hf_fddi_fc = -1;
+static int hf_fddi_fc_clf = -1;
+static int hf_fddi_fc_prio = -1;
+static int hf_fddi_fc_smt_subtype = -1;
+static int hf_fddi_fc_mac_subtype = -1;
+static int hf_fddi_dst = -1;
+static int hf_fddi_src = -1;
+static int hf_fddi_addr = -1;
+
+static gint ett_fddi = -1;
+static gint ett_fddi_fc = -1;
 
 /* FDDI Frame Control values */
 
@@ -71,6 +77,42 @@ int hf_fddi_src_vendor = -1;
 #define FDDI_FC_IMP_ASYNC_MAX  0x6f
 #define FDDI_FC_IMP_SYNC       0xe0            /* Implementor Synch. */
 
+#define FDDI_FC_CLFF           0xF0            /* Class/Length/Format bits */
+#define FDDI_FC_ZZZZ           0x0F            /* Control bits */
+
+/*
+ * Async frame ZZZZ bits:
+ */
+#define FDDI_FC_ASYNC_R                0x08            /* Reserved */
+#define FDDI_FC_ASYNC_PRI      0x07            /* Priority */
+
+#define CLFF_BITS(fc)  (((fc) & FDDI_FC_CLFF) >> 4)
+#define ZZZZ_BITS(fc)  ((fc) & FDDI_FC_ZZZZ)
+
+static const value_string clf_vals[] = {
+       { CLFF_BITS(FDDI_FC_VOID),      "Void" },
+       { CLFF_BITS(FDDI_FC_SMT),       "SMT" },
+       { CLFF_BITS(FDDI_FC_LLC_ASYNC), "Async LLC" },
+       { CLFF_BITS(FDDI_FC_IMP_ASYNC), "Implementor Async" },
+       { CLFF_BITS(FDDI_FC_NRT),       "Nonrestricted Token" },
+       { CLFF_BITS(FDDI_FC_MAC),       "MAC" },
+       { CLFF_BITS(FDDI_FC_LLC_SYNC),  "Sync LLC" },
+       { CLFF_BITS(FDDI_FC_IMP_SYNC),  "Implementor Sync" },
+       { 0,                            NULL }
+};
+
+static const value_string smt_subtype_vals[] = {
+       { ZZZZ_BITS(FDDI_FC_SMT_INFO), "Info" },
+       { ZZZZ_BITS(FDDI_FC_SMT_NSA),  "Next Station Address" },
+       { 0,                           NULL }
+};
+
+static const value_string mac_subtype_vals[] = {
+       { ZZZZ_BITS(FDDI_FC_MAC_BEACON), "Beacon" },
+       { ZZZZ_BITS(FDDI_FC_MAC_CLAIM),  "Claim" },
+       { 0,                             NULL }
+};
+
 #define FDDI_HEADER_SIZE       13
 
 /* field positions */
@@ -79,78 +121,26 @@ int hf_fddi_src_vendor = -1;
 #define FDDI_P_DHOST           1
 #define FDDI_P_SHOST           7
 
-/* On some systems, the FDDI MAC addresses are bit-swapped. */
-#if !defined(ultrix) && !defined(__alpha) && !defined(__bsdi)
-#define BIT_SWAPPED_MAC_ADDRS
-#endif
-
-#ifdef BIT_SWAPPED_MAC_ADDRS
-/* "swaptab[i]" is the value of "i" with the bits reversed. */
-static u_char swaptab[256] = {
-  0x00, 0x80, 0x40, 0xc0, 0x20, 0xa0, 0x60, 0xe0,
-  0x10, 0x90, 0x50, 0xd0, 0x30, 0xb0, 0x70, 0xf0,
-  0x08, 0x88, 0x48, 0xc8, 0x28, 0xa8, 0x68, 0xe8,
-  0x18, 0x98, 0x58, 0xd8, 0x38, 0xb8, 0x78, 0xf8,
-  0x04, 0x84, 0x44, 0xc4, 0x24, 0xa4, 0x64, 0xe4,
-  0x14, 0x94, 0x54, 0xd4, 0x34, 0xb4, 0x74, 0xf4,
-  0x0c, 0x8c, 0x4c, 0xcc, 0x2c, 0xac, 0x6c, 0xec,
-  0x1c, 0x9c, 0x5c, 0xdc, 0x3c, 0xbc, 0x7c, 0xfc,
-  0x02, 0x82, 0x42, 0xc2, 0x22, 0xa2, 0x62, 0xe2,
-  0x12, 0x92, 0x52, 0xd2, 0x32, 0xb2, 0x72, 0xf2,
-  0x0a, 0x8a, 0x4a, 0xca, 0x2a, 0xaa, 0x6a, 0xea,
-  0x1a, 0x9a, 0x5a, 0xda, 0x3a, 0xba, 0x7a, 0xfa,
-  0x06, 0x86, 0x46, 0xc6, 0x26, 0xa6, 0x66, 0xe6,
-  0x16, 0x96, 0x56, 0xd6, 0x36, 0xb6, 0x76, 0xf6,
-  0x0e, 0x8e, 0x4e, 0xce, 0x2e, 0xae, 0x6e, 0xee,
-  0x1e, 0x9e, 0x5e, 0xde, 0x3e, 0xbe, 0x7e, 0xfe,
-  0x01, 0x81, 0x41, 0xc1, 0x21, 0xa1, 0x61, 0xe1,
-  0x11, 0x91, 0x51, 0xd1, 0x31, 0xb1, 0x71, 0xf1,
-  0x09, 0x89, 0x49, 0xc9, 0x29, 0xa9, 0x69, 0xe9,
-  0x19, 0x99, 0x59, 0xd9, 0x39, 0xb9, 0x79, 0xf9,
-  0x05, 0x85, 0x45, 0xc5, 0x25, 0xa5, 0x65, 0xe5,
-  0x15, 0x95, 0x55, 0xd5, 0x35, 0xb5, 0x75, 0xf5,
-  0x0d, 0x8d, 0x4d, 0xcd, 0x2d, 0xad, 0x6d, 0xed,
-  0x1d, 0x9d, 0x5d, 0xdd, 0x3d, 0xbd, 0x7d, 0xfd,
-  0x03, 0x83, 0x43, 0xc3, 0x23, 0xa3, 0x63, 0xe3,
-  0x13, 0x93, 0x53, 0xd3, 0x33, 0xb3, 0x73, 0xf3,
-  0x0b, 0x8b, 0x4b, 0xcb, 0x2b, 0xab, 0x6b, 0xeb,
-  0x1b, 0x9b, 0x5b, 0xdb, 0x3b, 0xbb, 0x7b, 0xfb,
-  0x07, 0x87, 0x47, 0xc7, 0x27, 0xa7, 0x67, 0xe7,
-  0x17, 0x97, 0x57, 0xd7, 0x37, 0xb7, 0x77, 0xf7,
-  0x0f, 0x8f, 0x4f, 0xcf, 0x2f, 0xaf, 0x6f, 0xef,
-  0x1f, 0x9f, 0x5f, 0xdf, 0x3f, 0xbf, 0x7f, 0xff,
-};
-#endif
-
-static void get_mac_addr(u_char *swapped_addr, const u_char *addr)
-{
-  int i;
-
-  for (i = 0; i < 6; i++) {
-#ifdef BIT_SWAPPED_MAC_ADDRS
-    swapped_addr[i] = swaptab[addr[i]];
-#else
-    swapped_addr[i] = addr[i];
-#endif
-  }
-}
+static dissector_handle_t llc_handle;
+static dissector_handle_t data_handle;
 
 static void
-swap_mac_addr(u_char *swapped_addr, const u_char *orig_addr)
+swap_mac_addr(guchar *swapped_addr, const guchar *orig_addr)
 {
        int i;
 
        for (i = 0; i < 6; i++) {
-               swapped_addr[i] = swaptab[orig_addr[i]];
+               swapped_addr[i] = BIT_SWAP(orig_addr[i]);
        }
 }
 
 
 void
-capture_fddi(const u_char *pd, guint32 cap_len, packet_counts *ld) {
+capture_fddi(const guchar *pd, int len, packet_counts *ld)
+{
   int        offset = 0, fc;
 
-  if (cap_len < FDDI_HEADER_SIZE) {
+  if (!BYTES_ARE_IN_FRAME(0, len, FDDI_HEADER_SIZE)) {
     ld->other++;
     return;
   }
@@ -178,7 +168,7 @@ capture_fddi(const u_char *pd, guint32 cap_len, packet_counts *ld) {
     case FDDI_FC_LLC_ASYNC + 13 :
     case FDDI_FC_LLC_ASYNC + 14 :
     case FDDI_FC_LLC_ASYNC + 15 :
-      capture_llc(pd, offset, cap_len, ld);
+      capture_llc(pd, offset, len, ld);
       return;
     default :
       ld->other++;
@@ -188,64 +178,170 @@ capture_fddi(const u_char *pd, guint32 cap_len, packet_counts *ld) {
 
 } /* capture_fddi */
 
-void dissect_fddi(const u_char *pd, frame_data *fd, proto_tree *tree) 
+static gchar *
+fddifc_to_str(int fc)
 {
-  int        offset = 0, fc;
-  proto_tree *fh_tree;
-  proto_item *ti;
-  u_char     src[6], dst[6];
-  u_char     src_swapped[6], dst_swapped[6];
+  static gchar strbuf[128+1];
 
-  if (fd->cap_len < FDDI_HEADER_SIZE) {
-    dissect_data(pd, offset, fd, tree);
-    return;
+  switch (fc) {
+
+  case FDDI_FC_VOID:                   /* Void frame */
+    return "Void frame";
+
+  case FDDI_FC_NRT:                    /* Nonrestricted token */
+    return "Nonrestricted token";
+
+  case FDDI_FC_RT:                     /* Restricted token */
+    return "Restricted token";
+
+  case FDDI_FC_SMT_INFO:               /* SMT Info */
+    return "SMT info";
+
+  case FDDI_FC_SMT_NSA:                        /* SMT Next station adrs */
+    return "SMT Next station address";
+
+  case FDDI_FC_MAC_BEACON:             /* MAC Beacon frame */
+    return "MAC beacon";
+
+  case FDDI_FC_MAC_CLAIM:              /* MAC Claim frame */
+    return "MAC claim token";
+
+  default:
+    switch (fc & FDDI_FC_CLFF) {
+
+    case FDDI_FC_MAC:
+      sprintf(strbuf, "MAC frame, control %x", fc & FDDI_FC_ZZZZ);
+      return strbuf;
+
+    case FDDI_FC_SMT:
+      sprintf(strbuf, "SMT frame, control %x", fc & FDDI_FC_ZZZZ);
+      return strbuf;
+
+    case FDDI_FC_LLC_ASYNC:
+      if (fc & FDDI_FC_ASYNC_R)
+        sprintf(strbuf, "Async LLC frame, control %x", fc & FDDI_FC_ZZZZ);
+      else
+        sprintf(strbuf, "Async LLC frame, priority %d",
+                       fc & FDDI_FC_ASYNC_PRI);
+      return strbuf;
+
+    case FDDI_FC_LLC_SYNC:
+      if (fc & FDDI_FC_ZZZZ) {
+        sprintf(strbuf, "Sync LLC frame, control %x", fc & FDDI_FC_ZZZZ);
+        return strbuf;
+      } else
+        return "Sync LLC frame";
+
+    case FDDI_FC_IMP_ASYNC:
+      sprintf(strbuf, "Implementor async frame, control %x",
+                       fc & FDDI_FC_ZZZZ);
+      return strbuf;
+
+    case FDDI_FC_IMP_SYNC:
+      sprintf(strbuf, "Implementor sync frame, control %x",
+                       fc & FDDI_FC_ZZZZ);
+      return strbuf;
+      break;
+
+    default:
+      return "Unknown frame type";
+    }
   }
+}
 
-  /* Extract the source and destination addresses, possibly bit-swapping
-     them. */
-  get_mac_addr(dst, (u_char *)&pd[FDDI_P_DHOST]);
-  get_mac_addr(src, (u_char *)&pd[FDDI_P_SHOST]);
 
-  fc = (int) pd[FDDI_P_FC];
+static void
+dissect_fddi(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree,
+               gboolean bitswapped)
+{
+  int        fc;
+  proto_tree *fh_tree = NULL;
+  proto_item *ti;
+  gchar      *fc_str;
+  proto_tree *fc_tree;
+  static guchar src[6], dst[6];
+  guchar     src_swapped[6], dst_swapped[6];
+  tvbuff_t   *next_tvb;
 
-  if (check_col(fd, COL_RES_DL_SRC))
-    col_add_str(fd, COL_RES_DL_SRC, get_ether_name(src));
-  if (check_col(fd, COL_RES_DL_DST))
-    col_add_str(fd, COL_RES_DL_DST, get_ether_name(dst));
-  if (check_col(fd, COL_UNRES_DL_SRC))
-    col_add_str(fd, COL_UNRES_DL_SRC, ether_to_str(src));
-  if (check_col(fd, COL_UNRES_DL_DST))
-    col_add_str(fd, COL_UNRES_DL_DST, ether_to_str(dst));
-  if (check_col(fd, COL_PROTOCOL))
-    col_add_str(fd, COL_PROTOCOL, "N/A");
-  if (check_col(fd, COL_INFO))
-    col_add_str(fd, COL_INFO, "FDDI");
+  if (check_col(pinfo->cinfo, COL_PROTOCOL))
+    col_set_str(pinfo->cinfo, COL_PROTOCOL, "FDDI");
 
-  offset = FDDI_HEADER_SIZE;
+  fc = (int) tvb_get_guint8(tvb, FDDI_P_FC);
+  fc_str = fddifc_to_str(fc);
+
+  if (check_col(pinfo->cinfo, COL_INFO))
+    col_add_str(pinfo->cinfo, COL_INFO, fc_str);
 
   if (tree) {
-       ti = proto_tree_add_item_format(tree, proto_fddi, 0, offset, NULL,
-               "Fiber Distributed Data Interface, %s",
-               (fc >= FDDI_FC_LLC_ASYNC_MIN && fc <= FDDI_FC_LLC_ASYNC_MAX) ?
-                         "Async LLC" : "unsupported FC");
+    ti = proto_tree_add_protocol_format(tree, proto_fddi, tvb, 0, FDDI_HEADER_SIZE,
+               "Fiber Distributed Data Interface, %s", fc_str);
+    fh_tree = proto_item_add_subtree(ti, ett_fddi);
+    ti = proto_tree_add_uint_format(fh_tree, hf_fddi_fc, tvb, FDDI_P_FC, 1, fc,
+        "Frame Control: 0x%02x (%s)", fc, fc_str);
+    fc_tree = proto_item_add_subtree(ti, ett_fddi_fc);
+    proto_tree_add_uint(fc_tree, hf_fddi_fc_clf, tvb, FDDI_P_FC, 1, fc);
+    switch (fc & FDDI_FC_CLFF) {
+
+    case FDDI_FC_SMT:
+      proto_tree_add_uint(fc_tree, hf_fddi_fc_smt_subtype, tvb, FDDI_P_FC, 1, fc);
+      break;
+
+    case FDDI_FC_MAC:
+      if (fc != FDDI_FC_RT)
+        proto_tree_add_uint(fc_tree, hf_fddi_fc_mac_subtype, tvb, FDDI_P_FC, 1, fc);
+      break;
+
+    case FDDI_FC_LLC_ASYNC:
+      if (!(fc & FDDI_FC_ASYNC_R))
+        proto_tree_add_uint(fc_tree, hf_fddi_fc_prio, tvb, FDDI_P_FC, 1, fc);
+      break;
+    }
+  }
+
+  /* Extract the destination address, possibly bit-swapping it. */
+  if (bitswapped)
+    swap_mac_addr(dst, (guchar *) tvb_get_ptr(tvb, FDDI_P_DHOST, 6));
+  else
+    memcpy(dst, (guchar *) tvb_get_ptr(tvb, FDDI_P_DHOST, 6), sizeof dst);
+  swap_mac_addr(dst_swapped, (guchar*) tvb_get_ptr(tvb, FDDI_P_DHOST, 6));
+
+  /* XXX - copy them to some buffer associated with "pi", rather than
+     just making "dst" static? */
+  SET_ADDRESS(&pinfo->dl_dst, AT_ETHER, 6, &dst[0]);
+  SET_ADDRESS(&pinfo->dst, AT_ETHER, 6, &dst[0]);
+
+  if (fh_tree) {
+    proto_tree_add_ether(fh_tree, hf_fddi_dst, tvb, FDDI_P_DHOST, 6, dst);
+    proto_tree_add_ether_hidden(fh_tree, hf_fddi_addr, tvb, FDDI_P_DHOST, 6, dst);
+
+    /* hide some bit-swapped mac address fields in the proto_tree, just in case */
+    proto_tree_add_ether_hidden(fh_tree, hf_fddi_dst, tvb, FDDI_P_DHOST, 6, dst_swapped);
+    proto_tree_add_ether_hidden(fh_tree, hf_fddi_addr, tvb, FDDI_P_DHOST, 6, dst_swapped);
+  }
 
-      swap_mac_addr(dst_swapped, (u_char*)&pd[FDDI_P_DHOST]);
-      swap_mac_addr(src_swapped, (u_char*)&pd[FDDI_P_SHOST]);
+  /* Extract the source address, possibly bit-swapping it. */
+  if (bitswapped)
+    swap_mac_addr(src, (guchar *) tvb_get_ptr(tvb, FDDI_P_SHOST, 6));
+  else
+    memcpy(src, (guchar *) tvb_get_ptr(tvb, FDDI_P_SHOST, 6), sizeof src);
+  swap_mac_addr(src_swapped, (guchar*) tvb_get_ptr(tvb, FDDI_P_SHOST, 6));
 
-      fh_tree = proto_item_add_subtree(ti, ETT_FDDI);
-      proto_tree_add_item(fh_tree, hf_fddi_fc, FDDI_P_FC, 1, fc);
-      proto_tree_add_item(fh_tree, hf_fddi_dst, FDDI_P_DHOST, 6, dst);
-      proto_tree_add_item_hidden(fh_tree, hf_fddi_dst_vendor, FDDI_P_DHOST, 3, dst);
-      proto_tree_add_item(fh_tree, hf_fddi_src, FDDI_P_SHOST, 6, src);
-      proto_tree_add_item_hidden(fh_tree, hf_fddi_src_vendor, FDDI_P_SHOST, 3, src);
+  /* XXX - copy them to some buffer associated with "pi", rather than
+     just making "src" static? */
+  SET_ADDRESS(&pinfo->dl_src, AT_ETHER, 6, &src[0]);
+  SET_ADDRESS(&pinfo->src, AT_ETHER, 6, &src[0]);
+
+  if (fh_tree) {
+      proto_tree_add_ether(fh_tree, hf_fddi_src, tvb, FDDI_P_SHOST, 6, src);
+      proto_tree_add_ether_hidden(fh_tree, hf_fddi_addr, tvb, FDDI_P_SHOST, 6, src);
 
       /* hide some bit-swapped mac address fields in the proto_tree, just in case */
-      proto_tree_add_item_hidden(fh_tree, hf_fddi_dst, FDDI_P_DHOST, 6, dst_swapped);
-      proto_tree_add_item_hidden(fh_tree, hf_fddi_dst, FDDI_P_SHOST, 6, src_swapped);
-      proto_tree_add_item_hidden(fh_tree, hf_fddi_dst_vendor, FDDI_P_DHOST, 3, dst_swapped);
-      proto_tree_add_item_hidden(fh_tree, hf_fddi_src_vendor, FDDI_P_SHOST, 3, src_swapped);
+      proto_tree_add_ether_hidden(fh_tree, hf_fddi_src, tvb, FDDI_P_SHOST, 6, src_swapped);
+      proto_tree_add_ether_hidden(fh_tree, hf_fddi_addr, tvb, FDDI_P_SHOST, 6, src_swapped);
+  }
+
+  next_tvb = tvb_new_subset(tvb, FDDI_HEADER_SIZE, -1, -1);
 
-    }
   switch (fc) {
 
     /* From now, only 802.2 SNAP (Async. LCC frame) is supported */
@@ -266,29 +362,104 @@ void dissect_fddi(const u_char *pd, frame_data *fd, proto_tree *tree)
     case FDDI_FC_LLC_ASYNC + 13 :
     case FDDI_FC_LLC_ASYNC + 14 :
     case FDDI_FC_LLC_ASYNC + 15 :
-      dissect_llc(pd, offset, fd, tree);
+      call_dissector(llc_handle, next_tvb, pinfo, tree);
       return;
       
     default :
-      dissect_data(pd, offset, fd, tree);
+      call_dissector(data_handle,next_tvb, pinfo, tree);
       return;
 
   } /* fc */
 } /* dissect_fddi */
 
+       
+static void
+dissect_fddi_bitswapped(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree)
+{
+       dissect_fddi(tvb, pinfo, tree, TRUE);
+}
+
+static void
+dissect_fddi_not_bitswapped(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree)
+{
+       dissect_fddi(tvb, pinfo, tree, FALSE);
+}
+
 void
 proto_register_fddi(void)
 {
-       const hf_register_info hf[] = {
-               { "Frame Control",      "fddi.fc", &hf_fddi_fc, FT_UINT8, NULL },
-               { "Destination",        "fddi.dst", &hf_fddi_dst, FT_ETHER, NULL },
-               { "Source",             "fddi.src", &hf_fddi_src, FT_ETHER, NULL },
-               { "Destination Hardware Vendor",
-                                       "fddi.dst_vendor", &hf_fddi_dst_vendor, FT_ETHER_VENDOR, NULL },
-               { "Source Hardware Vendor",
-                                       "fddi.src_vendor", &hf_fddi_src_vendor, FT_ETHER_VENDOR, NULL }
+       static hf_register_info hf[] = {
+
+               /*
+                * XXX - we want this guy to have his own private formatting
+                * routine, using "fc_to_str()"; if "fc_to_str()" returns
+                * NULL, just show the hex value, else show the string.
+                */
+               { &hf_fddi_fc,
+               { "Frame Control",      "fddi.fc", FT_UINT8, BASE_HEX, NULL, 0x0,
+                       "", HFILL }},
+
+               { &hf_fddi_fc_clf,
+               { "Class/Length/Format", "fddi.fc.clf", FT_UINT8, BASE_HEX, VALS(clf_vals), FDDI_FC_CLFF,
+                       "", HFILL }},
+
+               { &hf_fddi_fc_prio,
+               { "Priority", "fddi.fc.prio", FT_UINT8, BASE_DEC, NULL, FDDI_FC_ASYNC_PRI,
+                       "", HFILL }},
+
+               { &hf_fddi_fc_smt_subtype,
+               { "SMT Subtype", "fddi.fc.smt_subtype", FT_UINT8, BASE_DEC, VALS(smt_subtype_vals), FDDI_FC_ZZZZ,
+                       "", HFILL }},
+
+               { &hf_fddi_fc_mac_subtype,
+               { "MAC Subtype", "fddi.fc.mac_subtype", FT_UINT8, BASE_DEC, VALS(mac_subtype_vals), FDDI_FC_ZZZZ,
+                       "", HFILL }},
+
+               { &hf_fddi_dst,
+               { "Destination",        "fddi.dst", FT_ETHER, BASE_NONE, NULL, 0x0,
+                       "Destination Hardware Address", HFILL }},
+
+               { &hf_fddi_src,
+               { "Source",             "fddi.src", FT_ETHER, BASE_NONE, NULL, 0x0,
+                       "", HFILL }},
+
+               { &hf_fddi_addr,
+               { "Source or Destination Address", "fddi.addr", FT_ETHER, BASE_NONE, NULL, 0x0,
+                       "Source or Destination Hardware Address", HFILL }},
+
+       };
+       static gint *ett[] = {
+               &ett_fddi,
+               &ett_fddi_fc,
        };
 
-       proto_fddi = proto_register_protocol ("Fiber Distributed Data Interface", "fddi" );
+       proto_fddi = proto_register_protocol("Fiber Distributed Data Interface",
+           "FDDI", "fddi");
        proto_register_field_array(proto_fddi, hf, array_length(hf));
+       proto_register_subtree_array(ett, array_length(ett));
+
+       /*
+        * Called from various dissectors for encapsulated FDDI frames.
+        * We assume the MAC addresses in them aren't bitswapped.
+        */
+       register_dissector("fddi", dissect_fddi_not_bitswapped, proto_fddi);
+}
+
+void
+proto_reg_handoff_fddi(void)
+{
+       dissector_handle_t fddi_handle, fddi_bitswapped_handle;
+
+       /*
+        * Get a handle for the LLC dissector.
+        */
+       llc_handle = find_dissector("llc");
+       data_handle = find_dissector("data");
+
+       fddi_handle = find_dissector("fddi");
+       dissector_add("wtap_encap", WTAP_ENCAP_FDDI, fddi_handle);
+       fddi_bitswapped_handle =
+           create_dissector_handle(dissect_fddi_bitswapped, proto_fddi);
+       dissector_add("wtap_encap", WTAP_ENCAP_FDDI_BITSWAPPED,
+           fddi_bitswapped_handle);
 }