Hi Wei,
On Mon, 2021-11-01 at 14:02 +0000, Wei Yongjun wrote:
> Fix to return negative error code -ENOMEM from the error handling
> case instead of 0, as done elsewhere in this function.
>
> Fixes: c9f608c38009 ("crypto: keembay-ocs-ecc - Add Keem Bay OCS ECC Driver")
> Reported-by: Hulk Robot <hulkci@xxxxxxxxxx>
> Signed-off-by: Wei Yongjun <weiyongjun1@xxxxxxxxxx>
> ---
> drivers/crypto/keembay/keembay-ocs-ecc.c | 1 +
> 1 file changed, 1 insertion(+)
Reviewed-by: Daniele Alessandrelli <daniele.alessandrelli@xxxxxxxxx>
Thanks for the fix!
>
> diff --git a/drivers/crypto/keembay/keembay-ocs-ecc.c b/drivers/crypto/keembay/keembay-ocs-ecc.c
> index 679e6ae295e0..5d0785d3f1b5 100644
> --- a/drivers/crypto/keembay/keembay-ocs-ecc.c
> +++ b/drivers/crypto/keembay/keembay-ocs-ecc.c
> @@ -930,6 +930,7 @@ static int kmb_ocs_ecc_probe(struct platform_device *pdev)
> ecc_dev->engine = crypto_engine_alloc_init(dev, 1);
> if (!ecc_dev->engine) {
> dev_err(dev, "Could not allocate crypto engine\n");
> + rc = -ENOMEM;
> goto list_del;
> }
>
>
