clk/Renesas-MSTP: Use kmalloc_array() in cpg_mstp_clocks_init()
authorMarkus Elfring <elfring@users.sourceforge.net>
Wed, 14 Sep 2016 19:10:47 +0000 (21:10 +0200)
committerStephen Boyd <sboyd@codeaurora.org>
Fri, 16 Sep 2016 23:13:09 +0000 (16:13 -0700)
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 <elfring@users.sourceforge.net>
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
Signed-off-by: Stephen Boyd <sboyd@codeaurora.org>
drivers/clk/renesas/clk-mstp.c

index 5093a25..9375777 100644 (file)
@@ -167,7 +167,7 @@ static void __init cpg_mstp_clocks_init(struct device_node *np)
        unsigned int i;
 
        group = kzalloc(sizeof(*group), GFP_KERNEL);
-       clks = kmalloc(MSTP_MAX_CLOCKS * sizeof(*clks), GFP_KERNEL);
+       clks = kmalloc_array(MSTP_MAX_CLOCKS, sizeof(*clks), GFP_KERNEL);
        if (group == NULL || clks == NULL) {
                kfree(group);
                kfree(clks);