diff options
author | Rich Salz <rsalz@openssl.org> | 2017-04-20 13:03:11 -0400 |
---|---|---|
committer | Rich Salz <rsalz@openssl.org> | 2017-04-20 13:03:11 -0400 |
commit | 9d9d2879627ba796db9dd7c688302f36c635a3c6 (patch) | |
tree | 8dbab197c5955d61f23cf5e29534988079b9d710 | |
parent | 112c4e02680eeb3f9369f270f009a577fdb4019b (diff) |
fix dh_test.
The issues were introduced by commit 93d0298.
Reviewed-by: Matt Caswell <matt@openssl.org>
Reviewed-by: Rich Salz <rsalz@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/3263)
-rw-r--r-- | test/dhtest.c | 23 |
1 files changed, 14 insertions, 9 deletions
diff --git a/test/dhtest.c b/test/dhtest.c index 196535b43e..eb6ec0e963 100644 --- a/test/dhtest.c +++ b/test/dhtest.c @@ -45,7 +45,7 @@ static int dh_test(void) unsigned char *abuf = NULL; unsigned char *bbuf = NULL; int i, alen, blen, aout, bout; - int ret = 1; + int ret = 0; RAND_seed(rnd_seed, sizeof rnd_seed); @@ -59,12 +59,14 @@ static int dh_test(void) if (!DH_check(a, &i)) goto err; - if (TEST_false(i & DH_CHECK_P_NOT_PRIME) - || TEST_false(i & DH_CHECK_P_NOT_SAFE_PRIME) - || TEST_false(i & DH_UNABLE_TO_CHECK_GENERATOR) - || TEST_false(i & DH_NOT_SUITABLE_GENERATOR)) + if (!TEST_false(i & DH_CHECK_P_NOT_PRIME) + || !TEST_false(i & DH_CHECK_P_NOT_SAFE_PRIME) + || !TEST_false(i & DH_UNABLE_TO_CHECK_GENERATOR) + || !TEST_false(i & DH_NOT_SUITABLE_GENERATOR)) goto err; + DH_get0_pqg(a, &ap, NULL, &ag); + if (!TEST_ptr(b = DH_new())) goto err; @@ -76,23 +78,26 @@ static int dh_test(void) if (!DH_generate_key(a)) goto err; + DH_get0_key(a, &apub_key, NULL); if (!DH_generate_key(b)) goto err; + DH_get0_key(b, &bpub_key, NULL); alen = DH_size(a); if (!TEST_ptr(abuf = OPENSSL_malloc(alen)) - || !TEST_true(aout = DH_compute_key(abuf, bpub_key, a))) + || !TEST_true((aout = DH_compute_key(abuf, bpub_key, a)) != -1)) goto err; blen = DH_size(b); if (!TEST_ptr(bbuf = OPENSSL_malloc(blen)) - || !TEST_true(bout = DH_compute_key(bbuf, apub_key, b))) + || !TEST_true((bout = DH_compute_key(bbuf, apub_key, b)) != -1)) goto err; - if (!TEST_true(aout < 4) + + if (!TEST_true(aout >= 4) || !TEST_mem_eq(abuf, aout, bbuf, bout)) goto err; - ret = 0; + ret = 1; err: |