diff options
author | guy <guy@f5534014-38df-0310-8fa8-9805f1628bb7> | 2000-11-15 08:27:14 +0000 |
---|---|---|
committer | guy <guy@f5534014-38df-0310-8fa8-9805f1628bb7> | 2000-11-15 08:27:14 +0000 |
commit | d37d0b1017e4f57a92bd00f1cbb183301ed714cc (patch) | |
tree | f3a59c65b1986e8e8de5fcbac117ed242125b30a /packet-ipp.c | |
parent | 021246c5e20ba3d33202cca602cc2144a8bf7dd2 (diff) |
Tvbuffify the IPP dissector, and have it register itself rather than
being a global function.
The HTTP dissector should set "pinfo->current_proto" to HTTP even if we
consider the packet to be IPP, so that if we run past the end of a
tvbuff while dissecting HTTP stuff it's reported as a problem with HTTP,
not IPP.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@2648 f5534014-38df-0310-8fa8-9805f1628bb7
Diffstat (limited to 'packet-ipp.c')
-rw-r--r-- | packet-ipp.c | 195 |
1 files changed, 96 insertions, 99 deletions
diff --git a/packet-ipp.c b/packet-ipp.c index 378d2de541..4bd9fab7e0 100644 --- a/packet-ipp.c +++ b/packet-ipp.c @@ -3,7 +3,7 @@ * * Guy Harris <guy@alum.mit.edu> * - * $Id: packet-ipp.c,v 1.14 2000/11/13 07:18:46 guy Exp $ + * $Id: packet-ipp.c,v 1.15 2000/11/15 08:27:14 guy Exp $ * * Ethereal - Network traffic analyzer * By Gerald Combs <gerald@zing.org> @@ -143,56 +143,61 @@ static const value_string status_vals[] = { { 0, NULL } }; -static int parse_attributes(const u_char *pd, int offset, frame_data *fd, - proto_tree *tree); -static proto_tree *add_integer_tree(proto_tree *tree, const u_char *pd, +static int parse_attributes(tvbuff_t *tvb, int offset, proto_tree *tree); +static proto_tree *add_integer_tree(proto_tree *tree, tvbuff_t *tvb, int offset, int name_length, int value_length); static void add_integer_value(guint tag, gchar *tag_desc, proto_tree *tree, - const u_char *pd, int offset, int name_length, int value_length); -static proto_tree *add_octetstring_tree(proto_tree *tree, const u_char *pd, + tvbuff_t *tvb, int offset, int name_length, int value_length); +static proto_tree *add_octetstring_tree(proto_tree *tree, tvbuff_t *tvb, int offset, int name_length, int value_length); static void add_octetstring_value(guint tag, gchar *tag_desc, proto_tree *tree, - const u_char *pd, int offset, int name_length, int value_length); -static proto_tree *add_charstring_tree(proto_tree *tree, const u_char *pd, + tvbuff_t *tvb, int offset, int name_length, int value_length); +static proto_tree *add_charstring_tree(proto_tree *tree, tvbuff_t *tvb, int offset, int name_length, int value_length); static void add_charstring_value(guint tag, gchar *tag_desc, proto_tree *tree, - const u_char *pd, int offset, int name_length, int value_length); + tvbuff_t *tvb, int offset, int name_length, int value_length); static int add_value_head(guint tag, gchar *tag_desc, proto_tree *tree, - const u_char *pd, int offset, int name_length, int value_length); + tvbuff_t *tvb, int offset, int name_length, int value_length); -void dissect_ipp(const u_char *pd, int offset, frame_data *fd, proto_tree *tree) +static void +dissect_ipp(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree) { proto_tree *ipp_tree; proto_item *ti; - gboolean is_request = (pi.destport == 631); + int offset = 0; + gboolean is_request = (pinfo->destport == 631); guint16 status_code; gchar *status_fmt; - OLD_CHECK_DISPLAY_AS_DATA(proto_ipp, pd, offset, fd, tree); + CHECK_DISPLAY_AS_DATA(proto_ipp, tvb, pinfo, tree); + + pinfo->current_proto = "IPP"; - if (check_col(fd, COL_PROTOCOL)) - col_add_str(fd, COL_PROTOCOL, "IPP"); - if (check_col(fd, COL_INFO)) { + if (check_col(pinfo->fd, COL_PROTOCOL)) + col_add_str(pinfo->fd, COL_PROTOCOL, "IPP"); + if (check_col(pinfo->fd, COL_INFO)) { if (is_request) - col_add_str(fd, COL_INFO, "IPP request"); + col_add_str(pinfo->fd, COL_INFO, "IPP request"); else - col_add_str(fd, COL_INFO, "IPP response"); + col_add_str(pinfo->fd, COL_INFO, "IPP response"); } if (tree) { - ti = proto_tree_add_item(tree, proto_ipp, NullTVB, offset, END_OF_FRAME, FALSE); + ti = proto_tree_add_item(tree, proto_ipp, tvb, offset, + tvb_length_remaining(tvb, offset), FALSE); ipp_tree = proto_item_add_subtree(ti, ett_ipp); - proto_tree_add_text(ipp_tree, NullTVB, offset, 2, "Version: %u.%u", - pd[offset], pd[offset + 1]); + proto_tree_add_text(ipp_tree, tvb, offset, 2, "Version: %u.%u", + tvb_get_guint8(tvb, offset), + tvb_get_guint8(tvb, offset + 1)); offset += 2; if (is_request) { - proto_tree_add_text(ipp_tree, NullTVB, offset, 2, "Operation-id: %s", - val_to_str(pntohs(&pd[offset]), operation_vals, + proto_tree_add_text(ipp_tree, tvb, offset, 2, "Operation-id: %s", + val_to_str(tvb_get_ntohs(tvb, offset), operation_vals, "Unknown (0x%04x)")); } else { - status_code = pntohs(&pd[offset]); + status_code = tvb_get_ntohs(tvb, offset); switch (status_code & STATUS_TYPE_MASK) { case STATUS_SUCCESSFUL: @@ -219,19 +224,23 @@ void dissect_ipp(const u_char *pd, int offset, frame_data *fd, proto_tree *tree) status_fmt = "Unknown (0x%04x)"; break; } - proto_tree_add_text(ipp_tree, NullTVB, offset, 2, "Status-code: %s", + proto_tree_add_text(ipp_tree, tvb, offset, 2, "Status-code: %s", val_to_str(status_code, status_vals, status_fmt)); } offset += 2; - proto_tree_add_text(ipp_tree, NullTVB, offset, 4, "Request ID: %u", - pntohl(&pd[offset])); + proto_tree_add_text(ipp_tree, tvb, offset, 4, "Request ID: %u", + tvb_get_ntohl(tvb, offset)); offset += 4; - offset = parse_attributes(pd, offset, fd, ipp_tree); + offset = parse_attributes(tvb, offset, ipp_tree); - if (IS_DATA_IN_FRAME(offset)) - old_dissect_data(pd, offset, fd, ipp_tree); + if (tvb_offset_exists(tvb, offset)) { + proto_tree_add_text(ipp_tree, tvb, offset, + tvb_length_remaining(tvb, offset), + "Data (%d bytes)", + tvb_length_remaining(tvb, offset)); + } } } @@ -296,7 +305,7 @@ static const value_string tag_vals[] = { }; static int -parse_attributes(const u_char *pd, int offset, frame_data *fd, proto_tree *tree) +parse_attributes(tvbuff_t *tvb, int offset, proto_tree *tree) { guint8 tag; gchar *tag_desc; @@ -306,8 +315,8 @@ parse_attributes(const u_char *pd, int offset, frame_data *fd, proto_tree *tree) int start_offset = offset; proto_tree *attr_tree = tree; - while (IS_DATA_IN_FRAME(offset)) { - tag = pd[offset]; + while (tvb_offset_exists(tvb, offset)) { + tag = tvb_get_guint8(tvb, offset); tag_desc = val_to_str(tag, tag_vals, "Reserved (0x%02x)"); if (TAG_TYPE(tag) == TAG_TYPE_DELIMITER) { /* @@ -338,7 +347,7 @@ parse_attributes(const u_char *pd, int offset, frame_data *fd, proto_tree *tree) /* * Now create a new item for this tag. */ - tas = proto_tree_add_text(tree, NullTVB, offset, 1, + tas = proto_tree_add_text(tree, tvb, offset, 1, "%s", tag_desc); offset++; if (tag == TAG_END_OF_ATTRIBUTES) { @@ -351,42 +360,17 @@ parse_attributes(const u_char *pd, int offset, frame_data *fd, proto_tree *tree) /* * Value tag - get the name length. */ - if (!BYTES_ARE_IN_FRAME(offset + 1, 2)) { - /* - * We ran past the end of the frame. - * Quit (we need to be able to handle - * stuff that crosses frames to do more) - */ - break; - } - name_length = pntohs(&pd[offset + 1]); + name_length = tvb_get_ntohs(tvb, offset + 1); /* * OK, get the value length. */ - if (!BYTES_ARE_IN_FRAME(offset + 1 + 2, name_length)) { - /* - * We ran past the end of the frame. - * Quit (we need to be able to handle - * stuff that crosses frames to do more) - */ - break; - } - value_length = pntohs(&pd[offset + 1 + 2 + name_length]); + value_length = tvb_get_ntohs(tvb, offset + 1 + 2 + name_length); /* * OK, does the value run past the end of the * frame? */ - if (!BYTES_ARE_IN_FRAME(offset + 1 + 2 + name_length + 2, - value_length)) { - /* - * We ran past the end of the frame. - * Quit (we need to be able to handle - * stuff that crosses frames to do more) - */ - break; - } if (as_tree == NULL) { /* * OK, there's an attribute to hang @@ -409,10 +393,10 @@ parse_attributes(const u_char *pd, int offset, frame_data *fd, proto_tree *tree) * start a tree for it. */ attr_tree = add_integer_tree(as_tree, - pd, offset, name_length, + tvb, offset, name_length, value_length); } - add_integer_value(tag, tag_desc, attr_tree, pd, + add_integer_value(tag, tag_desc, attr_tree, tvb, offset, name_length, value_length); break; @@ -424,11 +408,11 @@ parse_attributes(const u_char *pd, int offset, frame_data *fd, proto_tree *tree) * start a tree for it. */ attr_tree = add_octetstring_tree(as_tree, - pd, offset, name_length, + tvb, offset, name_length, value_length); } add_octetstring_value(tag, tag_desc, - attr_tree, pd, offset, name_length, + attr_tree, tvb, offset, name_length, value_length); break; @@ -440,11 +424,11 @@ parse_attributes(const u_char *pd, int offset, frame_data *fd, proto_tree *tree) * start a tree for it. */ attr_tree = add_charstring_tree(as_tree, - pd, offset, name_length, + tvb, offset, name_length, value_length); } add_charstring_value(tag, tag_desc, - attr_tree, pd, offset, name_length, + attr_tree, tvb, offset, name_length, value_length); break; } @@ -456,103 +440,108 @@ parse_attributes(const u_char *pd, int offset, frame_data *fd, proto_tree *tree) } static proto_tree * -add_integer_tree(proto_tree *tree, const u_char *pd, int offset, +add_integer_tree(proto_tree *tree, tvbuff_t *tvb, int offset, int name_length, int value_length) { proto_item *ti; if (value_length != 4) { - ti = proto_tree_add_text(tree, NullTVB, offset, + ti = proto_tree_add_text(tree, tvb, offset, 1 + 2 + name_length + 2 + value_length, "%.*s: Invalid integer (length is %u, should be 4)", - name_length, &pd[offset + 1 + 2], + name_length, + tvb_get_ptr(tvb, offset + 1 + 2, name_length), value_length); } else { - ti = proto_tree_add_text(tree, NullTVB, offset, + ti = proto_tree_add_text(tree, tvb, offset, 1 + 2 + name_length + 2 + value_length, "%.*s: %u", - name_length, &pd[offset + 1 + 2], - pntohl(&pd[offset + 1 + 2 + name_length + 2])); + name_length, + tvb_get_ptr(tvb, offset + 1 + 2, name_length), + tvb_get_ntohl(tvb, offset + 1 + 2 + name_length + 2)); } return proto_item_add_subtree(ti, ett_ipp_attr); } static void add_integer_value(guint tag, gchar *tag_desc, proto_tree *tree, - const u_char *pd, int offset, int name_length, int value_length) + tvbuff_t *tvb, int offset, int name_length, int value_length) { - offset = add_value_head(tag, tag_desc, tree, pd, offset, + offset = add_value_head(tag, tag_desc, tree, tvb, offset, name_length, value_length); if (value_length == 4) { - proto_tree_add_text(tree, NullTVB, offset, value_length, - "Value: %u", pntohl(&pd[offset])); + proto_tree_add_text(tree, tvb, offset, value_length, + "Value: %u", tvb_get_ntohl(tvb, offset)); } } static proto_tree * -add_octetstring_tree(proto_tree *tree, const u_char *pd, int offset, +add_octetstring_tree(proto_tree *tree, tvbuff_t *tvb, int offset, int name_length, int value_length) { proto_item *ti; - ti = proto_tree_add_text(tree, NullTVB, offset, + ti = proto_tree_add_text(tree, tvb, offset, 1 + 2 + name_length + 2 + value_length, "%.*s: %s", name_length, - &pd[offset + 1 + 2], - bytes_to_str(&pd[offset + 1 + 2 + name_length + 2], value_length)); + tvb_get_ptr(tvb, offset + 1 + 2, name_length), + tvb_bytes_to_str(tvb, offset + 1 + 2 + name_length + 2, value_length)); return proto_item_add_subtree(ti, ett_ipp_attr); } static void add_octetstring_value(guint tag, gchar *tag_desc, proto_tree *tree, - const u_char *pd, int offset, int name_length, int value_length) + tvbuff_t *tvb, int offset, int name_length, int value_length) { - offset = add_value_head(tag, tag_desc, tree, pd, offset, + offset = add_value_head(tag, tag_desc, tree, tvb, offset, name_length, value_length); - proto_tree_add_text(tree, NullTVB, offset, value_length, - "Value: %s", bytes_to_str(&pd[offset], value_length)); + proto_tree_add_text(tree, tvb, offset, value_length, + "Value: %s", tvb_bytes_to_str(tvb, offset, value_length)); } static proto_tree * -add_charstring_tree(proto_tree *tree, const u_char *pd, int offset, +add_charstring_tree(proto_tree *tree, tvbuff_t *tvb, int offset, int name_length, int value_length) { proto_item *ti; - ti = proto_tree_add_text(tree, NullTVB, offset, + ti = proto_tree_add_text(tree, tvb, offset, 1 + 2 + name_length + 2 + value_length, "%.*s: %.*s", - name_length, &pd[offset + 1 + 2], - value_length, &pd[offset + 1 + 2 + name_length + 2]); + name_length, + tvb_get_ptr(tvb, offset + 1 + 2, name_length), + value_length, + tvb_get_ptr(tvb, offset + 1 + 2 + name_length + 2, value_length)); return proto_item_add_subtree(ti, ett_ipp_attr); } static void add_charstring_value(guint tag, gchar *tag_desc, proto_tree *tree, - const u_char *pd, int offset, int name_length, int value_length) + tvbuff_t *tvb, int offset, int name_length, int value_length) { - offset = add_value_head(tag, tag_desc, tree, pd, offset, + offset = add_value_head(tag, tag_desc, tree, tvb, offset, name_length, value_length); - proto_tree_add_text(tree, NullTVB, offset, value_length, - "Value: %.*s", value_length, &pd[offset]); + proto_tree_add_text(tree, tvb, offset, value_length, + "Value: %.*s", value_length, tvb_get_ptr(tvb, offset, value_length)); } static int add_value_head(guint tag, gchar *tag_desc, proto_tree *tree, - const u_char *pd, int offset, int name_length, int value_length) + tvbuff_t *tvb, int offset, int name_length, int value_length) { - proto_tree_add_text(tree, NullTVB, offset, 1, "Tag: %s", tag_desc); + proto_tree_add_text(tree, tvb, offset, 1, "Tag: %s", tag_desc); offset += 1; - proto_tree_add_text(tree, NullTVB, offset, 2, "Name length: %u", + proto_tree_add_text(tree, tvb, offset, 2, "Name length: %u", name_length); offset += 2; if (name_length != 0) { - proto_tree_add_text(tree, NullTVB, offset, name_length, - "Name: %.*s", name_length, &pd[offset]); + proto_tree_add_text(tree, tvb, offset, name_length, + "Name: %.*s", name_length, + tvb_get_ptr(tvb, offset, name_length)); } offset += name_length; - proto_tree_add_text(tree, NullTVB, offset, 2, "Value length: %u", + proto_tree_add_text(tree, tvb, offset, 2, "Value length: %u", value_length); offset += 2; return offset; @@ -574,6 +563,14 @@ proto_register_ipp(void) proto_ipp = proto_register_protocol("Internet Printing Protocol", "ipp"); /* proto_register_field_array(proto_ipp, hf, array_length(hf));*/ proto_register_subtree_array(ett, array_length(ett)); + + /* + * Register the dissector by name, so other dissectors can + * grab it by name rather than just referring to it directly + * (you can't refer to it directly from a plugin dissector + * on Windows without stuffing it into the Big Transfer Vector). + */ + register_dissector("ipp", dissect_ipp); } void |