(/home/sate/Testcases/c/cve/wireshark-1.2.0/epan/emem.c) |
| |
| 913 | | | void * |
| 914 | | | emem_tree_lookup32(emem_tree_t *se_tree, guint32 key) |
| 915 | | | { |
| 916 | | | emem_tree_node_t *node; |
| 917 | | | |
| 918 | | | node=se_tree->tree; |
| 919 | | | |
| 920 | | | while(node){ |
| 921 | | | if(key==node->key32){ |
| 922 | | | return node->data; |
| 923 | | | } |
| 924 | | | if(key<node->key32){ |
| 925 | | | node=node->left; |
| 926 | | | continue; |
| 927 | | | } |
| 928 | | | if(key>node->key32){ |
Redundant Condition
key > node->key32 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 key > node->key32 cannot be false.
- A crashing bug occurs on every path where key > node->key32 could have evaluated to false. Look for a preceding Null Pointer Dereference or Division By Zero warning.
|
|
| 929 | | | node=node->right; |
| 930 | | | continue; |
| 931 | | | } |
| 932 | | | } |
| 933 | | | return NULL; |
| 934 | | | } |
| |