diff options
| author | Anders Broman <anders.broman@ericsson.com> | 2008-12-13 22:06:35 +0000 |
|---|---|---|
| committer | Anders Broman <anders.broman@ericsson.com> | 2008-12-13 22:06:35 +0000 |
| commit | a9551616cbb139480538c8691c0638d837788559 (patch) | |
| tree | d1e91c150b233599ba66f2040c9e9a872c233688 /epan/dissectors/packet-sndcp-xid.c | |
| parent | 3947676a844eaafb1760986faa3a588d3cd8758b (diff) | |
| download | wireshark-a9551616cbb139480538c8691c0638d837788559.tar.gz wireshark-a9551616cbb139480538c8691c0638d837788559.tar.bz2 wireshark-a9551616cbb139480538c8691c0638d837788559.zip | |
From Vincent Helfre:
Get rid of sndcp_xid_tree and fix the while loop.
svn path=/trunk/; revision=26991
Diffstat (limited to 'epan/dissectors/packet-sndcp-xid.c')
| -rw-r--r-- | epan/dissectors/packet-sndcp-xid.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/epan/dissectors/packet-sndcp-xid.c b/epan/dissectors/packet-sndcp-xid.c index f0c037e46d..ba74cc7dea 100644 --- a/epan/dissectors/packet-sndcp-xid.c +++ b/epan/dissectors/packet-sndcp-xid.c @@ -481,7 +481,7 @@ dissect_sndcp_xid(tvbuff_t *tvb, packet_info *pinfo _U_, proto_tree *tree) /* Set up structures needed to add the protocol subtree and manage it */ proto_item *ti, *version_item, *dcomp_item, *pcomp_item; - proto_tree *sndcp_xid_tree = NULL, *version_tree = NULL, *dcomp_tree, *pcomp_tree = NULL; + proto_tree *version_tree = NULL, *dcomp_tree, *pcomp_tree = NULL; guint16 offset = 0, l3_param_len; guint8 parameter_type, parameter_len; @@ -489,11 +489,10 @@ dissect_sndcp_xid(tvbuff_t *tvb, packet_info *pinfo _U_, proto_tree *tree) */ if (tree) { ti = proto_tree_add_item(tree, proto_sndcp_xid, tvb, 0, -1, FALSE); - sndcp_xid_tree = proto_item_add_subtree(ti, ett_sndcp_xid); } l3_param_len = tvb_reported_length(tvb); - while (offset < l3_param_len) + while (offset < l3_param_len-1) { parameter_type = tvb_get_guint8(tvb, offset); parameter_len = tvb_get_guint8(tvb, offset+1); |