(/home/sate/Testcases/c/cve/wireshark-1.2.0/epan/dissectors/packet-isakmp.c) |
| |
| 2843 | | | dissect_eap(tvbuff_t *tvb, int offset, int length, proto_tree *tree, |
| 2844 | | | proto_tree *p _U_, packet_info *pinfo _U_, int isakmp_version _U_, int unused _U_, guint8 inner_payload _U_) |
| 2845 | | | { |
| 2846 | | | tvbuff_t *eap_tvb = NULL; |
| 2847 | | | |
| 2848 | | | eap_tvb = tvb_new_subset(tvb, offset,length, length ); |
| 2849 | | | if ((eap_tvb != NULL)&& eap_handle != NULL){ |
Redundant Condition
eap_tvb != (void *)0 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 eap_tvb != (void *)0 cannot be false.
- A crashing bug occurs on every path where eap_tvb != (void *)0 could have evaluated to false. Look for a preceding Null Pointer Dereference or Division By Zero warning.
|
|
| 2850 | | | call_dissector(eap_handle, eap_tvb, pinfo, tree); |
| 2851 | | | }else{ |
| 2852 | | | proto_tree_add_text(tree, tvb, offset, length, "EAP Message"); |
| 2853 | | | } |
| 2854 | | | } |
| |