(/home/sate/Testcases/c/cve/wireshark-1.2.0/epan/dissectors/packet-dcerpc.c) |
| |
| 5085 | | | dcerpc_init_protocol (void) |
| 5086 | | | { |
| 5087 | | | |
| 5088 | | | if (dcerpc_binds){ |
| 5089 | | | g_hash_table_destroy (dcerpc_binds); |
| 5090 | | | dcerpc_binds=NULL; |
| 5091 | | | } |
| 5092 | | | if(!dcerpc_binds){ |
Redundant Condition
dcerpc_binds always evaluates to false. 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 dcerpc_binds cannot be true.
- A crashing bug occurs on every path where dcerpc_binds could have evaluated to true. Look for a preceding Null Pointer Dereference or Division By Zero warning.
|
|
| 5093 | | | dcerpc_binds = g_hash_table_new (dcerpc_bind_hash, dcerpc_bind_equal); |
| 5094 | | | } |
| 5095 | | | |
| 5096 | | | |
| 5097 | | | if (dcerpc_cn_calls){ |
| 5098 | | | g_hash_table_destroy (dcerpc_cn_calls); |
| 5099 | | | } |
| 5100 | | | dcerpc_cn_calls = g_hash_table_new (dcerpc_cn_call_hash, dcerpc_cn_call_equal); |
| 5101 | | | if (dcerpc_dg_calls){ |
| 5102 | | | g_hash_table_destroy (dcerpc_dg_calls); |
| 5103 | | | } |
| 5104 | | | dcerpc_dg_calls = g_hash_table_new (dcerpc_dg_call_hash, dcerpc_dg_call_equal); |
| 5105 | | | |
| 5106 | | | |
| 5107 | | | if (dcerpc_matched){ |
| 5108 | | | g_hash_table_destroy (dcerpc_matched); |
| 5109 | | | } |
| 5110 | | | dcerpc_matched = g_hash_table_new (dcerpc_matched_hash, dcerpc_matched_equal); |
| 5111 | | | |
| 5112 | | | |
| 5113 | | | g_hook_list_invoke(&dcerpc_hooks_init_protos, FALSE ); |
| 5114 | | | } |
| |