diff options
| author | cmaynard <cmaynard@f5534014-38df-0310-8fa8-9805f1628bb7> | 2011-08-08 20:08:24 +0000 |
|---|---|---|
| committer | cmaynard <cmaynard@f5534014-38df-0310-8fa8-9805f1628bb7> | 2011-08-08 20:08:24 +0000 |
| commit | 144c10a3ee34fa67e322e6e805cd1ed664968a5f (patch) | |
| tree | d85f2dd8a148c7cea0c59bc2f13354b10f2dc98c /epan/dissectors/packet-usb.c | |
| parent | 8c9f92dd763b116d25015d9db6018baba88cbba9 (diff) | |
No need to check that usb_trans_info is not NULL here. The only way it could have been NULL is if se_alloc0() returned NULL, but then it would have already been dereferenced before now. Fixes Coverity CID 612, "REVERSE INULL".
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@38417 f5534014-38df-0310-8fa8-9805f1628bb7
Diffstat (limited to 'epan/dissectors/packet-usb.c')
| -rw-r--r-- | epan/dissectors/packet-usb.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/epan/dissectors/packet-usb.c b/epan/dissectors/packet-usb.c index 2b981e25b7..d94055c51c 100644 --- a/epan/dissectors/packet-usb.c +++ b/epan/dissectors/packet-usb.c @@ -1946,7 +1946,7 @@ dissect_linux_usb_common(tvbuff_t *tvb, packet_info *pinfo, proto_tree *parent, } usb_conv_info->usb_trans_info=usb_trans_info; - if(usb_trans_info && usb_trans_info->response_in){ + if(usb_trans_info->response_in){ proto_item *ti; ti=proto_tree_add_uint(tree, hf_usb_response_in, tvb, 0, 0, usb_trans_info->response_in); |