blob: 3c6dc5469ee4aca9b4e8e7169f9d8052725cf336 [file] [log] [blame]
Thomas Gleixner7f904d72019-06-04 10:11:38 +02001// SPDX-License-Identifier: GPL-2.0-only
Nicolas Palix998f5522010-08-24 17:39:05 +02002/// Use kstrdup rather than duplicating its implementation
3///
4// Confidence: High
Thomas Gleixner7f904d72019-06-04 10:11:38 +02005// Copyright: (C) 2010-2012 Nicolas Palix.
6// Copyright: (C) 2010-2012 Julia Lawall, INRIA/LIP6.
7// Copyright: (C) 2010-2012 Gilles Muller, INRIA/LiP6.
Nicolas Palix998f5522010-08-24 17:39:05 +02008// URL: http://coccinelle.lip6.fr/
9// Comments:
Nicolas Palix93f14462013-06-20 13:10:56 +020010// Options: --no-includes --include-headers
Nicolas Palix998f5522010-08-24 17:39:05 +020011
12virtual patch
Julia Lawall29a36d42012-01-14 23:41:54 +010013virtual context
14virtual org
15virtual report
Nicolas Palix998f5522010-08-24 17:39:05 +020016
Julia Lawall29a36d42012-01-14 23:41:54 +010017@depends on patch@
Nicolas Palix998f5522010-08-24 17:39:05 +020018expression from,to;
19expression flag,E1,E2;
20statement S;
21@@
22
23- to = kmalloc(strlen(from) + 1,flag);
24+ to = kstrdup(from, flag);
25 ... when != \(from = E1 \| to = E1 \)
26 if (to==NULL || ...) S
27 ... when != \(from = E2 \| to = E2 \)
28- strcpy(to, from);
29
Julia Lawall29a36d42012-01-14 23:41:54 +010030@depends on patch@
Nicolas Palix998f5522010-08-24 17:39:05 +020031expression x,from,to;
32expression flag,E1,E2,E3;
33statement S;
34@@
35
36- x = strlen(from) + 1;
37 ... when != \( x = E1 \| from = E1 \)
38- to = \(kmalloc\|kzalloc\)(x,flag);
39+ to = kstrdup(from, flag);
40 ... when != \(x = E2 \| from = E2 \| to = E2 \)
41 if (to==NULL || ...) S
42 ... when != \(x = E3 \| from = E3 \| to = E3 \)
43- memcpy(to, from, x);
Julia Lawall29a36d42012-01-14 23:41:54 +010044
45// ---------------------------------------------------------------------
46
47@r1 depends on !patch exists@
48expression from,to;
49expression flag,E1,E2;
50statement S;
51position p1,p2;
52@@
53
54* to = kmalloc@p1(strlen(from) + 1,flag);
55 ... when != \(from = E1 \| to = E1 \)
56 if (to==NULL || ...) S
57 ... when != \(from = E2 \| to = E2 \)
58* strcpy@p2(to, from);
59
60@r2 depends on !patch exists@
61expression x,from,to;
62expression flag,E1,E2,E3;
63statement S;
64position p1,p2;
65@@
66
67* x = strlen(from) + 1;
68 ... when != \( x = E1 \| from = E1 \)
Denis Efremov2fbecb72020-06-19 16:23:07 +030069* to = \(kmalloc@p1\|kzalloc@p1\)(x,flag);
Julia Lawall29a36d42012-01-14 23:41:54 +010070 ... when != \(x = E2 \| from = E2 \| to = E2 \)
71 if (to==NULL || ...) S
72 ... when != \(x = E3 \| from = E3 \| to = E3 \)
73* memcpy@p2(to, from, x);
74
75@script:python depends on org@
76p1 << r1.p1;
77p2 << r1.p2;
78@@
79
Rikard Falkeborna3b0b6f2019-04-08 21:14:32 +020080cocci.print_main("WARNING opportunity for kstrdup",p1)
Julia Lawall29a36d42012-01-14 23:41:54 +010081cocci.print_secs("strcpy",p2)
82
83@script:python depends on org@
84p1 << r2.p1;
85p2 << r2.p2;
86@@
87
Rikard Falkeborna3b0b6f2019-04-08 21:14:32 +020088cocci.print_main("WARNING opportunity for kstrdup",p1)
Julia Lawall29a36d42012-01-14 23:41:54 +010089cocci.print_secs("memcpy",p2)
90
91@script:python depends on report@
92p1 << r1.p1;
93p2 << r1.p2;
94@@
95
Rikard Falkeborna3b0b6f2019-04-08 21:14:32 +020096msg = "WARNING opportunity for kstrdup (strcpy on line %s)" % (p2[0].line)
Julia Lawall29a36d42012-01-14 23:41:54 +010097coccilib.report.print_report(p1[0], msg)
98
99@script:python depends on report@
100p1 << r2.p1;
101p2 << r2.p2;
102@@
103
Rikard Falkeborna3b0b6f2019-04-08 21:14:32 +0200104msg = "WARNING opportunity for kstrdup (memcpy on line %s)" % (p2[0].line)
Julia Lawall29a36d42012-01-14 23:41:54 +0100105coccilib.report.print_report(p1[0], msg)