MIME-Version: 1.0
References: <0b32e30f2fb94ae3b7f4ee15bbb072c0@rutoken.ru>
 <107eb23a-8ebb-42bc-99c0-ca551733e94e@proxel.se>
 <CAOYmi+muy=Wtk0dv6J5HmcFTMvrPMe2MGN2X+hjZL7DKSUEXLQ@mail.gmail.com>
 <leyiwkml7kyrdxwjzhcmmi6rtyxxwofokfxhlkpbagfprmbqw2@jxdckvomcmug>
 <CAOYmi+mneT-m52GhWdA=XqH5SE6f5jJbAmjwL+wp+xk4j9VVpg@mail.gmail.com>
 <mazv3o3ua5pbvy5xwrz6n6zu6gyxkqtohbdohu3o73xyhtwa26@xcbb7ro2o42k>
 <CAOYmi+k__DbMjOxfySjVjCQB=-7Pd+qHb-1Vn9kuN1XOgbOsGg@mail.gmail.com>
 <b389c5e5-e4af-45c2-bd51-59dcd358e1e4@eisentraut.org>
In-Reply-To: <b389c5e5-e4af-45c2-bd51-59dcd358e1e4@eisentraut.org>
From: Jacob Champion <jacob.champion@enterprisedb.com>
Date: Thu, 9 Apr 2026 08:59:35 -0700
Message-ID: 
 <CAOYmi+m+g1dn8aSuBuKsNK+HCGm9HeD-U747rpSuDQ08vWJM6g@mail.gmail.com>
Subject: Re: DEREF_AFTER_NULL: src/common/jsonapi.c:2529
To: Peter Eisentraut <peter@eisentraut.org>
Cc: Andres Freund <andres@anarazel.de>, Andreas Karlsson <andreas@proxel.se>,
	=?UTF-8?B?0JPQsNC70LrQuNC9INCh0LXRgNCz0LXQuQ==?= <galkin@rutoken.ru>,
	"pgsql-hackers@lists.postgresql.org" <pgsql-hackers@lists.postgresql.org>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Archived-At: 
 <https://www.postgresql.org/message-id/CAOYmi%2Bm%2Bg1dn8aSuBuKsNK%2BHCGm9HeD-U747rpSuDQ08vWJM6g%40mail.gmail.com>
Precedence: bulk

On Wed, Apr 8, 2026 at 2:25=E2=80=AFPM Peter Eisentraut <peter@eisentraut.o=
rg> wrote:
> This was recently (as in: I still remember it) discussed:
>
> https://www.postgresql.org/message-id/flat/cf26e970-8e92-59f1-247a-aa2652=
35075b%40enterprisedb.com
>
> Probably not worth opening up again.

Fair enough.

> But it seems to me that the pfree() in fe_memutils.c should
> Assert(pointer !=3D NULL), to be consistent with the backend version.