diff options
author | miod <> | 2016-11-05 19:59:01 +0000 |
---|---|---|
committer | miod <> | 2016-11-05 19:59:01 +0000 |
commit | 9b44f0d3dbfa90fdde4fa8329c758570e64d8500 (patch) | |
tree | 04c2183b7fa65546d0669ed4b9cf3a502e0b5fc2 /src | |
parent | 26e4ef5835524ee5f3562d63c0da555514dae9aa (diff) | |
download | openbsd-9b44f0d3dbfa90fdde4fa8329c758570e64d8500.tar.gz openbsd-9b44f0d3dbfa90fdde4fa8329c758570e64d8500.tar.bz2 openbsd-9b44f0d3dbfa90fdde4fa8329c758570e64d8500.zip |
One of the error paths would attempt to access not-yet-initialized locals.
Simply return since there is nothing more to do.
Spotted by coverity. ok jsing@ beck@
Diffstat (limited to 'src')
-rw-r--r-- | src/lib/libssl/ssl_asn1.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/lib/libssl/ssl_asn1.c b/src/lib/libssl/ssl_asn1.c index 59e7cdf7b2..02124da520 100644 --- a/src/lib/libssl/ssl_asn1.c +++ b/src/lib/libssl/ssl_asn1.c | |||
@@ -1,4 +1,4 @@ | |||
1 | /* $OpenBSD: ssl_asn1.c,v 1.42 2016/11/04 15:09:29 jsing Exp $ */ | 1 | /* $OpenBSD: ssl_asn1.c,v 1.43 2016/11/05 19:59:01 miod Exp $ */ |
2 | 2 | ||
3 | /* | 3 | /* |
4 | * Copyright (c) 2016 Joel Sing <jsing@openbsd.org> | 4 | * Copyright (c) 2016 Joel Sing <jsing@openbsd.org> |
@@ -231,7 +231,7 @@ d2i_SSL_SESSION(SSL_SESSION **a, const unsigned char **pp, long length) | |||
231 | if (s == NULL) { | 231 | if (s == NULL) { |
232 | if ((s = SSL_SESSION_new()) == NULL) { | 232 | if ((s = SSL_SESSION_new()) == NULL) { |
233 | SSLerr(SSL_F_D2I_SSL_SESSION, ERR_R_MALLOC_FAILURE); | 233 | SSLerr(SSL_F_D2I_SSL_SESSION, ERR_R_MALLOC_FAILURE); |
234 | goto err; | 234 | return (NULL); |
235 | } | 235 | } |
236 | } | 236 | } |
237 | 237 | ||