diff options
| author | Shane Lontis <shane.lontis@oracle.com> | 2019-06-04 11:32:58 +1000 |
|---|---|---|
| committer | Shane Lontis <shane.lontis@oracle.com> | 2019-06-06 09:34:17 +1000 |
| commit | ad14e8e5085936bb495d15f4e0a1b653460ae4dd (patch) | |
| tree | 585b98a4bc7f4ea901a48ecb6da057c42612df95 /test/enginetest.c | |
| parent | b1f692575411a400c9c9d4f2d1f0377863abaca5 (diff) | |
Coverity fixes
covID 1445689 Resource leak (in error path)
covID 1445318 Resource leak (in test - minor)
covID 1443705 Unchecked return value (Needed if CRYPTO_atomic_add() was used)
covID 1443691 Resource leak (in app - minor)
Reviewed-by: Matt Caswell <matt@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/9071)
Diffstat (limited to 'test/enginetest.c')
| -rw-r--r-- | test/enginetest.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/test/enginetest.c b/test/enginetest.c index 4836cbe92f..b4d117efa2 100644 --- a/test/enginetest.c +++ b/test/enginetest.c @@ -279,7 +279,7 @@ static int test_redirect(void) * Try setting test key engine. Both should fail because the * engine has no public key methods. */ - if (!TEST_ptr_null(EVP_PKEY_CTX_new(pkey, e)) + if (!TEST_ptr_null(ctx = EVP_PKEY_CTX_new(pkey, e)) || !TEST_int_le(EVP_PKEY_set1_engine(pkey, e), 0)) goto err; |