From: Itagaki Takahiro <itagaki.takahiro@gmail.com>
Date: Wed, 9 Jun 2010 01:00:13 +0000 (+0000)
Subject: Fix connection leak in dblink when dblink_connect() or dblink_connect_u()
X-Git-Tag: REL8_1_22~40
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=47cf87a147478818d33ff24a7484a09f8c8d5e96;p=thirdparty%2Fpostgresql.git

Fix connection leak in dblink when dblink_connect() or dblink_connect_u()
end with "duplicate connection name" errors.

Backported to release 7.4.
---

diff --git a/contrib/dblink/dblink.c b/contrib/dblink/dblink.c
index 16f50200818..f67128e7519 100644
--- a/contrib/dblink/dblink.c
+++ b/contrib/dblink/dblink.c
@@ -2151,9 +2151,14 @@ createNewConnection(const char *name, remoteConn * rconn)
 											   HASH_ENTER, &found);
 
 	if (found)
+	{
+		PQfinish(rconn->conn);
+		pfree(rconn);
+
 		ereport(ERROR,
 				(errcode(ERRCODE_DUPLICATE_OBJECT),
 				 errmsg("duplicate connection name")));
+	}
 
 	hentry->rconn = rconn;
 	strncpy(hentry->name, name, NAMEDATALEN - 1);