sensitivity of packet range options fine tuning:
[obnox/wireshark/wip.git] / packet-v120.c
index c0a1223aa14363e61f8f90ac3d2e89e9a4bcf80c..25dc238d4e3ae3bd8cf872c04bf4ed901439d2e3 100644 (file)
@@ -2,23 +2,22 @@
  * Routines for v120 frame disassembly
  * Bert Driehuis <driehuis@playbeing.org>
  *
- * $Id: packet-v120.c,v 1.15 2000/11/29 05:16:15 gram Exp $
+ * $Id: packet-v120.c,v 1.31 2003/09/02 19:18:52 guy Exp $
  *
  * Ethereal - Network traffic analyzer
- * By Gerald Combs <gerald@zing.org>
+ * By Gerald Combs <gerald@ethereal.com>
  * Copyright 1998
  *
- * 
  * 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.
 # include "config.h"
 #endif
 
-#ifdef HAVE_SYS_TYPES_H
-# include <sys/types.h>
-#endif
-
 #include <stdio.h>
 #include <glib.h>
 #include <string.h>
-#include "packet.h"
+#include <epan/packet.h>
 #include "xdlc.h"
 
-#define FROM_DCE       0x80
-
 static int proto_v120 = -1;
 static int hf_v120_address = -1;
 static int hf_v120_control = -1;
@@ -50,7 +43,9 @@ static gint ett_v120_address = -1;
 static gint ett_v120_control = -1;
 static gint ett_v120_header = -1;
 
-static int dissect_v120_header(tvbuff_t *tvb, int offset, packet_info *pinfo, proto_tree *tree);
+static dissector_handle_t data_handle;
+
+static int dissect_v120_header(tvbuff_t *tvb, int offset, proto_tree *tree);
 
 static void
 dissect_v120(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree)
@@ -65,51 +60,45 @@ dissect_v120(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree)
     guint16    control;
     tvbuff_t   *next_tvb;
 
-    CHECK_DISPLAY_AS_DATA(proto_v120, tvb, pinfo, tree);
-
-    pinfo->current_proto = "V.120";
-
-    if (check_col(pinfo->fd, COL_PROTOCOL))
-       col_set_str(pinfo->fd, COL_PROTOCOL, "V.120");
+    if (check_col(pinfo->cinfo, COL_PROTOCOL))
+       col_set_str(pinfo->cinfo, COL_PROTOCOL, "V.120");
+    if (check_col(pinfo->cinfo, COL_INFO))
+       col_clear(pinfo->cinfo, COL_INFO);
 
     byte0 = tvb_get_guint8(tvb, 0);
 
-    if(check_col(pinfo->fd, COL_RES_DL_SRC))
-       col_add_fstr(pinfo->fd, COL_RES_DL_SRC, "0x%02X", byte0);
+    if(check_col(pinfo->cinfo, COL_RES_DL_SRC))
+       col_add_fstr(pinfo->cinfo, COL_RES_DL_SRC, "0x%02X", byte0);
 
     byte1 = tvb_get_guint8(tvb, 1);
 
     if ((byte0 & 0x01) != 0x00 && (byte1 && 0x01) != 0x01)
     {
-       if (check_col(pinfo->fd, COL_INFO))
-           col_set_str(pinfo->fd, COL_INFO, "Invalid V.120 frame");
+       if (check_col(pinfo->cinfo, COL_INFO))
+           col_set_str(pinfo->cinfo, COL_INFO, "Invalid V.120 frame");
        if (tree)
-           ti = proto_tree_add_protocol_format(tree, proto_v120, tvb, 0, tvb_length(tvb),
+           ti = proto_tree_add_protocol_format(tree, proto_v120, tvb, 0, -1,
                                            "Invalid V.120 frame");
        return;
     }
 
-    if (pinfo->pseudo_header->x25.flags & FROM_DCE) {
-       if(check_col(pinfo->fd, COL_RES_DL_DST))
-           col_set_str(pinfo->fd, COL_RES_DL_DST, "DTE");
-       if(check_col(pinfo->fd, COL_RES_DL_SRC))
-           col_set_str(pinfo->fd, COL_RES_DL_SRC, "DCE");
-    }
-    else {
-       if(check_col(pinfo->fd, COL_RES_DL_DST))
-           col_set_str(pinfo->fd, COL_RES_DL_DST, "DCE");
-       if(check_col(pinfo->fd, COL_RES_DL_SRC))
-           col_set_str(pinfo->fd, COL_RES_DL_SRC, "DTE");
+    if (pinfo->p2p_dir == P2P_DIR_SENT) {
+       is_response = (byte0 & 0x02) ? FALSE: TRUE;
+       if(check_col(pinfo->cinfo, COL_RES_DL_DST))
+           col_set_str(pinfo->cinfo, COL_RES_DL_DST, "DCE");
+       if(check_col(pinfo->cinfo, COL_RES_DL_SRC))
+           col_set_str(pinfo->cinfo, COL_RES_DL_SRC, "DTE");
+    } else {
+       /* XXX - what if the direction is unknown? */
+       is_response = (byte0 & 0x02) ? TRUE : FALSE;
+       if(check_col(pinfo->cinfo, COL_RES_DL_DST))
+           col_set_str(pinfo->cinfo, COL_RES_DL_DST, "DTE");
+       if(check_col(pinfo->cinfo, COL_RES_DL_SRC))
+           col_set_str(pinfo->cinfo, COL_RES_DL_SRC, "DCE");
     }
 
-    if (((pinfo->pseudo_header->x25.flags & FROM_DCE) && byte0 & 0x02) ||
-       (!(pinfo->pseudo_header->x25.flags & FROM_DCE) && !(byte0 & 0x02)))
-       is_response = TRUE;
-    else
-       is_response = FALSE;
-
     if (tree) {
-       ti = proto_tree_add_protocol_format(tree, proto_v120, tvb, 0, 0, "V.120");
+       ti = proto_tree_add_protocol_format(tree, proto_v120, tvb, 0, -1, "V.120");
        v120_tree = proto_item_add_subtree(ti, ett_v120);
        addr = byte1 << 8 | byte0;
        sprintf(info, "LLI: %d C/R: %s",
@@ -137,19 +126,19 @@ dissect_v120(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree)
        ti = NULL;
     }
     control = dissect_xdlc_control(tvb, 2, pinfo, v120_tree, hf_v120_control,
-           ett_v120_control, is_response, TRUE);
+           ett_v120_control, is_response, TRUE, FALSE);
     if (tree) {
        v120len = 2 + XDLC_CONTROL_LEN(control, TRUE);
        if (tvb_bytes_exist(tvb, v120len, 1))
-               v120len += dissect_v120_header(tvb, v120len, pinfo, v120_tree);
+               v120len += dissect_v120_header(tvb, v120len, v120_tree);
        proto_item_set_len(ti, v120len);
        next_tvb = tvb_new_subset(tvb, v120len, -1, -1);
-       dissect_data(next_tvb, 0, pinfo, v120_tree);
+       call_dissector(data_handle,next_tvb, pinfo, v120_tree);
     }
 }
 
 static int
-dissect_v120_header(tvbuff_t *tvb, int offset, packet_info *pinfo, proto_tree *tree)
+dissect_v120_header(tvbuff_t *tvb, int offset, proto_tree *tree)
 {
        char info[80];
        int header_len, nbits;
@@ -211,13 +200,13 @@ proto_register_v120(void)
     static hf_register_info hf[] = {
        { &hf_v120_address,
          { "Link Address", "v120.address", FT_UINT16, BASE_HEX, NULL,
-                 0x0, "" }},
+                 0x0, "", HFILL }},
        { &hf_v120_control,
          { "Control Field", "v120.control", FT_UINT16, BASE_HEX, NULL, 0x0,
-               "" }},
+               "", HFILL }},
        { &hf_v120_header,
          { "Header Field", "v120.header", FT_STRING, BASE_NONE, NULL, 0x0,
-               "" }},
+               "", HFILL }},
     };
     static gint *ett[] = {
         &ett_v120,
@@ -226,13 +215,16 @@ proto_register_v120(void)
         &ett_v120_header,
     };
 
-    proto_v120 = proto_register_protocol ("Async data over ISDN (V.120)", "v120");
+    proto_v120 = proto_register_protocol("Async data over ISDN (V.120)",
+                                        "V.120", "v120");
     proto_register_field_array (proto_v120, hf, array_length(hf));
     proto_register_subtree_array(ett, array_length(ett));
+
+    register_dissector("v120", dissect_v120, proto_v120);
 }
 
 void
 proto_reg_handoff_v120(void)
 {
-       dissector_add("wtap_encap", WTAP_ENCAP_V120, dissect_v120);
+    data_handle = find_dissector("data");
 }