pgsql: Fix incorrect logic in plpgsql for cleanup after evaluation of - Mailing list pgsql-committers

From tgl@postgresql.org (Tom Lane)
Subject pgsql: Fix incorrect logic in plpgsql for cleanup after evaluation of
Date
Msg-id 20100809185011.B20077541D7@cvs.postgresql.org
Whole thread Raw
List pgsql-committers
Log Message:
-----------
Fix incorrect logic in plpgsql for cleanup after evaluation of non-simple
expressions.  We need to deal with this when handling subscripts in an array
assignment, and also when catching an exception.  In an Assert-enabled build
these omissions led to Assert failures, but I think in a normal build the
only consequence would be short-term memory leakage; which may explain why
this wasn't reported from the field long ago.

Back-patch to all supported versions.  7.4 doesn't have exceptions, but
otherwise these bugs go all the way back.

Heikki Linnakangas and Tom Lane

Modified Files:
--------------
    pgsql/src/pl/plpgsql/src:
        pl_exec.c (r1.262 -> r1.263)
        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/pl/plpgsql/src/pl_exec.c?r1=1.262&r2=1.263)
    pgsql/src/test/regress/expected:
        plpgsql.out (r1.87 -> r1.88)
        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/test/regress/expected/plpgsql.out?r1=1.87&r2=1.88)
    pgsql/src/test/regress/sql:
        plpgsql.sql (r1.73 -> r1.74)
        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/test/regress/sql/plpgsql.sql?r1=1.73&r2=1.74)

pgsql-committers by date:

Previous
From: Robert Haas
Date:
Subject: Re: pgsql: Provide stable target anchors for libpq functions.
Next
From: tgl@postgresql.org (Tom Lane)
Date:
Subject: pgsql: Fix incorrect logic in plpgsql for cleanup after evaluation of