(/home/sate/Testcases/c/cve/wireshark-1.2.0/epan/dissectors/packet-smb2.c) |
| |
| 2880 | | | dissect_smb2_find_data(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, smb2_info_t *si) |
| 2881 | | | { |
| 2882 | | | smb2_find_dissector_t *dis = smb2_find_dissectors; |
| 2883 | | | |
| 2884 | | | while(dis->dissector){ |
| 2885 | | | if(si && si->saved && si->saved){ |
Redundant Condition
si->saved 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 si->saved cannot be false.
- A crashing bug occurs on every path where si->saved could have evaluated to false. Look for a preceding Null Pointer Dereference or Division By Zero warning.
|
|
| 2886 | | | if(dis->level ==si->saved->infolevel){ |
| 2887 | | | dis->dissector(tvb, pinfo, tree, si); |
| 2888 | | | return; |
| 2889 | | | } |
| 2890 | | | } |
| 2891 | | | dis++; |
| 2892 | | | } |
| 2893 | | | |
| 2894 | | | proto_tree_add_item(tree, hf_smb2_unknown, tvb, 0, tvb_length(tvb), FALSE); |
| 2895 | | | return; |
| 2896 | | | } |
| |