summaryrefslogtreecommitdiff
path: root/Cryptlib/OpenSSL/crypto/bn/bn_lib.c
diff options
context:
space:
mode:
authorPeter Jones <pjones@redhat.com>2021-03-08 11:54:33 -0500
committerJan Setje-Eilers <73182357+jsetje@users.noreply.github.com>2021-03-09 13:52:53 -0800
commit1954ef164a55cbc4369f513cca3fb43f3550d81a (patch)
tree124965e9d3355c9438d645fa0f1e881fe554fedf /Cryptlib/OpenSSL/crypto/bn/bn_lib.c
parentdddeaf3f7dc8d30640119e999bbfabf4ce068a68 (diff)
downloadefi-boot-shim-1954ef164a55cbc4369f513cca3fb43f3550d81a.tar.gz
efi-boot-shim-1954ef164a55cbc4369f513cca3fb43f3550d81a.zip
Minor OpenSSL fixes
These are all the NULL pointer dereferences (which all appear to be, at worst, very difficult to hit) that gcc -fanalyzer finds in our OpenSSL code. Signed-off-by: Peter Jones <pjones@redhat.com>
Diffstat (limited to 'Cryptlib/OpenSSL/crypto/bn/bn_lib.c')
-rw-r--r--Cryptlib/OpenSSL/crypto/bn/bn_lib.c3
1 files changed, 3 insertions, 0 deletions
diff --git a/Cryptlib/OpenSSL/crypto/bn/bn_lib.c b/Cryptlib/OpenSSL/crypto/bn/bn_lib.c
index 10b78f51..2671f35c 100644
--- a/Cryptlib/OpenSSL/crypto/bn/bn_lib.c
+++ b/Cryptlib/OpenSSL/crypto/bn/bn_lib.c
@@ -496,6 +496,9 @@ BIGNUM *BN_copy(BIGNUM *a, const BIGNUM *b)
if (bn_wexpand(a, b->top) == NULL)
return (NULL);
+ if (!a || !b || !a->d || !b->d)
+ return (NULL);
+
#if 1
A = a->d;
B = b->d;