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:47:21 +0100 |
| commit | 3c55aec6e9f68ec45162e85434d2911fa67e8f15 (patch) | |
| tree | db202b0df7af4d40b815e87af75df91d137a9003 /crypto/bn | |
| parent | 0baae1c01f4975cc6b92b59e34d1a2a05ae48414 (diff) | |
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
(cherry picked from commit eeab356c298248108b82157ef51172ba040646f7)
Diffstat (limited to 'crypto/bn')
| -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; } |