]> granicus.if.org Git - postgresql/commit
Fix volatile-safety issue in pltcl_SPI_execute_plan().
authorTom Lane <tgl@sss.pgh.pa.us>
Mon, 26 Jan 2015 17:18:25 +0000 (12:18 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Mon, 26 Jan 2015 17:18:45 +0000 (12:18 -0500)
commit3dd084c7fe4a2596ddb695d52e2d65910ac9206e
treef84bc7910a53d927cb6443d9dccee992bdaf47ac
parent5c393a0ac52c46f342f4a11acf8b0d4647cb9da3
Fix volatile-safety issue in pltcl_SPI_execute_plan().

The "callargs" variable is modified within PG_TRY and then referenced
within PG_CATCH, which is exactly the coding pattern we've now found
to be unsafe.  Marking "callargs" volatile would be problematic because
it is passed by reference to some Tcl functions, so fix the problem
by not modifying it within PG_TRY.  We can just postpone the free()
till we exit the PG_TRY construct, as is already done elsewhere in this
same file.

Also, fix failure to free(callargs) when exiting on too-many-arguments
error.  This is only a minor memory leak, but a leak nonetheless.

In passing, remove some unnecessary "volatile" markings in the same
function.  Those doubtless are there because gcc 2.95.3 whinged about
them, but we now know that its algorithm for complaining is many bricks
shy of a load.

This is certainly a live bug with compilers that optimize similarly
to current gcc, so back-patch to all active branches.
src/pl/tcl/pltcl.c