libpq: Fix memory leak

petere · petere · commit f9325df0fc6d · 2012-03-07T23:35:03.000+02:00
If a client encoding is specified as a connection parameter (or
environment variable), internal storage allocated for it would never
be freed.
diff --git a/src/interfaces/libpq/fe-connect.c b/src/interfaces/libpq/fe-connect.c
@@ -2749,6 +2749,8 @@ freePGconn(PGconn *conn)
 		free(conn->events[i].name);
 	}
 
+	if (conn->client_encoding_initial)
+		free(conn->client_encoding_initial);
 	if (conn->events)
 		free(conn->events);
 	if (conn->pghost)

Original file line number	Diff line number	Diff line change
`@@ -2749,6 +2749,8 @@ freePGconn(PGconn *conn)`
`2749`	`2749`	`free(conn->events[i].name);`
`2750`	`2750`	`}`
`2751`	`2751`
	`2752`	`+ if (conn->client_encoding_initial)`
	`2753`	`+ free(conn->client_encoding_initial);`
`2752`	`2754`	`if (conn->events)`
`2753`	`2755`	`free(conn->events);`
`2754`	`2756`	`if (conn->pghost)`