diff options
| author | Kurt Roeckx <kurt@roeckx.be> | 2016-12-15 20:23:52 +0100 |
|---|---|---|
| committer | Kurt Roeckx <kurt@roeckx.be> | 2016-12-15 21:45:25 +0100 |
| commit | eeab356c298248108b82157ef51172ba040646f7 (patch) | |
| tree | 08787dba1cf5bb3fe5ea0b935d31310e58b3f5d5 | |
| parent | poly1305/asm/poly1305-x86_64.pl: allow nasm to assemble AVX512 code. (diff) | |
| download | openssl-eeab356c298248108b82157ef51172ba040646f7.tar.xz openssl-eeab356c298248108b82157ef51172ba040646f7.zip | |
Don't call memcpy with NULL as source
Calling it with lenght 0 and NULL as source is undefined behaviour.
Reviewed-by: Rich Salz <rsalz@openssl.org>
GH: #2089
| -rw-r--r-- | crypto/bn/bn_intern.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/crypto/bn/bn_intern.c b/crypto/bn/bn_intern.c index 9227b6e241..2c970647de 100644 --- a/crypto/bn/bn_intern.c +++ b/crypto/bn/bn_intern.c @@ -167,7 +167,8 @@ int bn_copy_words(BN_ULONG *out, const BIGNUM *in, int size) return 0; memset(out, 0, sizeof(*out) * size); - memcpy(out, in->d, sizeof(*out) * in->top); + if (in->d != NULL) + memcpy(out, in->d, sizeof(*out) * in->top); return 1; } |