Thread (3 messages) 3 messages, 3 authors, 2021-11-20

Re: [PATCH -next] crypto: keembay-ocs-ecc - Fix error return code in kmb_ocs_ecc_probe()

From: Alessandrelli, Daniele <hidden>
Date: 2021-11-02 10:14:54
Also in: kernel-janitors

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 <redacted>
Signed-off-by: Wei Yongjun <redacted>
---
 drivers/crypto/keembay/keembay-ocs-ecc.c | 1 +
 1 file changed, 1 insertion(+)
Reviewed-by: Daniele Alessandrelli <redacted>

Thanks for the fix!
quoted hunk ↗ jump to hunk
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;
        }
 
  
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help