From 69bef7ff79eb33e8a355e79e7fbb5338d1557775 Mon Sep 17 00:00:00 2001 From: Tom Lane Date: Tue, 15 Feb 2011 15:50:17 -0500 Subject: [PATCH] Add CheckTableNotInUse calls in DROP TABLE and DROP INDEX. Recent releases had a check on rel->rd_refcnt in heap_drop_with_catalog, but failed to cover the possibility of pending trigger events at DROP time. (Before 8.4 we didn't even check the refcnt.) When the trigger events were eventually fired, you'd get "could not open relation with OID nnn" errors, as in recent report from strk. Better to throw a suitable error when the DROP is attempted. Also add a similar check in DROP INDEX. Back-patch to all supported branches. --- src/backend/catalog/heap.c | 7 +++++++ src/backend/catalog/index.c | 6 ++++++ 2 files changed, 13 insertions(+) diff --git a/src/backend/catalog/heap.c b/src/backend/catalog/heap.c index 72a97b8b7a2..dd63c7e1296 100644 --- a/src/backend/catalog/heap.c +++ b/src/backend/catalog/heap.c @@ -1236,6 +1236,13 @@ heap_drop_with_catalog(Oid relid) */ rel = relation_open(relid, AccessExclusiveLock); + /* + * There can no longer be anyone *else* touching the relation, but we + * might still have open queries or cursors, or pending trigger events, + * in our own session. + */ + CheckTableNotInUse(rel, "DROP TABLE"); + /* * Schedule unlinking of the relation's physical file at commit. */ diff --git a/src/backend/catalog/index.c b/src/backend/catalog/index.c index e57809427af..dae3af322b9 100644 --- a/src/backend/catalog/index.c +++ b/src/backend/catalog/index.c @@ -848,6 +848,12 @@ index_drop(Oid indexId) userIndexRelation = index_open(indexId, AccessExclusiveLock); + /* + * There can no longer be anyone *else* touching the index, but we + * might still have open queries using it in our own session. + */ + CheckTableNotInUse(userIndexRelation, "DROP INDEX"); + /* * Schedule physical removal of the file */