Re: [PATCH] OPP: switch to use kmemdup_array()

From: Viresh Kumar
Date: Mon May 19 2025 - 06:09:14 EST


On 15-05-25, 20:13, zhang.enpei@xxxxxxxxxx wrote:
> From: Zhang Enpei <zhang.enpei@xxxxxxxxxx>
>
> Use kmemdup_array() to avoid multiplication and possible overflows.
>
> Signed-off-by: Zhang Enpei <zhang.enpei@xxxxxxxxxx>
> ---
> drivers/opp/core.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/opp/core.c b/drivers/opp/core.c
> index 72fbb6cadc23..d85e3371493f 100644
> --- a/drivers/opp/core.c
> +++ b/drivers/opp/core.c
> @@ -2087,8 +2087,8 @@ static int _opp_set_supported_hw(struct opp_table *opp_table,
> if (opp_table->supported_hw)
> return 0;
>
> - opp_table->supported_hw = kmemdup(versions, count * sizeof(*versions),
> - GFP_KERNEL);
> + opp_table->supported_hw = kmemdup_array(versions, count, sizeof(*versions),
> + GFP_KERNEL);
> if (!opp_table->supported_hw)
> return -ENOMEM;

Applied. Thanks.

--
viresh