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.