[PATCH 1/1] lib/rsa: correct check after allocation in fdt_add_bignum()

[PATCH 1/1] lib/rsa: correct check after allocation in fdt_add_bignum()

[Heinrich Schuchardt]

After allocating to pointer ctx we should check that pointer and not
another pointer already checked above.

Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
---
 lib/rsa/rsa-sign.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/rsa/rsa-sign.c b/lib/rsa/rsa-sign.c
index 40ca1e1f57..1f0d81bd7a 100644
--- a/lib/rsa/rsa-sign.c
+++ b/lib/rsa/rsa-sign.c
@@ -708,7 +708,7 @@ static int fdt_add_bignum(void *blob, int noffset, const char *prop_name,
 		return -ENOMEM;
 	}
 	ctx = BN_CTX_new();
-	if (!tmp) {
+	if (!ctx) {
 		fprintf(stderr, "Out of memory (bignum context)\n");
 		return -ENOMEM;
 	}
--
2.28.0

[PATCH 1/1] lib/rsa: correct check after allocation in fdt_add_bignum()

[Simon Glass]

On Thu, 20 Aug 2020 at 13:11, Heinrich Schuchardt <xypron.glpk@gmx.de> wrote:
>
> After allocating to pointer ctx we should check that pointer and not
> another pointer already checked above.
>
> Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
> ---
>  lib/rsa/rsa-sign.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Reviewed-by: Simon Glass <sjg@chromium.org>

[PATCH 1/1] lib/rsa: correct check after allocation in fdt_add_bignum()

[Tom Rini]

On Thu, Aug 20, 2020 at 09:11:17PM +0200, Heinrich Schuchardt wrote:

> After allocating to pointer ctx we should check that pointer and not
> another pointer already checked above.
> 
> Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
> Reviewed-by: Simon Glass <sjg@chromium.org>

Applied to u-boot/master, thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20200828/e789e151/attachment.sig>