param_set: addresses can't be set
[smatch.git] / check_leaks.c
blob1bba606a3d4355503ff2b471990c71737945014a
1 /*
2 * Copyright (C) 2010 Dan Carpenter.
4 * This program is free software; you can redistribute it and/or
5 * modify it under the terms of the GNU General Public License
6 * as published by the Free Software Foundation; either version 2
7 * of the License, or (at your option) any later version.
9 * This program is distributed in the hope that it will be useful,
10 * but WITHOUT ANY WARRANTY; without even the implied warranty of
11 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
12 * GNU General Public License for more details.
14 * You should have received a copy of the GNU General Public License
15 * along with this program; if not, see http://www.gnu.org/copyleft/gpl.txt
19 * The point of this check is to look for leaks.
20 * foo = malloc(); // <- mark it as allocated.
21 * A variable becomes &ok if we:
22 * 1) assign it to another variable.
23 * 2) pass it to a function.
25 * One complication is dealing with stuff like:
26 * foo->bar = malloc();
27 * foo->baz = malloc();
28 * foo = something();
30 * The work around is that for now what this check only
31 * checks simple expressions and doesn't check whether
32 * foo->bar is leaked.
36 #include <fcntl.h>
37 #include <unistd.h>
38 #include "parse.h"
39 #include "smatch.h"
40 #include "smatch_slist.h"
42 static int my_id;
44 STATE(allocated);
45 STATE(ok);
47 static void set_parent(struct expression *expr, struct smatch_state *state);
49 static const char *allocation_funcs[] = {
50 "malloc",
51 "kmalloc",
52 "kzalloc",
53 "kmemdup",
56 static char *alloc_parent_str(struct symbol *sym)
58 static char buf[256];
60 if (!sym || !sym->ident)
61 return NULL;
63 snprintf(buf, 255, "%s", sym->ident->name);
64 buf[255] = '\0';
65 return alloc_string(buf);
68 static char *get_parent_from_expr(struct expression *expr, struct symbol **sym)
70 char *name;
72 expr = strip_expr(expr);
74 name = expr_to_str_sym(expr, sym);
75 free_string(name);
76 if (!name || !*sym || !(*sym)->ident) {
77 *sym = NULL;
78 return NULL;
80 return alloc_parent_str(*sym);
83 static int is_local(struct expression *expr)
85 char *name;
86 struct symbol *sym;
87 int ret = 0;
89 name = expr_to_str_sym(expr, &sym);
90 if (!name || !sym)
91 goto out;
92 if (sym->ctype.modifiers & (MOD_NONLOCAL | MOD_STATIC | MOD_ADDRESSABLE))
93 goto out;
94 ret = 1;
95 out:
96 free_string(name);
97 return ret;
100 static int is_param(struct expression *expr)
102 char *name;
103 struct symbol *sym;
104 struct symbol *tmp;
105 int ret = 0;
107 name = expr_to_str_sym(expr, &sym);
108 if (!name || !sym)
109 goto out;
110 FOR_EACH_PTR(cur_func_sym->ctype.base_type->arguments, tmp) {
111 if (tmp == sym) {
112 ret = 1;
113 goto out;
115 } END_FOR_EACH_PTR(tmp);
116 out:
117 free_string(name);
118 return ret;
121 static void match_alloc(const char *fn, struct expression *expr, void *unused)
123 if (__in_fake_var_assign)
124 return;
125 if (!is_local(expr->left))
126 return;
127 if (is_param(expr->left))
128 return;
129 if (expr->left->type != EXPR_SYMBOL)
130 return;
131 set_state_expr(my_id, expr->left, &allocated);
134 static void match_condition(struct expression *expr)
136 struct sm_state *sm;
138 expr = strip_expr(expr);
140 switch (expr->type) {
141 case EXPR_PREOP:
142 case EXPR_SYMBOL:
143 case EXPR_DEREF:
144 sm = get_sm_state_expr(my_id, expr);
145 if (sm && slist_has_state(sm->possible, &allocated))
146 set_true_false_states_expr(my_id, expr, NULL, &ok);
147 return;
148 case EXPR_ASSIGNMENT:
149 /* You have to deal with stuff like if (a = b = c) */
150 match_condition(expr->left);
151 return;
152 default:
153 return;
157 static void set_parent(struct expression *expr, struct smatch_state *state)
159 char *name;
160 struct symbol *sym;
162 expr = strip_expr(expr);
163 if (!expr)
164 return;
165 if (expr->type == EXPR_CONDITIONAL ||
166 expr->type == EXPR_SELECT) {
167 set_parent(expr->cond_true, state);
168 set_parent(expr->cond_false, state);
169 return;
172 name = get_parent_from_expr(expr, &sym);
173 if (!name || !sym)
174 goto free;
175 if (state == &ok && !get_state(my_id, name, sym))
176 goto free;
177 set_state(my_id, name, sym, state);
178 free:
179 free_string(name);
182 static void match_function_call(struct expression *expr)
184 struct expression *tmp;
186 FOR_EACH_PTR(expr->args, tmp) {
187 set_parent(tmp, &ok);
188 } END_FOR_EACH_PTR(tmp);
191 static void warn_if_allocated(struct expression *expr)
193 struct sm_state *sm;
194 char *name;
195 sval_t sval;
197 sm = get_sm_state_expr(my_id, expr);
198 if (!sm)
199 return;
200 if (!slist_has_state(sm->possible, &allocated))
201 return;
203 if (get_implied_value(expr, &sval) && sval.value == 0)
204 return;
206 name = expr_to_var(expr);
207 sm_warning("overwrite may leak '%s'", name);
208 free_string(name);
210 /* silence further warnings */
211 set_state_expr(my_id, expr, &ok);
214 static void match_assign(struct expression *expr)
216 struct expression *right;
218 right = expr->right;
220 while (right->type == EXPR_ASSIGNMENT)
221 right = right->left;
223 warn_if_allocated(expr->left);
224 set_parent(right, &ok);
227 static void check_for_allocated(void)
229 struct stree *stree;
230 struct sm_state *tmp;
232 stree = __get_cur_stree();
233 FOR_EACH_MY_SM(my_id, stree, tmp) {
234 if (!slist_has_state(tmp->possible, &allocated))
235 continue;
236 sm_warning("possible memory leak of '%s'", tmp->name);
237 } END_FOR_EACH_SM(tmp);
240 static void match_return(struct expression *ret_value)
242 if (__inline_fn)
243 return;
244 set_parent(ret_value, &ok);
245 check_for_allocated();
248 static void match_end_func(struct symbol *sym)
250 if (__inline_fn)
251 return;
252 check_for_allocated();
255 void check_leaks(int id)
257 int i;
259 my_id = id;
261 for (i = 0; i < ARRAY_SIZE(allocation_funcs); i++)
262 add_function_assign_hook(allocation_funcs[i], &match_alloc, NULL);
264 add_hook(&match_condition, CONDITION_HOOK);
266 add_hook(&match_function_call, FUNCTION_CALL_HOOK);
267 add_hook(&match_assign, ASSIGNMENT_HOOK);
269 add_hook(&match_return, RETURN_HOOK);
270 add_hook(&match_end_func, END_FUNC_HOOK);