Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1547709 > unrolled thread

[PATCH 1/1] Coccinelle: memdup: fix the report mode

Started byNicolas Iooss <nicolas.iooss_linux@m4x.org>
First post2016-12-27 18:50 +0100
Last post2016-12-27 18:50 +0100
Articles 1 — 1 participant

Back to article view | Back to linux.kernel


Contents

  [PATCH 1/1] Coccinelle: memdup: fix the report mode Nicolas Iooss <nicolas.iooss_linux@m4x.org> - 2016-12-27 18:50 +0100

#1547709 — [PATCH 1/1] Coccinelle: memdup: fix the report mode

FromNicolas Iooss <nicolas.iooss_linux@m4x.org>
Date2016-12-27 18:50 +0100
Subject[PATCH 1/1] Coccinelle: memdup: fix the report mode
Message-ID<sT42R-56S-5@gated-at.bofh.it>
The pattern which was used in report mode included an annoying kmemdup
call, which has no chance of being there. Remove this stray line.

While at it, the name of function kmemdup was misspelled in the warning
message. Fix this.

Fixes: 29a36d4dec6c ("scripts/coccinelle: improve the coverage of some
semantic patches")
Signed-off-by: Nicolas Iooss <nicolas.iooss_linux@m4x.org>
---
 scripts/coccinelle/api/memdup.cocci | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/scripts/coccinelle/api/memdup.cocci b/scripts/coccinelle/api/memdup.cocci
index 3d1aa71b7579..8fd6437beda8 100644
--- a/scripts/coccinelle/api/memdup.cocci
+++ b/scripts/coccinelle/api/memdup.cocci
@@ -49,7 +49,6 @@ statement S;
 @@
 
 *  to = \(kmalloc@p\|kzalloc@p\)(size,flag);
-   to = kmemdup(from,size,flag);
    if (to==NULL || ...) S
 *  memcpy(to, from, size);
 
@@ -57,10 +56,10 @@ statement S;
 p << r.p;
 @@
 
-coccilib.org.print_todo(p[0], "WARNING opportunity for kmemdep")
+coccilib.org.print_todo(p[0], "WARNING opportunity for kmemdup")
 
 @script:python depends on report@
 p << r.p;
 @@
 
-coccilib.report.print_report(p[0], "WARNING opportunity for kmemdep")
+coccilib.report.print_report(p[0], "WARNING opportunity for kmemdup")
-- 
2.11.0

[toc] | [standalone]


Back to top | Article view | linux.kernel


csiph-web