Fix for adding \n for zero-length win32 read_pipe return

bmomjian · bmomjian · commit 23717760d1eb · 2004-08-16T02:46:36.000Z
diff --git a/src/port/exec.c b/src/port/exec.c
@@ -7,7 +7,7 @@
  *
  *
  * IDENTIFICATION
- *	  $PostgreSQL: pgsql/src/port/exec.c,v 1.22 2004/08/16 01:26:31 momjian Exp $
+ *	  $PostgreSQL: pgsql/src/port/exec.c,v 1.23 2004/08/16 02:46:36 momjian Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -402,7 +402,7 @@ static char *pipe_read_line(char *cmd, char *line, int maxsize)
 			 *	We emulate fgets() behaviour. So if there is no newline
 			 *	at the end, we add one...
 			 */
-			if (line[len-1] != '\n')
+			if (len == 0 || line[len-1] != '\n')
 				strcat(line,"\n");
 		}
 

Original file line number	Diff line number	Diff line change
`@@ -7,7 +7,7 @@`
`7`	`7`	`*`
`8`	`8`	`*`
`9`	`9`	`* IDENTIFICATION`
`10`		`- * $PostgreSQL: pgsql/src/port/exec.c,v 1.22 2004/08/16 01:26:31 momjian Exp $`
	`10`	`+ * $PostgreSQL: pgsql/src/port/exec.c,v 1.23 2004/08/16 02:46:36 momjian Exp $`
`11`	`11`	`*`
`12`	`12`	`*-------------------------------------------------------------------------`
`13`	`13`	`*/`
`@@ -402,7 +402,7 @@ static char pipe_read_line(char cmd, char *line, int maxsize)`
`402`	`402`	`* We emulate fgets() behaviour. So if there is no newline`
`403`	`403`	`* at the end, we add one...`
`404`	`404`	`*/`
`405`		`- if (line[len-1] != '\n')`
	`405`	`+ if (len == 0 \|\| line[len-1] != '\n')`
`406`	`406`	`strcat(line,"\n");`
`407`	`407`	`}`
`408`	`408`