Fix string memory leaks.
[smatch.git] / smatch_implied.c
blob7df9669d7277704b351c9ee9cb0b4522d7b35d1d
1 /*
2 * sparse/smatch_implied.c
4 * Copyright (C) 2008 Dan Carpenter.
6 * Licensed under the Open Software License version 1.1
8 */
11 * Imagine we have this code:
12 * foo = 0;
13 * if (bar)
14 * foo = 1;
15 * // <-- point #1
16 * else
17 * frob();
18 * // <-- point #2
19 * if (foo == 1) // <-- point #3
20 * bar->baz; // <-- point #4
22 * Currently (Oct 2008) in smatch when we merge bar states
23 * null and nonnull, at point #2, the state becomes undefined.
24 * As a result we get an error at point #3.
26 * The idea behind "implied state pools" is to fix that.
28 * The implied pools get created in merge_slist(). Whatever
29 * is unique to one slist being merged gets put into a pool.
31 * If we set a state that removes it from all pools.
33 * When we come to an if statement where "foo" has some pools
34 * associated we take all the pools where "foo == 1" and keep
35 * all the states that are consistent across those pools.
37 * The point of doing this is to turn an undefined state into
38 * a defined state. This hopefully gets rid of some false positives.
39 * What it doesn't do is find new errors that were
40 * missed before.
42 * There are quite a few implementation details I haven't figured
43 * out. How do you create implied state pools inside a
44 * complex condition? How do you determine what is implied
45 * from a complex condition? The initial patch is extremely rudimentary...
48 #include "smatch.h"
49 #include "smatch_slist.h"
51 #define EQUALS 0
52 #define NOTEQUALS 1
55 * What are the implications if (foo == num) ...
58 static struct state_list_stack *get_eq_neq(struct sm_state *sm_state,
59 int eq_neq, int num)
61 struct state_list *list;
62 struct smatch_state *s;
63 struct state_list_stack *ret = NULL;
65 FOR_EACH_PTR(sm_state->my_pools, list) {
66 s = get_state_slist(list, sm_state->name, sm_state->owner,
67 sm_state->sym);
68 if (s == &undefined) {
69 free_stack(&ret);
70 return NULL;
72 if (s->data && ((eq_neq == EQUALS && *(int *)s->data == num) ||
73 (eq_neq == NOTEQUALS && *(int *)s->data != num))) {
74 push_slist(&ret, list);
76 } END_FOR_EACH_PTR(list);
77 return ret;
80 static struct state_list *filter_stack(struct state_list_stack *stack)
82 struct state_list *tmp;
83 struct state_list *ret = NULL;
84 int i = 0;
86 FOR_EACH_PTR(stack, tmp) {
87 if (!i++)
88 ret = clone_states_in_pool(tmp, __get_cur_slist());
89 else
90 filter(&ret, tmp, __get_cur_slist());
91 } END_FOR_EACH_PTR(tmp);
92 return ret;
95 void __implied_states_hook(struct expression *expr)
97 struct symbol *sym;
98 char *name;
99 struct sm_state *state;
100 struct state_list_stack *true_pools;
101 struct state_list_stack *false_pools;
102 struct state_list *implied_true;
103 struct state_list *implied_false;
105 name = get_variable_from_expr(expr, &sym);
106 if (!name || !sym) {
107 free_string(name);
108 return;
110 state = get_sm_state(name, SMATCH_EXTRA, sym);
111 free_string(name);
112 if (!state)
113 return;
114 if (!state->my_pools)
115 return;
117 true_pools = get_eq_neq(state, NOTEQUALS, 0);
118 false_pools = get_eq_neq(state, EQUALS, 0);
119 implied_true = filter_stack(true_pools);
120 implied_false = filter_stack(false_pools);
121 if (debug_states) {
122 printf("Setting the following implied states for the true path.\n");
123 __print_slist(implied_true);
126 FOR_EACH_PTR(implied_true, state) {
127 __set_true_false_sm(state, NULL);
128 } END_FOR_EACH_PTR(state);
130 if (debug_states) {
131 printf("Setting the following implied states for the false path.\n");
132 __print_slist(implied_false);
135 FOR_EACH_PTR(implied_false, state) {
136 __set_true_false_sm(NULL, state);
137 } END_FOR_EACH_PTR(state);
139 free_stack(&true_pools);
140 free_stack(&false_pools);
141 free_slist(&implied_true);
142 free_slist(&implied_false);
145 void register_implications(int id)
147 add_hook(&__implied_states_hook, CONDITION_HOOK);