Use sizeof() for snprintf() buffer length.

bmomjian · bmomjian · commit a6017cff0b37 · 2007-01-29T20:22:35.000Z
diff --git a/src/port/exec.c b/src/port/exec.c
@@ -9,7 +9,7 @@
  *
  *
  * IDENTIFICATION
- *	  $PostgreSQL: pgsql/src/port/exec.c,v 1.54 2007/01/29 20:17:40 momjian Exp $
+ *	  $PostgreSQL: pgsql/src/port/exec.c,v 1.55 2007/01/29 20:22:35 momjian Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -590,7 +590,7 @@ pclose_check(FILE *stream)
 	{
 		char str[256];
 
-		snprintf(str, 256, "%d: %s", WTERMSIG(exitstatus),
+		snprintf(str, sizeof(str), "%d: %s", WTERMSIG(exitstatus),
 			  WTERMSIG(exitstatus) < NSIG ?
 			  sys_siglist[WTERMSIG(exitstatus)] : "(unknown)");
 		log_error(_("child process was terminated by signal %s"), str);

Original file line number	Diff line number	Diff line change
`@@ -9,7 +9,7 @@`
`9`	`9`	`*`
`10`	`10`	`*`
`11`	`11`	`* IDENTIFICATION`
`12`		`- * $PostgreSQL: pgsql/src/port/exec.c,v 1.54 2007/01/29 20:17:40 momjian Exp $`
	`12`	`+ * $PostgreSQL: pgsql/src/port/exec.c,v 1.55 2007/01/29 20:22:35 momjian Exp $`
`13`	`13`	`*`
`14`	`14`	`*-------------------------------------------------------------------------`
`15`	`15`	`*/`
`@@ -590,7 +590,7 @@ pclose_check(FILE *stream)`
`590`	`590`	`{`
`591`	`591`	`char str[256];`
`592`	`592`
`593`		`- snprintf(str, 256, "%d: %s", WTERMSIG(exitstatus),`
	`593`	`+ snprintf(str, sizeof(str), "%d: %s", WTERMSIG(exitstatus),`
`594`	`594`	`WTERMSIG(exitstatus) < NSIG ?`
`595`	`595`	`sys_siglist[WTERMSIG(exitstatus)] : "(unknown)");`
`596`	`596`	`log_error(_("child process was terminated by signal %s"), str);`