diff options
author | gerald <gerald@f5534014-38df-0310-8fa8-9805f1628bb7> | 2007-01-25 06:45:09 +0000 |
---|---|---|
committer | gerald <gerald@f5534014-38df-0310-8fa8-9805f1628bb7> | 2007-01-25 06:45:09 +0000 |
commit | f8b1d6199948188f86bba477a57056de47908964 (patch) | |
tree | 47c47562a1fc8a12d347842bef1bb0a2deebf275 /epan/dissectors/packet-scsi.c | |
parent | 20ed6d0adc6ded1979d715455a53fae94044f091 (diff) |
Attempt to squelch Coverity CID 222.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@20551 f5534014-38df-0310-8fa8-9805f1628bb7
Diffstat (limited to 'epan/dissectors/packet-scsi.c')
-rw-r--r-- | epan/dissectors/packet-scsi.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/epan/dissectors/packet-scsi.c b/epan/dissectors/packet-scsi.c index 831492ddd6..99c7ea4726 100644 --- a/epan/dissectors/packet-scsi.c +++ b/epan/dissectors/packet-scsi.c @@ -4021,8 +4021,10 @@ dissect_scsi_payload (tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, /* Start reassembly */ - if (tvb_length_remaining(tvb, offset) >= 0 && - (tvb_length_remaining(tvb,offset) + relative_offset) != expected_length) { + if (tvb_length_remaining(tvb, offset) < 0) { + goto end_of_payload; + } + if ((tvb_length_remaining(tvb,offset) + relative_offset) != expected_length) { more_frags = TRUE; } ipfd_head = fragment_add_check(tvb, offset, pinfo, |