]> granicus.if.org Git - postgresql/commit
Fix snapshot leak if lo_open called on non-existent object.
authorHeikki Linnakangas <heikki.linnakangas@iki.fi>
Mon, 30 Sep 2013 08:29:09 +0000 (11:29 +0300)
committerHeikki Linnakangas <heikki.linnakangas@iki.fi>
Mon, 30 Sep 2013 09:55:57 +0000 (12:55 +0300)
commitcd6c03b5c15e2b8bf306d97ed9befaf83ace5f93
tree4c74b6e5f37de7f9851cd9e6dbeda04d560ae896
parent03b6a6b48e932c4123bee4f6b2a71944da5bd05c
Fix snapshot leak if lo_open called on non-existent object.

lo_open registers the currently active snapshot, and checks if the
large object exists after that. Normally, snapshots registered by lo_open
are unregistered at end of transaction when the lo descriptor is closed, but
if we error out before the lo descriptor is added to the list of open
descriptors, it is leaked. Fix by moving the snapshot registration to after
checking if the large object exists.

Reported by Pavel Stehule. Backpatch to 8.4. The snapshot registration
system was introduced in 8.4, so prior versions are not affected (and not
supported, anyway).
src/backend/storage/large_object/inv_api.c