hc
2024-02-20 102a0743326a03cd1a1202ceda21e175b7d3575c
kernel/scripts/coccinelle/api/memdup_user.cocci
....@@ -1,10 +1,11 @@
1
+// SPDX-License-Identifier: GPL-2.0-only
12 /// Use memdup_user rather than duplicating its implementation
23 /// This is a little bit restricted to reduce false positives
34 ///
45 // Confidence: High
5
-// Copyright: (C) 2010-2012 Nicolas Palix. GPLv2.
6
-// Copyright: (C) 2010-2012 Julia Lawall, INRIA/LIP6. GPLv2.
7
-// Copyright: (C) 2010-2012 Gilles Muller, INRIA/LiP6. GPLv2.
6
+// Copyright: (C) 2010-2012 Nicolas Palix.
7
+// Copyright: (C) 2010-2012 Julia Lawall, INRIA/LIP6.
8
+// Copyright: (C) 2010-2012 Gilles Muller, INRIA/LiP6.
89 // URL: http://coccinelle.lip6.fr/
910 // Comments:
1011 // Options: --no-includes --include-headers
....@@ -14,13 +15,46 @@
1415 virtual org
1516 virtual report
1617
18
+@initialize:python@
19
+@@
20
+filter = frozenset(['memdup_user', 'vmemdup_user'])
21
+
22
+def relevant(p):
23
+ return not (filter & {el.current_element for el in p})
24
+
1725 @depends on patch@
1826 expression from,to,size;
1927 identifier l1,l2;
28
+position p : script:python() { relevant(p) };
2029 @@
2130
22
-- to = \(kmalloc\|kzalloc\)(size,GFP_KERNEL);
31
+- to = \(kmalloc@p\|kzalloc@p\)
32
+- (size,\(GFP_KERNEL\|GFP_USER\|
33
+- \(GFP_KERNEL\|GFP_USER\)|__GFP_NOWARN\));
2334 + to = memdup_user(from,size);
35
+ if (
36
+- to==NULL
37
++ IS_ERR(to)
38
+ || ...) {
39
+ <+... when != goto l1;
40
+- -ENOMEM
41
++ PTR_ERR(to)
42
+ ...+>
43
+ }
44
+- if (copy_from_user(to, from, size) != 0) {
45
+- <+... when != goto l2;
46
+- -EFAULT
47
+- ...+>
48
+- }
49
+
50
+@depends on patch@
51
+expression from,to,size;
52
+identifier l1,l2;
53
+position p : script:python() { relevant(p) };
54
+@@
55
+
56
+- to = \(kvmalloc@p\|kvzalloc@p\)(size,\(GFP_KERNEL\|GFP_USER\));
57
++ to = vmemdup_user(from,size);
2458 if (
2559 - to==NULL
2660 + IS_ERR(to)
....@@ -38,11 +72,24 @@
3872
3973 @r depends on !patch@
4074 expression from,to,size;
41
-position p;
75
+position p : script:python() { relevant(p) };
4276 statement S1,S2;
4377 @@
4478
45
-* to = \(kmalloc@p\|kzalloc@p\)(size,GFP_KERNEL);
79
+* to = \(kmalloc@p\|kzalloc@p\)
80
+ (size,\(GFP_KERNEL\|GFP_USER\|
81
+ \(GFP_KERNEL\|GFP_USER\)|__GFP_NOWARN\));
82
+ if (to==NULL || ...) S1
83
+ if (copy_from_user(to, from, size) != 0)
84
+ S2
85
+
86
+@rv depends on !patch@
87
+expression from,to,size;
88
+position p : script:python() { relevant(p) };
89
+statement S1,S2;
90
+@@
91
+
92
+* to = \(kvmalloc@p\|kvzalloc@p\)(size,\(GFP_KERNEL\|GFP_USER\));
4693 if (to==NULL || ...) S1
4794 if (copy_from_user(to, from, size) != 0)
4895 S2
....@@ -58,3 +105,15 @@
58105 @@
59106
60107 coccilib.report.print_report(p[0], "WARNING opportunity for memdup_user")
108
+
109
+@script:python depends on org@
110
+p << rv.p;
111
+@@
112
+
113
+coccilib.org.print_todo(p[0], "WARNING opportunity for vmemdup_user")
114
+
115
+@script:python depends on report@
116
+p << rv.p;
117
+@@
118
+
119
+coccilib.report.print_report(p[0], "WARNING opportunity for vmemdup_user")