extra: dereferencing a pointer means it is valid
[smatch.git] / check_deref_check.c
blobe75e5879efdaf4e007567231dff62d50dbd2a87f
1 /*
2 * sparse/check_deref_check.c
4 * Copyright (C) 2009 Dan Carpenter.
6 * Licensed under the Open Software License version 1.1
8 */
10 #include "smatch.h"
11 #include "smatch_extra.h"
13 static int my_id;
15 STATE(derefed);
16 STATE(oktocheck);
18 static void underef(const char *name, struct symbol *sym, struct expression *expr, void *unused)
20 set_state(my_id, name, sym, &oktocheck);
23 static void match_dereference(struct expression *expr)
25 char *name;
27 if (expr->type != EXPR_PREOP)
28 return;
29 if (getting_address())
30 return;
32 expr = strip_expr(expr->unop);
33 set_state_expr(my_id, expr, &derefed);
34 name = get_variable_from_expr(expr, NULL);
35 if (!name)
36 return;
37 add_modification_hook(my_id, name, &underef, NULL);
38 free_string(name);
41 static void set_param_dereferenced(struct expression *arg, char *unused)
43 set_state_expr(my_id, arg, &derefed);
44 add_modification_hook_expr(my_id, arg, &underef, NULL);
47 static void match_condition(struct expression *expr)
49 struct sm_state *sm;
51 if (__in_pre_condition)
52 return;
54 if (get_macro_name(expr->pos))
55 return;
57 sm = get_sm_state_expr(my_id, expr);
58 if (!sm || sm->state != &derefed)
59 return;
61 sm_msg("warn: variable dereferenced before check '%s' (see line %d)", sm->name, sm->line);
62 set_state_expr(my_id, expr, &oktocheck);
65 void check_deref_check(int id)
67 my_id = id;
68 add_hook(&match_dereference, DEREF_HOOK);
69 add_hook(&match_condition, CONDITION_HOOK);
70 add_db_fn_call_callback(DEREFERENCE, &set_param_dereferenced);