pgsql: Fix a couple of contrib/dblink bugs. - Mailing list pgsql-committers

From Tom Lane
Subject pgsql: Fix a couple of contrib/dblink bugs.
Date
Msg-id E1SFEKd-0002en-7h@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Fix a couple of contrib/dblink bugs.

dblink_exec leaked temporary database connections if any error occurred
after connection setup, for example
    SELECT dblink_exec('...connect string...', 'select 1/0');
Add a PG_TRY block to ensure PQfinish gets done when it is needed.
(dblink_record_internal is on the hairy edge of needing similar treatment,
but seems not to be actively broken at the moment.)

Also, in 9.0 and up, only one of the three functions using tuplestore
return mode was properly checking that the query context would allow
a tuplestore result.

Noted while reviewing dblink patch.  Back-patch to all supported branches.

Branch
------
REL8_3_STABLE

Details
-------
http://git.postgresql.org/pg/commitdiff/11efdb06eef786abedffbbdcdb5c7fbd38416860

Modified Files
--------------
contrib/dblink/dblink.c |   24 ++++++++++++++++++------
1 files changed, 18 insertions(+), 6 deletions(-)


pgsql-committers by date:

Previous
From: Tom Lane
Date:
Subject: pgsql: Fix a couple of contrib/dblink bugs.
Next
From: Tom Lane
Date:
Subject: pgsql: Fix a couple of contrib/dblink bugs.