NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commitf318ee8

committed

Fix GET DIAGNOSTICS for case of assignment to function's first variable.

An incorrect and entirely unnecessary "safety check" in exec_stmt_getdiag()caused the code to treat an assignment to a variable with dno zero as ano-op. Unfortunately, that's a perfectly valid dno. This has been brokensince GET DIAGNOSTICS was invented. It's not terribly surprising that thebug went unnoticed for so long, since in most cases you probably wouldn'tuse the function's first-created variable (normally its first parameter)as a GET DIAGNOSTICS target. Nonetheless, it's broken. Per bug #6551from Adam Buraczewski.

1 parent5bd06e6 commitf318ee8Copy full SHA for f318ee8

File tree

1 file changed

-9

lines changed

src/pl/plpgsql/src
- pl_exec.c

1 file changed

-9

lines changed

`‎src/pl/plpgsql/src/pl_exec.c`

Lines changed: 1 addition & 9 deletions

Original file line number	Diff line number	Diff line change
`@@ -1430,17 +1430,9 @@ exec_stmt_getdiag(PLpgSQL_execstate estate, PLpgSQL_stmt_getdiag stmt)`
`1430`	`1430`	`foreach(lc,stmt->diag_items)`
`1431`	`1431`	`{`
`1432`	`1432`	`PLpgSQL_diag_itemdiag_item= (PLpgSQL_diag_item)lfirst(lc);`
`1433`		`-PLpgSQL_datum*var;`
	`1433`	`+PLpgSQL_datum*var=estate->datums[diag_item->target];`
`1434`	`1434`	`boolisnull= false;`
`1435`	`1435`
`1436`		`-if (diag_item->target <=0)`
`1437`		`-continue;`
`1438`		`-`
`1439`		`-var=estate->datums[diag_item->target];`
`1440`		`-`
`1441`		`-if (var==NULL)`
`1442`		`-continue;`
`1443`		`-`
`1444`	`1436`	`switch (diag_item->kind)`
`1445`	`1437`	`{`
`1446`	`1438`	`casePLPGSQL_GETDIAG_ROW_COUNT:`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commitf318ee8

File tree

1 file changed

1 file changed

`‎src/pl/plpgsql/src/pl_exec.c`

0 commit comments