Skip to content

Commit

Permalink
clk-kona-setup: Use kmalloc_array() in parent_process()
Browse files Browse the repository at this point in the history
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: Alex Elder <elder@linaro.org>
[sboyd@codeaurora.org: Save a line]
Signed-off-by: Stephen Boyd <sboyd@codeaurora.org>
  • Loading branch information
Markus Elfring authored and Stephen Boyd committed Sep 14, 2016
1 parent ece5974 commit 6e2e7c9
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions drivers/clk/bcm/clk-kona-setup.c
Original file line number Diff line number Diff line change
Expand Up @@ -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);
Expand Down

0 comments on commit 6e2e7c9

Please sign in to comment.