diff options
author | wmeier <wmeier@f5534014-38df-0310-8fa8-9805f1628bb7> | 2011-08-03 18:38:23 +0000 |
---|---|---|
committer | wmeier <wmeier@f5534014-38df-0310-8fa8-9805f1628bb7> | 2011-08-03 18:38:23 +0000 |
commit | d90173923240a741fad7e42115fe08bd70a6728b (patch) | |
tree | 69f842a20993553b10f86686f37c262b2fbcf6c8 /epan/dissectors | |
parent | 27efeafb1ae40ef764b3276433de5f1d7c0bef43 (diff) |
AFAICT, the item for 'hf_gtpv2_stn_sr' should be used as the parent tree-item in dissect_gtpv2_stn_sr().
This is a different fix for Coverity 1248 than that committed in SVN 38314.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@38323 f5534014-38df-0310-8fa8-9805f1628bb7
Diffstat (limited to 'epan/dissectors')
-rw-r--r-- | epan/dissectors/packet-gtpv2.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/epan/dissectors/packet-gtpv2.c b/epan/dissectors/packet-gtpv2.c index aec37b6c34..951afa0de9 100644 --- a/epan/dissectors/packet-gtpv2.c +++ b/epan/dissectors/packet-gtpv2.c @@ -939,13 +939,14 @@ dissect_gtpv2_recovery(tvbuff_t *tvb, packet_info *pinfo _U_, proto_tree *tree, static void dissect_gtpv2_stn_sr(tvbuff_t *tvb, packet_info *pinfo _U_, proto_tree *tree, proto_item *item _U_, guint16 length _U_,guint8 message_type _U_, guint8 instance _U_) { + proto_item *stn_sr_item; proto_tree *sub_tree; tvbuff_t *new_tvb; int offset = 0; - proto_tree_add_item(tree, hf_gtpv2_stn_sr, tvb, offset, length, FALSE); + stn_sr_item = proto_tree_add_item(tree, hf_gtpv2_stn_sr, tvb, offset, length, FALSE); new_tvb = tvb_new_subset(tvb, offset, length, length ); - sub_tree = proto_item_add_subtree(item, ett_gtpv2_stn_sr); + sub_tree = proto_item_add_subtree(stn_sr_item, ett_gtpv2_stn_sr); /* Octet 5 * contains the Nature of Address and Numbering Plan Indicator (NANPI) of the "AddressString" ASN.1 type (see 3GPP |