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

From: Julia Lawall
Date: Tue Dec 27 2016 - 12:44:58 EST




On Tue, 27 Dec 2016, Nicolas Iooss wrote:

> 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@xxxxxxx>

Acked-by: Julia Lawall <julia.lawall@xxxxxxx>

Thanks for the fixes.

julia

> ---
> 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
>
>