Skip to main content

Browse the archive

https://github.com/postgres/postgres
02 May 2024, 11:03:54 UTC
Revision 6eb61d578f3c9fc4fff78d69851b7558821668e3 authored by Tom Lane on 08 February 2007, 18:38:28 UTC, committed by Tom Lane on 08 February 2007, 18:38:28 UTC 
Rearrange use of plpgsql_add_initdatums() so that only the parsing of a
 
DECLARE section needs to know about it.  Formerly, everyplace besides DECLARE
that created variables needed to do "plpgsql_add_initdatums(NULL)" to prevent
those variables from being sucked up as part of a subsequent DECLARE block.
This is obviously error-prone, and in fact the SQLSTATE/SQLERRM patch had
failed to do it for those two variables, leading to the bug recently exhibited
by Asif Ali Rehman: a DECLARE within an exception handler tried to reinitialize
SQLERRM.

Although the SQLSTATE/SQLERRM patch isn't in any pre-8.1 branches, and so
I can't point to a demonstrable failure there, it seems wise to back-patch
this into the older branches anyway, just to keep the logic similar to HEAD.
1 parent af8a424
History
Tip revision: 6eb61d578f3c9fc4fff78d69851b7558821668e3 authored by Tom Lane on 08 February 2007, 18:38:28 UTC
Rearrange use of plpgsql_add_initdatums() so that only the parsing of a
Tip revision: 6eb61d5
File Mode Size
config
contrib
doc
src
COPYRIGHT -rw-r--r-- 1.2 KB
GNUmakefile.in -rw-r--r-- 3.4 KB
Makefile -rw-r--r-- 1.4 KB
README -rw-r--r-- 1.3 KB
aclocal.m4 -rw-r--r-- 442 bytes
configure -rwxr-xr-x 459.1 KB
configure.in -rw-r--r-- 36.1 KB

README

back to top