Re: BUG #9923: "reassign owned" does not change permissions grantor

Alvaro Herrera <alvherre@2ndquadrant.com>

From: Alvaro Herrera <alvherre@2ndquadrant.com>
To: Bruce Momjian <bruce@momjian.us>
Cc: Tom Lane <tgl@sss.pgh.pa.us>, Alexey Bashtanov <bashtanov@imap.cc>, pgsql-bugs@postgresql.org
Date: 2015-12-21T22:54:50Z
Lists: pgsql-bugs
Bruce Momjian wrote:
> On Thu, Dec 17, 2015 at 03:21:46PM -0500, Tom Lane wrote:
> > Bruce Momjian <bruce@momjian.us> writes:
> > > Backpatching seems fine to me.  I was just concerned if anyone was
> > > relying on the existing buggy behavior.
> > 
> > Seems unlikely.  Even if they are, the potential for catalog corruption
> > later seems to trump that argument.
> > 
> > > We do list this item as a 9.5
> > > incompatibility, so the question is whether we can add an
> > > incompatibility to back branches:
> > 
> > >       Fix <link linkend="SQL-REASSIGN-OWNED"><command>REASSIGN OWNED</></>
> > >       and <link linkend="SQL-ALTERTYPE"><command>ALTER OWNER TO</></>
> > >       to properly update permissions lists (ACLs) when changing ownership of
> > >       types, foreign data wrappers, and foreign servers (Bruce Momjian)
> > 
> > Actually, I guess we should take out that 9.5 release note item altogether
> > if we back-patch this.
> 
> Yes, that was one of my points.

Backpatched all the way back to 9.1.  Are you, Bruce, on the hook for
removing the 9.5 release note entry, or should I do that?

-- 
Álvaro Herrera                http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services


Commits

  1. adjust ACL owners for REASSIGN and ALTER OWNER TO