On Wed, Jan 5, 2022 at 7:05 PM Mark Dilger <mark.dil...@enterprisedb.com> wrote:
> > On Jan 4, 2022, at 12:47 PM, Joshua Brindle > > <joshua.brin...@crunchydata.com> wrote: > > > >> I was able to reproduce that using REASSIGN OWNED BY to cause a user to > >> own itself. Is that how you did it, or is there yet another way to get > >> into that state? > > > > I did: > > ALTER ROLE brindle OWNER TO brindle; > > Ok, thanks. I have rebased, fixed both REASSIGN OWNED BY and ALTER ROLE .. > OWNER TO cases, and added regression coverage for them. > > The last patch set to contain significant changes was v2, with v3 just being > a rebase. Relative to those sets: > > 0001 -- rebased. > 0002 -- rebased; extend AlterRoleOwner_internal to disallow making a role its > own immediate owner. > 0003 -- rebased; extend AlterRoleOwner_internal to disallow cycles in the > role ownership graph. > 0004 -- rebased. > 0005 -- new; removes the broken pg_auth_members.grantor field. > LGTM +1