(/home/sate/Testcases/c/cve/wireshark-1.2.0/epan/stats_tree.c) |
| |
| 730 | | | stats_tree_create_pivot_by_pname(stats_tree *st, const gchar *name, |
| 731 | | | const gchar *parent_name) |
| 732 | | | { |
| 733 | | | int parent_id = stats_tree_parent_id_by_name(st,parent_name); |
| 734 | | | stat_node *node; |
| 735 | | | |
| 736 | | | node = new_stat_node(st,name,parent_id,TRUE,TRUE); |
| 737 | | | |
| 738 | | | if (node) |
Redundant Condition
node 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 node cannot be false.
- A crashing bug occurs on every path where node could have evaluated to false. Look for a preceding Null Pointer Dereference or Division By Zero warning.
|
|
| 739 | | | return node->id; |
| 740 | | | else |
| 741 | | | return 0; |
| 742 | | | } |
| |