Huge thanks to our Platinum Members Endace and LiveAction,
and our Silver Member Veeam, for supporting the Wireshark Foundation and project.

Wireshark-bugs: [Wireshark-bugs] [Bug 12503] New: Coverity CID 1325714: Dead code in packet-isns

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 [email protected]
Reporter [email protected]
CC [email protected]

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.