From: Tom Lane <tgl@sss.pgh.pa.us>
Date: Wed, 8 Aug 2001 22:25:45 +0000 (+0000)
Subject: Back-patch fix to prevent infinite loop when $PGDATA is not writable.
X-Git-Tag: REL7_1_3~14
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=a11c8be973a253aad75fce6f65805beabba92bdd;p=thirdparty%2Fpostgresql.git

Back-patch fix to prevent infinite loop when $PGDATA is not writable.
---

diff --git a/src/backend/utils/init/miscinit.c b/src/backend/utils/init/miscinit.c
index 33b9e11203e..956acdede4f 100644
--- a/src/backend/utils/init/miscinit.c
+++ b/src/backend/utils/init/miscinit.c
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *	  $Header: /cvsroot/pgsql/src/backend/utils/init/miscinit.c,v 1.65 2001/04/16 02:42:01 tgl Exp $
+ *	  $Header: /cvsroot/pgsql/src/backend/utils/init/miscinit.c,v 1.65.2.1 2001/08/08 22:25:45 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -490,15 +490,18 @@ CreateLockFile(const char *filename, bool amPostmaster,
 {
 	int			fd;
 	char		buffer[MAXPGPATH + 100];
+	int			ntries;
 	int			len;
 	int			encoded_pid;
 	pid_t		other_pid;
 	pid_t		my_pid = getpid();
 
 	/*
-	 * We need a loop here because of race conditions.
+	 * We need a loop here because of race conditions.  But don't loop
+	 * forever (for example, a non-writable $PGDATA directory might cause
+	 * a failure that won't go away).  100 tries seems like plenty.
 	 */
-	for (;;)
+	for (ntries = 0; ; ntries++)
 	{
 
 		/*
@@ -511,7 +514,7 @@ CreateLockFile(const char *filename, bool amPostmaster,
 		/*
 		 * Couldn't create the pid file. Probably it already exists.
 		 */
-		if (errno != EEXIST && errno != EACCES)
+		if ((errno != EEXIST && errno != EACCES) || ntries > 100)
 			elog(FATAL, "Can't create lock file %s: %m", filename);
 
 		/*