diff options
author | wmeier <wmeier@f5534014-38df-0310-8fa8-9805f1628bb7> | 2011-04-12 16:00:28 +0000 |
---|---|---|
committer | wmeier <wmeier@f5534014-38df-0310-8fa8-9805f1628bb7> | 2011-04-12 16:00:28 +0000 |
commit | 12f9e2678e2d511d31448e2ce8a47a097d77b7b0 (patch) | |
tree | f323a3922b2f3dc678d42bc5941c032b4544b98c /epan/dissectors/packet-ieee8021ah.c | |
parent | a97684b7ac47de10fa5a2ebcdc3817949e075d8f (diff) |
Don't assign to a proto_item * if the value won't be used: Coverity 933;
Fix apparent use of incorrect tree: Coverity 934.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@36588 f5534014-38df-0310-8fa8-9805f1628bb7
Diffstat (limited to 'epan/dissectors/packet-ieee8021ah.c')
-rw-r--r-- | epan/dissectors/packet-ieee8021ah.c | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/epan/dissectors/packet-ieee8021ah.c b/epan/dissectors/packet-ieee8021ah.c index be40e62a38..870c60ee9a 100644 --- a/epan/dissectors/packet-ieee8021ah.c +++ b/epan/dissectors/packet-ieee8021ah.c @@ -224,7 +224,6 @@ dissect_ieee8021ah_common(tvbuff_t *tvb, packet_info *pinfo, proto_tree *volatile ieee8021ah_tag_tree; /* for parsing out ethernet addrs */ - proto_item *addr_item; const guint8 *src_addr, *dst_addr; tci = tvb_get_ntohl( tvb, 0 ); @@ -264,10 +263,10 @@ dissect_ieee8021ah_common(tvbuff_t *tvb, packet_info *pinfo, dst_addr = tvb_get_ptr(tvb, 4, 6); /* safe to use this function? */ src_addr = tvb_get_ptr(tvb, 10, 6); - addr_item = proto_tree_add_ether(tree, hf_ieee8021ah_c_daddr, + proto_tree_add_ether(tree, hf_ieee8021ah_c_daddr, tvb, 4, 6, dst_addr); - addr_item = proto_tree_add_ether(tree, hf_ieee8021ah_c_saddr, + proto_tree_add_ether(tree, hf_ieee8021ah_c_saddr, tvb, 10, 6, src_addr); /* add text to 802.1ad label */ @@ -329,7 +328,7 @@ dissect_ieee8021ah(tvbuff_t *tvb, packet_info *pinfo, ptree = proto_tree_add_item(tree, proto_tree_index, tvb, 0, IEEE8021AH_LEN, FALSE); ieee8021ah_tree = proto_item_add_subtree(ptree, ett_ieee8021ah); - dissect_ieee8021ah_common(tvb, pinfo, ptree, tree, proto_tree_index); + dissect_ieee8021ah_common(tvb, pinfo, ieee8021ah_tree, tree, proto_tree_index); } } |