Re: Add \pset options for boolean value display

From: "David G(dot) Johnston" <david(dot)g(dot)johnston(at)gmail(dot)com>
To: Álvaro Herrera <alvherre(at)kurilemu(dot)de>
Cc: "pgsql-hackers(at)lists(dot)postgresql(dot)org" <pgsql-hackers(at)lists(dot)postgresql(dot)org>, "a(dot)kozhemyakin" <a(dot)kozhemyakin(at)postgrespro(dot)ru>
Subject: Re: Add \pset options for boolean value display
Date: 2026-04-21 17:43:02
Message-ID: CAKFQuwaYEq5sQsWZZZn-u3Fx4UQL2TFMVbG8HnxrL-=KFM2AjA@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Mon, Apr 20, 2026 at 11:30 PM David G. Johnston <
david(dot)g(dot)johnston(at)gmail(dot)com> wrote:

> On Monday, April 20, 2026, a.kozhemyakin <a(dot)kozhemyakin(at)postgrespro(dot)ru>
> wrote:
>>
>> The following script, when built with addresssanitizer, fails with a
>> DoubleFree error. reproduce on master (d3bba0415435)
>>
>
> Thanks for the report. To be honest, I’m not sure where to start
> debugging it - but I plan to give it a go. I’m curious if it fails for
> “\pset null ‘N’” “\g (null=csv)” too and I just got bit by inheriting its
> layout. Don’t see why these new ones would be different.
>
>
Had issues getting a server to run meson+asan but did get psql to do so and
confirmed.

I just didn't find all of the patterns.

savePsetInfo and restorePsetInfo need explicit knowledge of these options
as well to clean up the popt struct.

Attached.

David J.

Attachment Content-Type Size
save-restore-pset-info-fix.diff text/x-patch 814 bytes

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Alexander Lakhin 2026-04-21 18:00:00 Re: Incorrect checksum in control file with pg_rewind test
Previous Message Yura Sokolov 2026-04-21 17:06:38 Re: PoC: Add condition variable support to WaitEventSetWait()