Prev: [PATCH 4/4] v4l: Remove struct v4l2_file_operations::ioctl
Next: [PATCH] x86, MCE, fix MSR_IA32_MCI_CTL2 CMCI threshold setup
From: Kalle Valo on 17 May 2010 03:50
On 16 May 2010 00:15, Julia Lawall <julia(a)diku.dk> wrote:
> From: Julia Lawall <julia(a)diku.dk>
>
> Use kmemdup when some other buffer is immediately copied into the
> allocated region.

Thanks, looks good to me.

> Signed-off-by: Julia Lawall <julia(a)diku.dk>

Acked-by: Kalle Valo <kvalo(a)adurom.com>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo(a)vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
From: Luciano Coelho on 17 May 2010 04:10
On Sat, 2010-05-15 at 23:16 +0200, ext Julia Lawall wrote:
> From: Julia Lawall <julia(a)diku.dk>
>
> Use kmemdup when some other buffer is immediately copied into the
> allocated region.
>
> A simplified version of the semantic patch that makes this change is as
> follows: (http://coccinelle.lip6.fr/)
>
> // <smpl>
> @@
> expression from,to,size,flag;
> statement S;
> @@
>
> - to = \(kmalloc\|kzalloc\)(size,flag);
> + to = kmemdup(from,size,flag);
> if (to==NULL || ...) S
> - memcpy(to, from, size);
> // </smpl>
>
> Signed-off-by: Julia Lawall <julia(a)diku.dk>

Thanks.

Acked-by: Luciano Coelho <luciano.coelho(a)nokia.com>

--
Cheers,
Luca.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo(a)vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
 | 
Pages: 1
Prev: [PATCH 4/4] v4l: Remove struct v4l2_file_operations::ioctl
Next: [PATCH] x86, MCE, fix MSR_IA32_MCI_CTL2 CMCI threshold setup