Wireshark-bugs: [Wireshark-bugs] [Bug 12503] New: Coverity CID 1325714: Dead code in packet-isns
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: bugzilla-daemon@xxxxxxxxxxxxx
Date: Thu, 09 Jun 2016 06:37:16 +0000
Bug ID 12503
Summary Coverity CID 1325714: Dead code in packet-isns.c
Product Wireshark
Version Git
Hardware All
OS All
Status UNCONFIRMED
Severity Minor
Priority Low
Component Dissection engine (libwireshark)
Assignee bugzilla-admin@wireshark.org
Reporter jaap.keuter@xs4all.nl
CC wireshark@kaiser.cx 

Build Information:
Paste the COMPLETE build information from "Help->About Wireshark", "wireshark
-v", or "tshark -v".
--
In commit 0be98aa0ec0c23d4a0fcfdc4ca3bd69c28c9a1b6 code was added (line 989 as
of now) to check for (len == 0) while the whole switch its contained in does
not run when (len == 0). Non of the other switch cases have this check, but
this one seems explicitly added, while resulting in dead code.
Martin, since you've added this code, could you shed some light?

You are receiving this mail because:
  • You are watching all bug changes.