]> 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:45 +0000 (14:13 -0400)
commitf318ee848c9a7c3dd289fe67245e4a12ba136bca
tree8fb59377e15e93d331f7789c893e62de7c3acc0f
parent5bd06e619c82c3b2e29fed40aae5fc39a9620908
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