Re: BUG #17346: pg_upgrade fails with role granted by other role

Tom Lane <tgl@sss.pgh.pa.us>

From: Tom Lane <tgl@sss.pgh.pa.us>
To: andrewbille@gmail.com
Cc: pgsql-bugs@lists.postgresql.org, Daniel Gustafsson <daniel@yesql.se>
Date: 2021-12-27T16:02:27Z
Lists: pgsql-bugs
PG Bug reporting form <noreply@postgresql.org> writes:
> After the commit:

> commit 371087d006e04991080bf17cf2287db38d3ea92e
> Author: Daniel Gustafsson <dgustafsson@postgresql.org>
> Date:   Fri Nov 26 14:02:01 2021 +0100
>     Fix GRANTED BY support in REVOKE ROLE statements
    
> pg_upgrade for example from 10.19 version causes the error:

Yeah, you don't even need pg_upgrade.  Just do

regression=# CREATE ROLE user1; CREATE ROLE user2; GRANT user1 TO user2 GRANTED BY user1;
CREATE ROLE
CREATE ROLE
ERROR:  grantor must be current user

A superuser, or really anyone who's a member of the user1 role,
ought to be able to do that (especially since it used to be allowed).
So it seems the permissions check was coded incorrectly.

			regards, tom lane



Commits

  1. Revert b2a459edf "Fix GRANTED BY support in REVOKE ROLE statements"

  2. Fix GRANTED BY support in REVOKE ROLE statements

  3. Allow GRANTED BY clause in normal GRANT and REVOKE statements