(/home/sate/Testcases/c/cve/wireshark-1.2.0/epan/dissectors/packet-isakmp.c) |
| |
| 1711 | | | static const char * |
| 1712 | | | v2_aft2str(guint16 aft) |
| 1713 | | | { |
| 1714 | | | if (aft < 14 || (aft > 14 && aft < 18)) return "RESERVED"; |
| 1715 | | | if (aft == 14) return "Key Length (in bits)"; |
| 1716 | | | if (aft >= 18 && aft < 16384) return "RESERVED TO IANA"; |
Redundant Condition
aft >= 18 always evaluates to true. This may be because: - There is a constant assignment to one or more of the variables involved.
- An earlier conditional statement has already ensured that aft >= 18 cannot be false.
- A crashing bug occurs on every path where aft >= 18 could have evaluated to false. Look for a preceding Null Pointer Dereference or Division By Zero warning.
|
|
| 1717 | | | return "PRIVATE USE"; |
| 1718 | | | } |
| |