summaryrefslogtreecommitdiff
path: root/src/bin/psql/mainloop.h
diff options
context:
space:
mode:
authorTom Lane <tgl@sss.pgh.pa.us>2009-01-01 17:12:16 +0000
committerTom Lane <tgl@sss.pgh.pa.us>2009-01-01 17:12:16 +0000
commit87081b2d35057c4919a8a05a7c52fe87845874d5 (patch)
treef231a0cdc069ac6a84a804983bd91bd290cbf11d /src/bin/psql/mainloop.h
parentdf0ea5a1cdc9b5a8ed172124818d42a9d33bc971 (diff)
downloadpostgresql-87081b2d35057c4919a8a05a7c52fe87845874d5.tar.gz
Fix an oversight in my patch of a couple weeks ago that ensured a snapshot
is available during datatype input in Bind message processing. I put the PopActiveSnapshot() or equivalent just before PortalDefineQuery, which is an unsafe spot for it (in 8.3 and later) because we are carrying a plancache refcount that hasn't yet been assigned to the portal. Any error thrown there would result in leaking the refcount. It's not exactly likely that PopActiveSnapshot would throw an elog, perhaps, but it could happen. Reorder the code and add another comment warning not to do that.
Diffstat (limited to 'src/bin/psql/mainloop.h')
0 files changed, 0 insertions, 0 deletions