From 737add8165cf211ab9072dedd4d73ef563e49c02 Mon Sep 17 00:00:00 2001 From: Guy Harris Date: Wed, 14 Feb 2001 20:03:05 +0000 Subject: Get rid of the "CHECK_DISPLAY_AS_DATA()" call and the setting of "pinfo->current_proto" - this routine is called only through a dissector table, and the code to call through a dissector table does that stuff for you. Clear the Info column before doing anything that could throw an execption, so that if an exception is thrown the display doesn't show junk left over from the protocol above us. Get rid of the GCCism "case N ... M". svn path=/trunk/; revision=3036 --- packet-mip.c | 17 ++++++++++------- 1 file changed, 10 insertions(+), 7 deletions(-) (limited to 'packet-mip.c') diff --git a/packet-mip.c b/packet-mip.c index c94c838328..70eaf54932 100644 --- a/packet-mip.c +++ b/packet-mip.c @@ -2,7 +2,7 @@ * Routines for Mobile IP dissection * Copyright 2000, Stefan Raab * - * $Id: packet-mip.c,v 1.14 2001/02/14 17:01:43 gram Exp $ + * $Id: packet-mip.c,v 1.15 2001/02/14 20:03:05 guy Exp $ * * Ethereal - Network traffic analyzer * By Gerald Combs @@ -136,13 +136,12 @@ dissect_mip( tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree) struct timeval ident_time; int eoffset, elen; - CHECK_DISPLAY_AS_DATA(proto_mip, tvb, pinfo, tree); - /* Make entries in Protocol column and Info column on summary display */ - pinfo->current_proto = "Mobile IP"; if (check_col(pinfo->fd, COL_PROTOCOL)) col_set_str(pinfo->fd, COL_PROTOCOL, "MobileIP"); + if (check_col(pinfo->fd, COL_INFO)) + col_clear(pinfo->fd, COL_INFO); type = tvb_get_guint8(tvb, 0); @@ -185,7 +184,9 @@ dissect_mip( tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree) proto_tree_add_int(ext_tree, hf_mip_ext_len, tvb, eoffset+1, 1, elen); switch (tvb_get_guint8(tvb, eoffset)) { - case 32 ... 34: + case 32: + case 33: + case 34: proto_tree_add_item(ext_tree, hf_mip_aext_spi, tvb, eoffset+2, 4, FALSE); proto_tree_add_item(ext_tree, hf_mip_aext_auth, tvb, eoffset+6, elen-4, FALSE); break; @@ -239,7 +240,9 @@ dissect_mip( tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree) proto_tree_add_int(ext_tree, hf_mip_ext_len, tvb, eoffset+1, 1, elen); switch (tvb_get_guint8(tvb, eoffset)) { - case 32 ... 34: + case 32: + case 33: + case 34: proto_tree_add_item(ext_tree, hf_mip_aext_spi, tvb, eoffset+2, 4, FALSE); proto_tree_add_item(ext_tree, hf_mip_aext_auth, tvb, eoffset+6, elen-4, FALSE); break; @@ -375,5 +378,5 @@ void proto_register_mip(void) void proto_reg_handoff_mip(void) { - dissector_add("udp.port", UDP_PORT_MIP, dissect_mip, proto_mip); + dissector_add("udp.port", UDP_PORT_MIP, dissect_mip, proto_mip); } -- cgit v1.2.3