pgsql: Fix GET DIAGNOSTICS for case of assignment to function's first v

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: pgsql-committers(at)postgresql(dot)org
Subject: pgsql: Fix GET DIAGNOSTICS for case of assignment to function's first v
Date: 2012-03-22 18:14:50
Message-ID: E1SAmXC-0006EU-A5@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: 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
------
master

Details
-------
http://git.postgresql.org/pg/commitdiff/88a4cb30a4c48a6b11fd4641855595d2678aa123

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

Browse pgsql-committers by date

  From Date Subject
Next Message Robert Haas 2012-03-22 19:41:12 pgsql: Update docs on numeric storage requirements.
Previous Message Tom Lane 2012-03-22 16:09:19 pgsql: Refactor to eliminate duplicate copies of conninfo default-findi