summaryrefslogtreecommitdiffstats
path: root/lib/libssl/d1_clnt.c
diff options
context:
space:
mode:
authorjsing <jsing@openbsd.org>2015-03-27 12:29:54 +0000
committerjsing <jsing@openbsd.org>2015-03-27 12:29:54 +0000
commit4c5ec7da26a11a298d79cded54aa76d431fdda94 (patch)
tree59e19ccbcd28ea53aebc59248e7f35610a5d20e4 /lib/libssl/d1_clnt.c
parentBUF_MEM_free() has its own explicit NULL check. (diff)
downloadwireguard-openbsd-4c5ec7da26a11a298d79cded54aa76d431fdda94.tar.xz
wireguard-openbsd-4c5ec7da26a11a298d79cded54aa76d431fdda94.zip
Factor out the init_buf initialisation code, rather than duplicating it
in four different places. ok doug@ guenther@
Diffstat (limited to 'lib/libssl/d1_clnt.c')
-rw-r--r--lib/libssl/d1_clnt.c26
1 files changed, 6 insertions, 20 deletions
diff --git a/lib/libssl/d1_clnt.c b/lib/libssl/d1_clnt.c
index cf25183de50..e44c8a0c94c 100644
--- a/lib/libssl/d1_clnt.c
+++ b/lib/libssl/d1_clnt.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: d1_clnt.c,v 1.43 2015/02/09 10:53:28 jsing Exp $ */
+/* $OpenBSD: d1_clnt.c,v 1.44 2015/03/27 12:29:54 jsing Exp $ */
/*
* DTLS implementation written by Nagendra Modadugu
* (nagendra@cs.stanford.edu) for the OpenSSL project 2005.
@@ -176,7 +176,6 @@ dtls1_get_client_method(int ver)
int
dtls1_connect(SSL *s)
{
- BUF_MEM *buf = NULL;
void (*cb)(const SSL *ssl, int type, int val) = NULL;
int ret = -1;
int new_state, state, skip = 0;
@@ -223,25 +222,14 @@ dtls1_connect(SSL *s)
/* s->version=SSL3_VERSION; */
s->type = SSL_ST_CONNECT;
- if (s->init_buf == NULL) {
- if ((buf = BUF_MEM_new()) == NULL) {
- ret = -1;
- goto end;
- }
- if (!BUF_MEM_grow(buf, SSL3_RT_MAX_PLAIN_LENGTH)) {
- ret = -1;
- goto end;
- }
- s->init_buf = buf;
- buf = NULL;
+ if (!ssl3_setup_init_buffer(s)) {
+ ret = -1;
+ goto end;
}
-
if (!ssl3_setup_buffers(s)) {
ret = -1;
goto end;
}
-
- /* setup buffing BIO */
if (!ssl_init_wbio_buffer(s, 0)) {
ret = -1;
goto end;
@@ -603,14 +591,12 @@ dtls1_connect(SSL *s)
}
skip = 0;
}
+
end:
s->in_handshake--;
-
-
- if (buf != NULL)
- BUF_MEM_free(buf);
if (cb != NULL)
cb(s, SSL_CB_CONNECT_EXIT, ret);
+
return (ret);
}