commit 8d6ad9e39d3e104427c133ac45ce05f30da9f9a3
author Richard Levitte <levitte@openssl.org> Sat Dec 21 23:49:21 2002 +0000
committer Richard Levitte <levitte@openssl.org> Sat Dec 21 23:49:21 2002 +0000
tree f18a6ce6c1ff25b3b7503659e86783f05fda5dc3
parent aa18245f7eb0837f1ddd3e057cc1e65bae4f821d

Stop a possible memory leak.
(I wonder why s2_connect() handles the initial buffer allocation slightly
differently...)
PR: 416

ssl/s3_clnt.c

diff --git a/ssl/s3_clnt.c b/ssl/s3_clnt.c
index aff0d9e..477b681 100644
--- a/ssl/s3_clnt.c
+++ b/ssl/s3_clnt.c
@@ -183,7 +183,7 @@
 
 int ssl3_connect(SSL *s)
 	{
-	BUF_MEM *buf;
+	BUF_MEM *buf=NULL;
 	unsigned long Time=time(NULL),l;
 	long num1;
 	void (*cb)(const SSL *ssl,int type,int val)=NULL;
@@ -244,6 +244,7 @@
 					goto end;
 					}
 				s->init_buf=buf;
+				buf=NULL;
 				}
 
 			if (!ssl3_setup_buffers(s)) { ret= -1; goto end; }
@@ -529,6 +530,8 @@
 		}
 end:
 	s->in_handshake--;
+	if (buf != NULL)
+		BUF_MEM_free(buf);
 	if (cb != NULL)
 		cb(s,SSL_CB_CONNECT_EXIT,ret);
 	return(ret);