Re: Fix search_path for all maintenance commands

From: Noah Misch <noah(at)leadboat(dot)com>
To: "David G(dot) Johnston" <david(dot)g(dot)johnston(at)gmail(dot)com>
Cc: Gurjeet Singh <gurjeet(at)singh(dot)im>, Jeff Davis <pgsql(at)j-davis(dot)com>, pgsql-hackers(at)postgresql(dot)org, Nathan Bossart <nathandbossart(at)gmail(dot)com>, Robert Haas <robertmhaas(at)gmail(dot)com>, Greg Stark <stark(at)mit(dot)edu>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Subject: Re: Fix search_path for all maintenance commands
Date: 2023-07-15 21:13:33
Message-ID: 20230715211333.GB3675150@rfd.leadboat.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Thu, Jul 13, 2023 at 02:07:27PM -0700, David G. Johnston wrote:
> On Thu, Jul 13, 2023 at 2:00 PM Gurjeet Singh <gurjeet(at)singh(dot)im> wrote:
> > On Thu, Jul 13, 2023 at 1:37 PM David G. Johnston <david(dot)g(dot)johnston(at)gmail(dot)com> wrote:
> > > I'm against simply breaking the past without any recourse as what we
> > did for pg_dump/pg_restore still bothers me.
> >
> > I'm sure this is tangential, but can you please provide some
> > context/links to the change you're referring to here.
>
> Here is the instigating issue and a discussion thread on the aftermath:
> https://wiki.postgresql.org/wiki/A_Guide_to_CVE-2018-1058%3A_Protect_Your_Search_Path
> https://www.postgresql.org/message-id/flat/13033.1531517020%40sss.pgh.pa.us#2aa2e25816d899d62f168926e3ff17b1

I don't blame you for feeling bothered about it. A benefit of having done it
is that we gained insight into the level of pain it caused. If it had been
sufficiently painful, someone would have quickly added an escape hatch. Five
years later, nobody has added one.

The 2018 security fixes instigated many function repairs that $SUBJECT would
otherwise instigate. That wasn't too painful. The net new pain of $SUBJECT
will be less, since the 2018 security fixes prepared the path. Hence, I
remain +1 for the latest Davis proposal.

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Andres Freund 2023-07-16 01:56:56 Improve heapgetpage() performance, overhead from serializable
Previous Message Nikita Malakhov 2023-07-15 20:57:30 Protect extension' internal tables - how?