diff options
| author | Emilia Kasper <emilia@openssl.org> | 2015-08-31 13:57:44 +0200 |
|---|---|---|
| committer | Emilia Kasper <emilia@openssl.org> | 2015-08-31 16:31:45 +0200 |
| commit | 4d04226c2ec7e7f69f6234def63631648e35e828 (patch) | |
| tree | 942333475d608694444652756cd4b2ecf7d41439 /test/bntest.c | |
| parent | 9db0c91c39fb548c36d6c3c944f50d4c068eefb7 (diff) | |
| download | openssl-4d04226c2ec7e7f69f6234def63631648e35e828.zip openssl-4d04226c2ec7e7f69f6234def63631648e35e828.tar.gz openssl-4d04226c2ec7e7f69f6234def63631648e35e828.tar.bz2 | |
Fix spurious bntest failures.
BN_bntest_rand generates a single-word zero BIGNUM with quite a large probability.
A zero BIGNUM in turn will end up having a NULL |d|-buffer, which we shouldn't dereference without checking.
Reviewed-by: Richard Levitte <levitte@openssl.org>
Diffstat (limited to 'test/bntest.c')
| -rw-r--r-- | test/bntest.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/test/bntest.c b/test/bntest.c index cf4d2ab..430d2a0 100644 --- a/test/bntest.c +++ b/test/bntest.c @@ -526,9 +526,9 @@ int test_div_word(BIO *bp) do { BN_bntest_rand(a, 512, -1, 0); BN_bntest_rand(b, BN_BITS2, -1, 0); - s = b->d[0]; - } while (!s); + } while (BN_is_zero(b)); + s = b->d[0]; BN_copy(b, a); r = BN_div_word(b, s); |