libpq: fix sslcompression leak

Started by Marko Kreenalmost 14 years ago2 messages
#1Marko Kreen
markokr@gmail.com
1 attachment(s)

freePGconn() does not free sslcompression. Fix.

Found with valgrind.

--
marko

Attachments:

fix_sslcomp.difftext/x-diff; charset=us-asciiDownload
diff --git a/src/interfaces/libpq/fe-connect.c b/src/interfaces/libpq/fe-connect.c
index 5add143..4605e49 100644
--- a/src/interfaces/libpq/fe-connect.c
+++ b/src/interfaces/libpq/fe-connect.c
@@ -2803,6 +2803,8 @@ freePGconn(PGconn *conn)
 		free(conn->sslrootcert);
 	if (conn->sslcrl)
 		free(conn->sslcrl);
+	if (conn->sslcompression)
+		free(conn->sslcompression);
 	if (conn->requirepeer)
 		free(conn->requirepeer);
 #if defined(KRB5) || defined(ENABLE_GSS) || defined(ENABLE_SSPI)
#2Magnus Hagander
magnus@hagander.net
In reply to: Marko Kreen (#1)
Re: libpq: fix sslcompression leak

On Wed, Feb 1, 2012 at 13:56, Marko Kreen <markokr@gmail.com> wrote:

freePGconn() does not free sslcompression.  Fix.

Thanks, applied.

--
 Magnus Hagander
 Me: http://www.hagander.net/
 Work: http://www.redpill-linpro.com/