math: array parameters can be NULL
[smatch.git] / check_leaks.c
blob00fc963176fd8af15be410dd3f57602e2a8e6604
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;
122 static void match_alloc(const char *fn, struct expression *expr, void *unused)
124 if (__in_fake_var_assign)
125 return;
126 if (!is_local(expr->left))
127 return;
128 if (is_param(expr->left))
129 return;
130 if (expr->left->type != EXPR_SYMBOL)
131 return;
132 set_state_expr(my_id, expr->left, &allocated);
135 static void match_condition(struct expression *expr)
137 struct sm_state *sm;
139 expr = strip_expr(expr);
141 switch (expr->type) {
142 case EXPR_PREOP:
143 case EXPR_SYMBOL:
144 case EXPR_DEREF:
145 sm = get_sm_state_expr(my_id, expr);
146 if (sm && slist_has_state(sm->possible, &allocated))
147 set_true_false_states_expr(my_id, expr, NULL, &ok);
148 return;
149 case EXPR_ASSIGNMENT:
150 /* You have to deal with stuff like if (a = b = c) */
151 match_condition(expr->left);
152 return;
153 default:
154 return;
158 static void set_parent(struct expression *expr, struct smatch_state *state)
160 char *name;
161 struct symbol *sym;
163 expr = strip_expr(expr);
164 if (!expr)
165 return;
166 if (expr->type == EXPR_CONDITIONAL ||
167 expr->type == EXPR_SELECT) {
168 set_parent(expr->cond_true, state);
169 set_parent(expr->cond_false, state);
170 return;
173 name = get_parent_from_expr(expr, &sym);
174 if (!name || !sym)
175 goto free;
176 if (state == &ok && !get_state(my_id, name, sym))
177 goto free;
178 set_state(my_id, name, sym, state);
179 free:
180 free_string(name);
183 static void match_function_call(struct expression *expr)
185 struct expression *tmp;
187 FOR_EACH_PTR(expr->args, tmp) {
188 set_parent(tmp, &ok);
189 } END_FOR_EACH_PTR(tmp);
192 static void warn_if_allocated(struct expression *expr)
194 struct sm_state *sm;
195 char *name;
196 sval_t sval;
198 if (get_implied_value(expr, &sval) && sval.value == 0)
199 return;
201 sm = get_sm_state_expr(my_id, expr);
202 if (!sm)
203 return;
204 if (!slist_has_state(sm->possible, &allocated))
205 return;
207 name = expr_to_var(expr);
208 sm_warning("overwrite may leak '%s'", name);
209 free_string(name);
211 /* silence further warnings */
212 set_state_expr(my_id, expr, &ok);
215 static void match_assign(struct expression *expr)
217 struct expression *right;
219 right = expr->right;
221 while (right->type == EXPR_ASSIGNMENT)
222 right = right->left;
224 warn_if_allocated(expr->left);
225 set_parent(right, &ok);
228 static void check_for_allocated(void)
230 struct stree *stree;
231 struct sm_state *tmp;
233 stree = __get_cur_stree();
234 FOR_EACH_MY_SM(my_id, stree, tmp) {
235 if (!slist_has_state(tmp->possible, &allocated))
236 continue;
237 sm_warning("possible memory leak of '%s'", tmp->name);
238 } END_FOR_EACH_SM(tmp);
241 static void match_return(struct expression *ret_value)
243 if (__inline_fn)
244 return;
245 set_parent(ret_value, &ok);
246 check_for_allocated();
249 static void match_end_func(struct symbol *sym)
251 if (__inline_fn)
252 return;
253 check_for_allocated();
256 void check_leaks(int id)
258 int i;
260 my_id = id;
262 for (i = 0; i < ARRAY_SIZE(allocation_funcs); i++)
263 add_function_assign_hook(allocation_funcs[i], &match_alloc, NULL);
265 add_hook(&match_condition, CONDITION_HOOK);
267 add_hook(&match_function_call, FUNCTION_CALL_HOOK);
268 add_hook(&match_assign, ASSIGNMENT_HOOK);
270 add_hook(&match_return, RETURN_HOOK);
271 add_hook(&match_end_func, END_FUNC_HOOK);