On Tue, 2014-03-18 at 21:07 +0100, Silvio F wrote:
> This patch solves the Coccinelle warning: "kzalloc should be used
> instead of kmalloc/memset"
[]
> diff --git a/drivers/staging/unisys/include/uisutils.h
> b/drivers/staging/unisys/include/uisutils.h
[]
> @@ -190,10 +190,7 @@ struct chaninfo {
> }
>
> #define ALLOC_CMDRSP(cmdrsp) { \
> - cmdrsp = kmalloc(SIZEOF_CMDRSP, GFP_ATOMIC); \
> - if (cmdrsp != NULL) { \
> - memset(cmdrsp, 0, SIZEOF_CMDRSP); \
> - } \
> + cmdrsp = kzalloc(SIZEOF_CMDRSP, GFP_ATOMIC); \
> }
It'd be nicer to get rid of this macro altogether
and substitute kzalloc in the 3 places it's used.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
