public inbox for [email protected]
help / color / mirror / Atom feedFrom: Michael Paquier <[email protected]>
To: Ayush Tiwari <[email protected]>
Cc: Postgres hackers <[email protected]>
Cc: Alvaro Herrera <[email protected]>
Subject: Re: NULL pointer dereference in syslogger with load_libraries() and -DEXEC_BACKEND at startup
Date: Tue, 26 May 2026 08:06:10 +0900
Message-ID: <[email protected]> (raw)
In-Reply-To: <CAJTYsWVuyVK4OTQok2t1=Wz2F5bG+rS6hKuGqK8aBaw-2T7LgA@mail.gmail.com>
References: <[email protected]>
<[email protected]>
<CAJTYsWVuyVK4OTQok2t1=Wz2F5bG+rS6hKuGqK8aBaw-2T7LgA@mail.gmail.com>
On Mon, May 25, 2026 at 01:54:28PM +0530, Ayush Tiwari wrote:
> So I think the bad assumption is that MyBackendType == B_LOGGER means
> write_syslogger_file() is ready to use. A fix probably needs to make the
> direct syslogger-file path conditional on the FILE handles being ready, and
> that should cover stderr, CSV and JSON paths.
I am not sure. We are attempting to write to the syslogger file but
we should not do so yet. This code path should not be taken and
properly redirected to stderr, we should not use a shortcut based on
the FILE existing or not.
I have forgotten to add Alvaro in CC yesterday, done now.
--
Michael
Attachments:
[application/pgp-signature] signature.asc (833B, 2-signature.asc)
download
reply
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: [email protected]
Cc: [email protected], [email protected], [email protected], [email protected]
Subject: Re: NULL pointer dereference in syslogger with load_libraries() and -DEXEC_BACKEND at startup
In-Reply-To: <[email protected]>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox