]> granicus.if.org Git - postgresql/commit
Prevent memory leaks from accumulating across printtup() calls.
authorTom Lane <tgl@sss.pgh.pa.us>
Sun, 3 Nov 2013 16:33:13 +0000 (11:33 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Sun, 3 Nov 2013 16:33:13 +0000 (11:33 -0500)
commit85de126246cd44315965c08f4af3d7d75fb9c5f1
treeddc66a04a2cb90b7cc2db8dc8623050a4a773d39
parentf2d6bdf8480bf4f3ecd787a8b02e393860bf824b
Prevent memory leaks from accumulating across printtup() calls.

Historically, printtup() has assumed that it could prevent memory leakage
by pfree'ing the string result of each output function and manually
managing detoasting of toasted values.  This amounts to assuming that
datatype output functions never leak any memory internally; an assumption
we've already decided to be bogus elsewhere, for example in COPY OUT.
range_out in particular is known to leak multiple kilobytes per call, as
noted in bug #8573 from Godfried Vanluffelen.  While we could go in and fix
that leak, it wouldn't be very notationally convenient, and in any case
there have been and undoubtedly will again be other leaks in other output
functions.  So what seems like the best solution is to run the output
functions in a temporary memory context that can be reset after each row,
as we're doing in COPY OUT.  Some quick experimentation suggests this is
actually a tad faster than the retail pfree's anyway.

This patch fixes all the variants of printtup, except for debugtup()
which is used in standalone mode.  It doesn't seem worth worrying
about query-lifespan leaks in standalone mode, and fixing that case
would be a bit tedious since debugtup() doesn't currently have any
startup or shutdown functions.

While at it, remove manual detoast management from several other
output-function call sites that had copied it from printtup().  This
doesn't make a lot of difference right now, but in view of recent
discussions about supporting "non-flattened" Datums, we're going to
want that code gone eventually anyway.

Back-patch to 9.2 where range_out was introduced.  We might eventually
decide to back-patch this further, but in the absence of known major
leaks in older output functions, I'll refrain for now.
src/backend/access/common/printtup.c
src/backend/bootstrap/bootstrap.c
src/backend/executor/spi.c
src/backend/utils/adt/rowtypes.c