diff options
| author | Chris Maynard <Christopher.Maynard@GTECH.COM> | 2011-08-08 20:08:24 +0000 |
|---|---|---|
| committer | Chris Maynard <Christopher.Maynard@GTECH.COM> | 2011-08-08 20:08:24 +0000 |
| commit | 3037a852a82d6ff5170ffe3f66e07208b660aa47 (patch) | |
| tree | d85f2dd8a148c7cea0c59bc2f13354b10f2dc98c /epan/dissectors/packet-usb.c | |
| parent | e59bb0ad7224dc3560c58077df0ac9695afb2049 (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".
svn path=/trunk/; revision=38417
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); |