From: Heikki Linnakangas <heikki.linnakangas@iki.fi>
Date: Fri, 12 Nov 2010 13:19:14 +0000 (+0200)
Subject: Add missing support for removing foreign data wrapper / server privileges
X-Git-Tag: REL9_1_ALPHA3~190
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=e356743f3ed45c36dcc4d0dbf6c1e8751b3d70b5;p=postgresql

Add missing support for removing foreign data wrapper / server privileges
belonging to a user at DROP OWNED BY. Foreign data wrappers and servers
don't do anything useful yet, which is why no-one has noticed, but since we
have them, seems prudent to fix this. Per report from Chetan Suttraway.
Backpatch to 9.0, 8.4 has the same problem but this patch didn't apply
there so I'm not going to bother.
---

diff --git a/src/backend/catalog/aclchk.c b/src/backend/catalog/aclchk.c
index fb080e9298..fefa335e09 100644
--- a/src/backend/catalog/aclchk.c
+++ b/src/backend/catalog/aclchk.c
@@ -1340,6 +1340,12 @@ RemoveRoleFromObjectACL(Oid roleid, Oid classid, Oid objid)
 			case TableSpaceRelationId:
 				istmt.objtype = ACL_OBJECT_TABLESPACE;
 				break;
+			case ForeignServerRelationId:
+				istmt.objtype = ACL_OBJECT_FOREIGN_SERVER;
+				break;
+			case ForeignDataWrapperRelationId:
+				istmt.objtype = ACL_OBJECT_FDW;
+				break;
 			default:
 				elog(ERROR, "unexpected object class %u", classid);
 				break;