Commit4b98d42

Michael Meskes

committed

Added missing braces to prevent a segfault after usage of an undeclared cursor.

1 parent6bf44d5 commit4b98d42Copy full SHA for 4b98d42

File tree

+10

-2

lines changed

+10

-2

lines changed

Lines changed: 5 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -2035,5 +2035,10 @@ Mo Jun 26 11:05:25 CEST 2006`
`2035`	`2035`
`2036`	`2036`	`- Added some more coverity report patches send in by Joachim Wieland`
`2037`	`2037`	`<joe@mcknight.de>.`
	`2038`	`+`
	`2039`	`+Mo Jun 26 16:08:23 CEST 2006`
	`2040`	`+`
	`2041`	`+- Added missing braces to prevent a segfault after usage of an`
	`2042`	`+ undeclared cursor.`
`2038`	`2043`	`- Set ecpg library version to 5.2.`
`2039`	`2044`	`- Set ecpg version to 4.2.1.`

Lines changed: 5 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`		`-/* $PostgreSQL: pgsql/src/interfaces/ecpg/preproc/preproc.y,v 1.324 2006/06/06 11:31:55 meskes Exp $*/`
	`1`	`+/* $PostgreSQL: pgsql/src/interfaces/ecpg/preproc/preproc.y,v 1.325 2006/06/26 14:12:02 meskes Exp $*/`
`2`	`2`
`3`	`3`	`/* Copyright comment*/`
`4`	`4`	`%{`
`@@ -292,6 +292,7 @@ add_additional_variables(char *name, bool insert)`
`292`	`292`	`mmerror(PARSE_ERROR, ET_ERROR,"trying to access an undeclared cursor %s\n", name);`
`293`	`293`	`returnNULL;`
`294`	`294`	`}`
	`295`	`+`
`295`	`296`	`if (insert)`
`296`	`297`	`{`
`297`	`298`	`/* add all those input variables that were given earlier`
`@@ -837,9 +838,11 @@ stmt: AlterDatabaseStmt{ output_statement($1, 0, connection); }`
`837`	`838`	`structcursor *ptr;`
`838`	`839`
`839`	`840`	`if ((ptr = add_additional_variables($1,true)) !=NULL)`
	`841`	`+{`
`840`	`842`	`output_statement(mm_strdup(ptr->command), 0,`
`841`	`843`	`ptr->connection ? mm_strdup(ptr->connection) : NULL);`
`842`		`-ptr->opened =true;`
	`844`	`+ptr->opened =true;`
	`845`	`+}`
`843`	`846`	`}`
`844`	`847`	`\|ECPGPrepare`
`845`	`848`	`{`

Comments

(0)