> -----Original Message-----
> From: Kees Cook [mailto:keesc...@chromium.org]
> Sent: Wednesday, July 04, 2018 8:29 PM
> To: Darren Hart <dvh...@infradead.org>
> Cc: Vadim Pasternak <vad...@mellanox.com>; Andy Shevchenko
> <a...@infradead.org>; platform-driver-...@vger.kernel.org; linux-
> ker...@vger.kernel.org
> Subject: [PATCH] platform/mellanox: Use 2-factor allocator calls
> 
> As already done treewide, switch from open-coded multiplication to using 2-
> factor allocation helpers.
> 
> Signed-off-by: Kees Cook <keesc...@chromium.org>

Acked-by: Vadim Pasternak <vad...@mellanox.com>

> ---
>  drivers/platform/mellanox/mlxreg-io.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/platform/mellanox/mlxreg-io.c
> b/drivers/platform/mellanox/mlxreg-io.c
> index c192dfe6bd65..acfaf64ffde6 100644
> --- a/drivers/platform/mellanox/mlxreg-io.c
> +++ b/drivers/platform/mellanox/mlxreg-io.c
> @@ -152,8 +152,8 @@ static int mlxreg_io_attr_init(struct mlxreg_io_priv_data
> *priv)  {
>       int i;
> 
> -     priv->group.attrs = devm_kzalloc(&priv->pdev->dev,
> -                                      priv->pdata->counter *
> +     priv->group.attrs = devm_kcalloc(&priv->pdev->dev,
> +                                      priv->pdata->counter,
>                                        sizeof(struct attribute *),
>                                        GFP_KERNEL);
>       if (!priv->group.attrs)
> --
> 2.17.1
> 
> 
> --
> Kees Cook
> Pixel Security

Reply via email to