Fix buffer overflow on particularly named files and clarify documentation about

Michael Meskes · Michael Meskes · commit 4032ef18d06a · 2016-12-22T08:28:13.000+01:00
output file naming.

Patch by Tsunakawa, Takayuki &lt;tsunakawa.takay@jp.fujitsu.com&gt;
diff --git a/doc/src/sgml/ref/ecpg-ref.sgml b/doc/src/sgml/ref/ecpg-ref.sgml
@@ -42,11 +42,9 @@ PostgreSQL documentation
   <para>
    <command>ecpg</command> will convert each input file given on the
    command line to the corresponding C output file.  Input files
-   preferably have the extension <filename>.pgc</filename>, in which
-   case the extension will be replaced by <filename>.c</filename> to
-   determine the output file name.  If the extension of the input file
-   is not <filename>.pgc</filename>, then the output file name is
-   computed by appending <literal>.c</literal> to the full file name.
+   preferably have the extension <filename>.pgc</filename>.
+   The extension will be replaced by <filename>.c</filename> to
+   determine the output file name.
    The output file name can also be overridden using the
    <option>-o</option> option.
   </para>
diff --git a/src/interfaces/ecpg/preproc/ecpg.c b/src/interfaces/ecpg/preproc/ecpg.c
@@ -313,7 +313,8 @@ main(int argc, char *const argv[])
 					base_yyout = stdout;
 				else
 				{
-					output_filename = mm_strdup(input_filename);
+					output_filename = mm_alloc(strlen(input_filename) + 3);
+					strcpy(output_filename, input_filename);
 
 					ptr2ext = strrchr(output_filename, '.');
 					/* make extension = .c resp. .h */

Original file line number	Diff line number	Diff line change
`@@ -313,7 +313,8 @@ main(int argc, char *const argv[])`
`313`	`313`	`base_yyout = stdout;`
`314`	`314`	`else`
`315`	`315`	`{`
`316`		`- output_filename = mm_strdup(input_filename);`
	`316`	`+ output_filename = mm_alloc(strlen(input_filename) + 3);`
	`317`	`+ strcpy(output_filename, input_filename);`
`317`	`318`
`318`	`319`	`ptr2ext = strrchr(output_filename, '.');`
`319`	`320`	`/* make extension = .c resp. .h */`