| From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
|---|---|
| To: | Dagfinn Ilmari Mannsåker <ilmari(at)ilmari(dot)org> |
| Cc: | Nathan Bossart <nathandbossart(at)gmail(dot)com>, Kyotaro Horiguchi <horikyota(dot)ntt(at)gmail(dot)com>, michael(at)paquier(dot)xyz, gurjeet(at)singh(dot)im, pgsql-hackers(at)postgresql(dot)org |
| Subject: | Re: Unprivileged user can induce crash by using an SUSET param in PGOPTIONS |
| Date: | 2022-07-27 23:01:00 |
| Message-ID: | 3159520.1658962860@sss.pgh.pa.us |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
I wrote:
> =?utf-8?Q?Dagfinn_Ilmari_Manns=C3=A5ker?= <ilmari(at)ilmari(dot)org> writes:
>> Thanks! Just one minor nitpick: setting an %ENV entry to `undef`
>> doesn't unset the environment variable, it sets it to the empty string.
>> To unset a variable it needs to be deleted from %ENV, i.e. `delete
>> $ENV{PGUSER};`.
> Ah. Still, libpq doesn't distinguish, so the test works anyway.
> Not sure if it's worth changing.
Meh ... I had to un-break the test for Windows, so did this while
at it, using the local-in-block method. Thanks for the suggestion.
regards, tom lane
| From | Date | Subject | |
|---|---|---|---|
| Next Message | David G. Johnston | 2022-07-27 23:43:11 | Re: Proposal: add a debug message about using geqo |
| Previous Message | Jacob Champion | 2022-07-27 22:35:44 | Re: [Patch] ALTER SYSTEM READ ONLY |