From: Michael Paquier <michael@paquier.xyz>
Date: Sun, 13 Jun 2021 11:08:06 +0000 (+0900)
Subject: Ignore more environment variables in pg_regress.c
X-Git-Tag: REL9_6_23~78
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=1df6bee296ed2be0fda11ee7aa42853cf0469f8d;p=thirdparty%2Fpostgresql.git

Ignore more environment variables in pg_regress.c

This is similar to the work done in 8279f68 for TestLib.pm, where
environment variables set may cause unwanted failures if using a
temporary installation with pg_regress.  The list of variables reset is
adjusted in each stable branch depending on what is supported.

Comments are added to remember that the lists in TestLib.pm and
pg_regress.c had better be kept in sync.

Reviewed-by: Álvaro Herrera
Discussion: https://postgr.es/m/YMNR9GYDn+fHlMta@paquier.xyz
Backpatch-through: 9.6
---

diff --git a/src/test/perl/TestLib.pm b/src/test/perl/TestLib.pm
index 1536cc9ba2a..35c4732b9e4 100644
--- a/src/test/perl/TestLib.pm
+++ b/src/test/perl/TestLib.pm
@@ -51,6 +51,7 @@ BEGIN
 	delete $ENV{LC_ALL};
 	$ENV{LC_MESSAGES} = 'C';
 
+	# This list should be kept in sync with pg_regress.c.
 	my @envkeys = qw (
 	  PGCLIENTENCODING
 	  PGCONNECT_TIMEOUT
diff --git a/src/test/regress/pg_regress.c b/src/test/regress/pg_regress.c
index 56f277bef69..8c9f13c812a 100644
--- a/src/test/regress/pg_regress.c
+++ b/src/test/regress/pg_regress.c
@@ -797,14 +797,31 @@ initialize_environment(void)
 		 * we also use psql's -X switch consistently, so that ~/.psqlrc files
 		 * won't mess things up.)  Also, set PGPORT to the temp port, and set
 		 * PGHOST depending on whether we are using TCP or Unix sockets.
+		 *
+		 * This list should be kept in sync with TestLib.pm.
 		 */
+		/* PGCLIENTENCODING, see above */
+		unsetenv("PGCONNECT_TIMEOUT");
+		unsetenv("PGDATA");
 		unsetenv("PGDATABASE");
-		unsetenv("PGUSER");
+		unsetenv("PGGSSLIB");
+		/* PGHOSTADDR, see below */
+		unsetenv("PGKRBSRVNAME");
+		unsetenv("PGPASSFILE");
+		unsetenv("PGPASSWORD");
+		unsetenv("PGREQUIREPEER");
+		unsetenv("PGREQUIRESSL");
 		unsetenv("PGSERVICE");
+		unsetenv("PGSERVICEFILE");
+		unsetenv("PGSSLCERT");
+		unsetenv("PGSSLCRL");
+		unsetenv("PGSSLKEY");
 		unsetenv("PGSSLMODE");
-		unsetenv("PGREQUIRESSL");
-		unsetenv("PGCONNECT_TIMEOUT");
-		unsetenv("PGDATA");
+		unsetenv("PGSSLROOTCERT");
+		unsetenv("PGUSER");
+		/* PGPORT, see below */
+		/* PGHOST, see below */
+
 #ifdef HAVE_UNIX_SOCKETS
 		if (hostname != NULL)
 			doputenv("PGHOST", hostname);