diff options
| author | Pauli <pauli@openssl.org> | 2022-04-01 03:18:44 +0200 |
|---|---|---|
| committer | Pauli <pauli@openssl.org> | 2022-05-06 10:21:22 +0200 |
| commit | 66cb4fcdc5039fe5b1476ed48a936137a307a58b (patch) | |
| tree | 36f0a10a7987501dfa6212f3f60b57f54389a9b1 /crypto/store | |
| parent | Fix coverity 1503330 use after free (diff) | |
| download | openssl-66cb4fcdc5039fe5b1476ed48a936137a307a58b.tar.xz openssl-66cb4fcdc5039fe5b1476ed48a936137a307a58b.zip | |
Fix Coverity 1503329 use after free
Another false positive tagged as such
Reviewed-by: Tomas Mraz <tomas@openssl.org>
Reviewed-by: Shane Lontis <shane.lontis@oracle.com>
(Merged from https://github.com/openssl/openssl/pull/18014)
Diffstat (limited to 'crypto/store')
| -rw-r--r-- | crypto/store/store_lib.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/crypto/store/store_lib.c b/crypto/store/store_lib.c index 696056e370..3d91f159b6 100644 --- a/crypto/store/store_lib.c +++ b/crypto/store/store_lib.c @@ -215,6 +215,8 @@ OSSL_STORE_open_ex(const char *uri, OSSL_LIB_CTX *libctx, const char *propq, */ (void)ossl_store_close_it(&tmpctx); } + /* Coverity false positive, the reference counting is confusing it */ + /* coverity[pass_freed_arg] */ OSSL_STORE_LOADER_free(fetched_loader); OPENSSL_free(propq_copy); OPENSSL_free(ctx); |