diff options
author | cmaynard <cmaynard@f5534014-38df-0310-8fa8-9805f1628bb7> | 2011-09-19 20:37:20 +0000 |
---|---|---|
committer | cmaynard <cmaynard@f5534014-38df-0310-8fa8-9805f1628bb7> | 2011-09-19 20:37:20 +0000 |
commit | ddbc3b7fdc63deec10068d9d2f10211fd7c6321c (patch) | |
tree | 3a82d4370b168b9a27644e77d8290ab33069d9fd /epan/dissectors/packet-cisco-wids.c | |
parent | 1e0c9e02f731d1aebf586d1198d1282f1c530c20 (diff) |
Since the call to proto_tree_add_text() was #if 0'd out, there's no need to assign the return value of tvb_new_subset to wlan_tvb. Actually, there's probably no need to call tvb_new_subset() in the first place, but for now let's just tackle the unused value warning and leave the FIXME for someone who knows more about this. Fixes Coverity CID 864.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@39055 f5534014-38df-0310-8fa8-9805f1628bb7
Diffstat (limited to 'epan/dissectors/packet-cisco-wids.c')
-rw-r--r-- | epan/dissectors/packet-cisco-wids.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/epan/dissectors/packet-cisco-wids.c b/epan/dissectors/packet-cisco-wids.c index 927a6c0119..fefc487d27 100644 --- a/epan/dissectors/packet-cisco-wids.c +++ b/epan/dissectors/packet-cisco-wids.c @@ -127,12 +127,13 @@ dissect_cwids(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree) " [Malformed or short IEEE80211 subpacket] " ); col_set_fence(pinfo->cinfo, COL_INFO); } - - wlan_tvb = tvb_new_subset(tvb, offset, capturelen, capturelen); #if 0 + wlan_tvb = tvb_new_subset(tvb, offset, capturelen, capturelen); /* FIXME: Why does this throw an exception? */ proto_tree_add_text(cwids_tree, wlan_tvb, offset, capturelen, "[Malformed or short IEEE80211 subpacket]"); +#else + tvb_new_subset(tvb, offset, capturelen, capturelen); #endif ; } ENDTRY; |