(/home/sate/Testcases/c/cve/wireshark-1.2.0/epan/dissectors/packet-sctp.c) |
| |
| 2026 | | | frag_equal(gconstpointer k1, gconstpointer k2) |
| 2027 | | | { |
| 2028 | | | const frag_key* key1 = (const frag_key*) k1; |
| 2029 | | | const frag_key* key2 = (const frag_key*) k2; |
| 2030 | | | |
| 2031 | | | return ( (key1->sport == key2->sport) && |
| 2032 | | | (key1->dport == key1->dport) && |
| 2033 | | | (key1->verification_tag == key1->verification_tag) && |
| 2034 | | | (key1->stream_id == key1->stream_id) && |
| 2035 | | | (key1->stream_seq_num == key1->stream_seq_num) |
Redundant Condition
key1->stream_seq_num == key1->stream_seq_num 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 key1->stream_seq_num == key1->stream_seq_num cannot be false.
- A crashing bug occurs on every path where key1->stream_seq_num == key1->stream_seq_num could have evaluated to false. Look for a preceding Null Pointer Dereference or Division By Zero warning.
|
|
| 2036 | | | ? TRUE : FALSE); |
| 2037 | | | } |
| |