summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authormiod <>2016-11-05 19:59:01 +0000
committermiod <>2016-11-05 19:59:01 +0000
commit9b44f0d3dbfa90fdde4fa8329c758570e64d8500 (patch)
tree04c2183b7fa65546d0669ed4b9cf3a502e0b5fc2 /src
parent26e4ef5835524ee5f3562d63c0da555514dae9aa (diff)
downloadopenbsd-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.c4
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