From: Bruce Momjian <bruce@momjian.us>
Date: Thu, 17 Nov 2011 18:24:55 +0000 (-0500)
Subject: Fix pg_upgrade's pg_scandir_internal() to properly handle a NULL
X-Git-Tag: REL9_1_2~26
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=379aa2f5a40c8f46911d9979f3222f8c5abc59b0;p=thirdparty%2Fpostgresql.git

Fix pg_upgrade's pg_scandir_internal() to properly handle a NULL
pattern, which is used on PG 9.1 and HEAD (but not pre-9.1).  Fixes
crash on Windows.

Backpatched to 9.1.

Reported by Mark Dilger
---

diff --git a/contrib/pg_upgrade/file.c b/contrib/pg_upgrade/file.c
index 02c81a7fc4e..4983e7f5ad9 100644
--- a/contrib/pg_upgrade/file.c
+++ b/contrib/pg_upgrade/file.c
@@ -294,7 +294,7 @@ pg_scandir_internal(const char *dirname,
 	while ((direntry = readdir(dirdesc)) != NULL)
 	{
 		/* Invoke the selector function to see if the direntry matches */
-		if ((*selector) (direntry))
+		if (selector && (*selector) (direntry))
 		{
 			count++;