diff options
| author |   miod <miod@openbsd.org> | 2016-11-05 19:59:01 +0000 |
|---|---|---|
| committer | miod <miod@openbsd.org> | 2016-11-05 19:59:01 +0000 |
| commit | b084c069af5236044e7cede7a73522c397f5b5d9 (patch) | |
| tree | b1c3a977e5d3ad457b5d7ba50f42c7245e1a23f7 | |
| parent | Do a partial CBB conversion of ssl3_send_server_key_exchange(), which will (diff) | |
| download | wireguard-openbsd-b084c069af5236044e7cede7a73522c397f5b5d9.tar.xz wireguard-openbsd-b084c069af5236044e7cede7a73522c397f5b5d9.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@
| -rw-r--r-- | lib/libssl/ssl_asn1.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/lib/libssl/ssl_asn1.c b/lib/libssl/ssl_asn1.c index 59e7cdf7b25..02124da5204 100644 --- a/lib/libssl/ssl_asn1.c +++ b/lib/libssl/ssl_asn1.c @@ -1,4 +1,4 @@ -/* $OpenBSD: ssl_asn1.c,v 1.42 2016/11/04 15:09:29 jsing Exp $ */ +/* $OpenBSD: ssl_asn1.c,v 1.43 2016/11/05 19:59:01 miod Exp $ */ /* * Copyright (c) 2016 Joel Sing <jsing@openbsd.org> @@ -231,7 +231,7 @@ d2i_SSL_SESSION(SSL_SESSION **a, const unsigned char **pp, long length) if (s == NULL) { if ((s = SSL_SESSION_new()) == NULL) { SSLerr(SSL_F_D2I_SSL_SESSION, ERR_R_MALLOC_FAILURE); - goto err; + return (NULL); } } |