pgsql: Fix GET DIAGNOSTICS for case of assignment to function's first v - Mailing list pgsql-committers

From Tom Lane
Subject pgsql: Fix GET DIAGNOSTICS for case of assignment to function's first v
Date
Msg-id E1SAmXC-0006EY-CF@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
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.

Branch
------
REL9_0_STABLE

Details
-------
http://git.postgresql.org/pg/commitdiff/7bdf9b863f57f13fe73b3bed1c456aa905068272

Modified Files
--------------
src/pl/plpgsql/src/pl_exec.c |   10 +---------
1 files changed, 1 insertions(+), 9 deletions(-)


pgsql-committers by date:

Previous
From: Tom Lane
Date:
Subject: pgsql: Refactor to eliminate duplicate copies of conninfo default-findi
Next
From: Tom Lane
Date:
Subject: pgsql: Fix GET DIAGNOSTICS for case of assignment to function's first v