From 6e2e7c9fdae316bfb2724d2dbf230678d3f09092 Mon Sep 17 00:00:00 2001 From: Markus Elfring Date: Wed, 14 Sep 2016 20:26:05 +0200 Subject: [PATCH] clk-kona-setup: Use kmalloc_array() in parent_process() A multiplication for the size determination of a memory allocation indicated that an array data structure should be processed. Thus use the corresponding function "kmalloc_array". This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring Reviewed-by: Alex Elder [sboyd@codeaurora.org: Save a line] Signed-off-by: Stephen Boyd --- drivers/clk/bcm/clk-kona-setup.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/clk/bcm/clk-kona-setup.c b/drivers/clk/bcm/clk-kona-setup.c index f2359b3b73bc..c37a7f0e83aa 100644 --- a/drivers/clk/bcm/clk-kona-setup.c +++ b/drivers/clk/bcm/clk-kona-setup.c @@ -586,8 +586,8 @@ static u32 *parent_process(const char *clocks[], } /* There is at least one parent, so allocate a selector array */ - - parent_sel = kmalloc(parent_count * sizeof(*parent_sel), GFP_KERNEL); + parent_sel = kmalloc_array(parent_count, sizeof(*parent_sel), + GFP_KERNEL); if (!parent_sel) { pr_err("%s: error allocating %u parent selectors\n", __func__, parent_count); -- 2.39.2