postgres.git / summary / log / commit / refs

commit    c879e82b3746150171fcec605a6d5b2b21e5c6cf
Author:   Andrew Dunstan <andrew@dunslane.net>
Date:     Mon Sep 03 19:32:58 2012 +0000

    Fix bugs in exec.c that prevented pg_upgrade working in Windows.
    
    Backpatch to 9.2 - code before that is quite different and should
    not have these defects.


 contrib/pg_upgrade/exec.c | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/contrib/pg_upgrade/exec.c b/contrib/pg_upgrade/exec.c
index c75d9dbcc97..ac46a9b9369 100644
--- a/contrib/pg_upgrade/exec.c
+++ b/contrib/pg_upgrade/exec.c
@@ -52,7 +52,7 @@ exec_prog(const char *log_file, const char *opt_log_file,
 
 	old_umask = umask(S_IRWXG | S_IRWXO);
 
-	written = strlcpy(cmd, SYSTEMQUOTE, strlen(SYSTEMQUOTE));
+	written = strlcpy(cmd, SYSTEMQUOTE, sizeof(cmd));
 	va_start(ap, fmt);
 	written += vsnprintf(cmd + written, MAXCMDLEN - written, fmt, ap);
 	va_end(ap);
@@ -95,10 +95,16 @@ exec_prog(const char *log_file, const char *opt_log_file,
 				   log_file);
 	}
 
+#ifndef WIN32
+	/* 
+	 * Can't do this on Windows, postmaster will still hold the log file
+	 * open if the command was "pg_ctl start".
+	 */
 	if ((log = fopen_priv(log_file, "a+")) == NULL)
 		pg_log(PG_FATAL, "cannot write to log file %s\n", log_file);
 	fprintf(log, "\n\n");
 	fclose(log);
+#endif
 
 	return result == 0;
 }


[parent: 5c7e91e9c3d6]