public inbox for [email protected]  

help / color / mirror / Atom feed

From: Amit Langote <[email protected]>
To: Euler Taveira <[email protected]>
Cc: [email protected]
Cc: [email protected]
Subject: Re: BUG #19470: PostgreSQL backend aborts (assert failure) when a prepared statement returns a composite type cast t
Date: Wed, 10 Jun 2026 18:40:52 +0900
Message-ID: <CA+HiwqEoNtf=UAe2vHeCY3cHOgUfrzWJp2PC474KKfd4aGz-uA@mail.gmail.com> (raw)
In-Reply-To: <[email protected]>
References: <[email protected]>
	<[email protected]>

Hi,

On Tue, May 5, 2026 at 11:37 AM Euler Taveira <[email protected]> wrote:
> On Wed, Apr 29, 2026, at 8:27 AM, PG Bug reporting form wrote:
> >
> > Reproduction steps (minimal):
> >   BEGIN;
> >   CREATE TYPE foo AS (a int, b text);
> >   PREPARE p AS SELECT CAST(ROW(1, 'hello') AS foo)::text;
> >   EXECUTE p;
> >   ALTER TYPE foo ALTER ATTRIBUTE a TYPE VARCHAR(100);
> >   EXECUTE p;
> >   COMMIT;
> >
>
> Thanks for your report! The attached patch fixes this case. I included a
> similar test case but I'm fine if the test is not included with this fix. I
> decided to create a separate function instead of adding the new conditions to
> record_plan_type_dependency() because it keeps the fix simple and isolated.
>
> The crash is reproduced back to v11 which means this fix should be backpatched
> to all supported versions.

Thanks for working on this.

I think this is on the right track. Using relationOids is the right
mechanism, since the DDL doesn't touch the composite's pg_type row and
a TYPEOID inval item wouldn't work here. Doing it in fix_expr_common
covers extract_query_dependencies too.

What I'm less easy about is whether Const and RowExpr are the only
carriers. As written, the TYPTYPE_COMPOSITE test won't look through an
array over a composite or a domain over one, and nodes like
FieldSelect and ConvertRowtypeExpr also carry composite type OIDs.
Could you check whether any of those reproduce a similar crash? If so
we'd want to look through to the element/base type and/or cover more
node types before this goes in.

Modulo that, I'll prepare to push to all supported branches.

-- 
Thanks, Amit Langote

view thread (3+ messages)

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: BUG #19470: PostgreSQL backend aborts (assert failure) when a prepared statement returns a composite type cast t
  In-Reply-To: <CA+HiwqEoNtf=UAe2vHeCY3cHOgUfrzWJp2PC474KKfd4aGz-uA@mail.gmail.com>

* 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