You can not select more than 25 topics Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.

Tom Lane 3d13623d75 Prevent leakage of SPI tuple tables during subtransaction abort. ... plpgsql often just remembers SPI-result tuple tables in local variables, and has no mechanism for freeing them if an ereport(ERROR) causes an escape out of the execution function whose local variable it is. In the original coding, that wasn't a problem because the tuple table would be cleaned up when the function's SPI context went away during transaction abort. However, once plpgsql grew the ability to trap exceptions, repeated trapping of errors within a function could result in significant intra-function-call memory leakage, as illustrated in bug #8279 from Chad Wagner. We could fix this locally in plpgsql with a bunch of PG_TRY/PG_CATCH coding, but that would be tedious, probably slow, and prone to bugs of omission; moreover it would do nothing for similar risks elsewhere. What seems like a better plan is to make SPI itself responsible for freeing tuple tables at subtransaction abort. This patch attacks the problem that way, keeping a list of live tuple tables within each SPI function context. Currently, such freeing is automatic for tuple tables made within the failed subtransaction. We might later add a SPI call to mark a tuple table as not to be freed this way, allowing callers to opt out; but until someone exhibits a clear use-case for such behavior, it doesn't seem worth bothering. A very useful side-effect of this change is that SPI_freetuptable() can now defend itself against bad calls, such as duplicate free requests; this should make things more robust in many places. (In particular, this reduces the risks involved if a third-party extension contains now-redundant SPI_freetuptable() calls in error cleanup code.) Even though the leakage problem is of long standing, it seems imprudent to back-patch this into stable branches, since it does represent an API semantics change for SPI users. We'll patch this in 9.3, but live with the leakage in older branches.		13 years ago
..
backend	Prevent leakage of SPI tuple tables during subtransaction abort.	13 years ago
bin	WITH CHECK OPTION support for auto-updatable VIEWs	13 years ago
common	pgindent run for release 9.3	13 years ago
include	Prevent leakage of SPI tuple tables during subtransaction abort.	13 years ago
interfaces	Check for NULL result from strdup	13 years ago
makefiles	Install all a Makefile's extension controls, not just the first.	13 years ago
pl	Prevent leakage of SPI tuple tables during subtransaction abort.	13 years ago
port	Send SIGKILL to children if they don't die quickly in immediate shutdown	13 years ago
template	…
test	Improvements to GetErrorContextStack()	13 years ago
timezone	pgindent run for release 9.3	13 years ago
tools	Get rid of pg_class.reltoastidxid.	13 years ago
tutorial	Modernize string literal syntax in tutorial example.	13 years ago
.gitignore	…
DEVELOPERS	…
Makefile	Create libpgcommon, and move pg_malloc et al to it	13 years ago
Makefile.global.in	Improve support for building PGXS modules with VPATH.	13 years ago
Makefile.shlib	Add pkg-config files for libpq and ecpg libraries	13 years ago
bcc32.mak	…
nls-global.mk	…
win32.mak	…