pgsql: Fix an oversight in the support for storing/retrieving "minimal

From: tgl(at)postgresql(dot)org (Tom Lane)
To: pgsql-committers(at)postgresql(dot)org
Subject: pgsql: Fix an oversight in the support for storing/retrieving "minimal
Date: 2009-03-30 04:09:18
Message-ID: 20090330040918.5FDF1754ADE@cvs.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Log Message:
-----------
Fix an oversight in the support for storing/retrieving "minimal tuples" in
TupleTableSlots. We have functions for retrieving a minimal tuple from a slot
after storing a regular tuple in it, or vice versa; but these were implemented
by converting the internal storage from one format to the other. The problem
with that is it invalidates any pass-by-reference Datums that were already
fetched from the slot, since they'll be pointing into the just-freed version
of the tuple. The known problem cases involve fetching both a whole-row
variable and a pass-by-reference value from a slot that is fed from a
tuplestore or tuplesort object. The added regression tests illustrate some
simple cases, but there may be other failure scenarios traceable to the same
bug. Note that the added tests probably only fail on unpatched code if it's
built with --enable-cassert; otherwise the bug leads to fetching from freed
memory, which will not have been overwritten without additional conditions.

Fix by allowing a slot to contain both formats simultaneously; which turns out
not to complicate the logic much at all, if anything it seems less contorted
than before.

Back-patch to 8.2, where minimal tuples were introduced.

Tags:
----
REL8_2_STABLE

Modified Files:
--------------
pgsql/src/backend/access/common:
heaptuple.c (r1.112 -> r1.112.2.1)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/backend/access/common/heaptuple.c?r1=1.112&r2=1.112.2.1)
pgsql/src/backend/executor:
execTuples.c (r1.98 -> r1.98.2.1)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/backend/executor/execTuples.c?r1=1.98&r2=1.98.2.1)
pgsql/src/include/executor:
tuptable.h (r1.36 -> r1.36.2.1)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/include/executor/tuptable.h?r1=1.36&r2=1.36.2.1)
pgsql/src/test/regress/expected:
rangefuncs.out (r1.15 -> r1.15.2.1)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/test/regress/expected/rangefuncs.out?r1=1.15&r2=1.15.2.1)
pgsql/src/test/regress/sql:
rangefuncs.sql (r1.7 -> r1.7.2.1)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/test/regress/sql/rangefuncs.sql?r1=1.7&r2=1.7.2.1)

Browse pgsql-committers by date

  From Date Subject
Next Message User Itagaki 2009-03-30 04:36:39 textsearch-ja - textsearch_senna: New Directory
Previous Message Tom Lane 2009-03-30 04:09:09 pgsql: Fix an oversight in the support for storing/retrieving "minimal