]> granicus.if.org Git - postgresql/commit
Fix GET DIAGNOSTICS for case of assignment to function's first variable.
authorTom Lane <tgl@sss.pgh.pa.us>
Thu, 22 Mar 2012 18:13:17 +0000 (14:13 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Thu, 22 Mar 2012 18:13:17 +0000 (14:13 -0400)
commit88a4cb30a4c48a6b11fd4641855595d2678aa123
tree87c193d3f0e106612ccb0fad03d3a302d5bcce43
parente9ce658b6203e9052bff5c89ed6265fba69cdb36
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 a
no-op.  Unfortunately, that's a perfectly valid dno.  This has been broken
since GET DIAGNOSTICS was invented.  It's not terribly surprising that the
bug went unnoticed for so long, since in most cases you probably wouldn't
use the function's first-created variable (normally its first parameter)
as a GET DIAGNOSTICS target.  Nonetheless, it's broken.  Per bug #6551
from Adam Buraczewski.
src/pl/plpgsql/src/pl_exec.c