(/home/sate/Testcases/c/cve/wireshark-1.2.0/epan/ftypes/ftype-integer.c) |
| |
| 134 | | | ipxnet_from_unparsed(fvalue_t *fv, char *s, gboolean allow_partial_value _U_, LogFunc logfunc) |
| 135 | | | { |
| 136 | | | guint32 val; |
| 137 | | | gboolean known; |
| 138 | | | |
| 139 | | | |
| 140 | | | |
| 141 | | | |
| 142 | | | |
| 143 | | | |
| 144 | | | if (val_from_unparsed(fv, s, TRUE, NULL)) { |
Redundant Condition
val_from_unparsed(...) 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 val_from_unparsed(...) cannot be true.
- A crashing bug occurs on every path where val_from_unparsed(...) could have evaluated to true. Look for a preceding Null Pointer Dereference or Division By Zero warning.
|
|
| 145 | | | return TRUE; |
| 146 | | | } |
| 147 | | | |
| 148 | | | val = get_ipxnet_addr(s, &known); |
| 149 | | | if (known) { |
| 150 | | | fv->value.uinteger = val; |
| 151 | | | return TRUE; |
| 152 | | | } |
| 153 | | | |
| 154 | | | logfunc("\"%s\" is not a valid IPX network name or address.", s); |
| 155 | | | return FALSE; |
| 156 | | | } |
| |