diff options
author | Pauli <pauli@openssl.org> | 2022-07-29 12:08:04 +1000 |
---|---|---|
committer | Hugo Landau <hlandau@openssl.org> | 2022-08-01 08:18:36 +0100 |
commit | 76ad9ae6fa459af0bd804c01d3d681ec02cddb4b (patch) | |
tree | 839ec142529f29016103947e35af7510fd0de134 | |
parent | 83ab43da0c9f67c5069605552b1332ca5fadecf1 (diff) | |
download | openssl-76ad9ae6fa459af0bd804c01d3d681ec02cddb4b.tar.gz |
Coverity 1507484: dereference before null check
Reviewed-by: Hugo Landau <hlandau@openssl.org>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/18907)
-rw-r--r-- | ssl/tls_srp.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/ssl/tls_srp.c b/ssl/tls_srp.c index 2f1d2198fd..962e4d7074 100644 --- a/ssl/tls_srp.c +++ b/ssl/tls_srp.c @@ -90,9 +90,9 @@ int SSL_SRP_CTX_free(SSL *s) */ int ssl_srp_ctx_init_intern(SSL_CONNECTION *s) { - SSL_CTX *ctx = SSL_CONNECTION_GET_CTX(s); + SSL_CTX *ctx; - if (s == NULL || ctx == NULL) + if (s == NULL || (ctx = SSL_CONNECTION_GET_CTX(s)) == NULL) return 0; memset(&s->srp_ctx, 0, sizeof(s->srp_ctx)); |