diff options
author | Julia Lawall <Julia.Lawall@lip6.fr> | 2012-01-27 15:41:27 -0500 |
---|---|---|
committer | Michal Marek <mmarek@suse.cz> | 2012-02-24 18:07:11 -0500 |
commit | cd0207a7bfd29016a4eeb0b0203fa900b41e4eee (patch) | |
tree | 2d479b9c71e07ac46d8c03b3f9253aa9b2448f56 /scripts | |
parent | 53302c1dfa8a911b88e68634ed56fcf577037fac (diff) |
coccinelle: semantic patch for missing iounmap
Signed-off-by: Julia Lawall <Julia.Lawall@lip6.fr>
Signed-off-by: Michal Marek <mmarek@suse.cz>
Diffstat (limited to 'scripts')
-rw-r--r-- | scripts/coccinelle/free/iounmap.cocci | 67 |
1 files changed, 67 insertions, 0 deletions
diff --git a/scripts/coccinelle/free/iounmap.cocci b/scripts/coccinelle/free/iounmap.cocci new file mode 100644 index 000000000000..5384f4ba1192 --- /dev/null +++ b/scripts/coccinelle/free/iounmap.cocci | |||
@@ -0,0 +1,67 @@ | |||
1 | /// Find missing iounmaps. | ||
2 | /// | ||
3 | //# This only signals a missing iounmap when there is an iounmap later | ||
4 | //# in the same function. | ||
5 | //# False positives can be due to loops. | ||
6 | // | ||
7 | // Confidence: Moderate | ||
8 | // Copyright: (C) 2012 Julia Lawall, INRIA/LIP6. GPLv2. | ||
9 | // Copyright: (C) 2012 Gilles Muller, INRIA/LiP6. GPLv2. | ||
10 | // URL: http://coccinelle.lip6.fr/ | ||
11 | // Comments: | ||
12 | // Options: | ||
13 | |||
14 | virtual context | ||
15 | virtual org | ||
16 | virtual report | ||
17 | |||
18 | @iom@ | ||
19 | expression e; | ||
20 | statement S,S1; | ||
21 | int ret; | ||
22 | position p1,p2,p3; | ||
23 | @@ | ||
24 | |||
25 | e = \(ioremap@p1\|ioremap_nocache@p1\)(...) | ||
26 | ... when != iounmap(e) | ||
27 | if (<+...e...+>) S | ||
28 | ... when any | ||
29 | when != iounmap(e) | ||
30 | when != if (...) { ... iounmap(e); ... } | ||
31 | ( | ||
32 | if (ret == 0) S1 | ||
33 | | | ||
34 | if (...) | ||
35 | { ... | ||
36 | return 0; } | ||
37 | | | ||
38 | if (...) | ||
39 | { ... | ||
40 | return <+...e...+>; } | ||
41 | | | ||
42 | *if@p2 (...) | ||
43 | { ... when != iounmap(e) | ||
44 | when forall | ||
45 | return@p3 ...; } | ||
46 | ) | ||
47 | ... when any | ||
48 | iounmap(e); | ||
49 | |||
50 | @script:python depends on org@ | ||
51 | p1 << iom.p1; | ||
52 | p2 << iom.p2; | ||
53 | p3 << iom.p3; | ||
54 | @@ | ||
55 | |||
56 | cocci.print_main("ioremap",p1) | ||
57 | cocci.print_secs("if",p2) | ||
58 | cocci.print_secs("needed iounmap",p3) | ||
59 | |||
60 | @script:python depends on report@ | ||
61 | p1 << iom.p1; | ||
62 | p2 << iom.p2; | ||
63 | p3 << iom.p3; | ||
64 | @@ | ||
65 | |||
66 | msg = "ERROR: missing iounmap; ioremap on line %s and execution via conditional on line %s" % (p1[0].line,p2[0].line) | ||
67 | coccilib.report.print_report(p3[0],msg) | ||