diff options
author | Chris Maynard <Christopher.Maynard@GTECH.COM> | 2011-09-19 20:37:20 +0000 |
---|---|---|
committer | Chris Maynard <Christopher.Maynard@GTECH.COM> | 2011-09-19 20:37:20 +0000 |
commit | fb19d6ab0f57c02b1dff93557612490ab71db3e2 (patch) | |
tree | 3a82d4370b168b9a27644e77d8290ab33069d9fd /epan/dissectors/packet-cisco-wids.c | |
parent | a4d1ae6547e8d76080674fe68915241d1e4bebe0 (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.
svn path=/trunk/; revision=39055
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; |