]> git.ipfire.org Git - thirdparty/postgresql.git/commit
Avoid Assert failure when processing empty statement in aborted xact.
authorTom Lane <tgl@sss.pgh.pa.us>
Wed, 21 Jun 2023 15:07:11 +0000 (11:07 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Wed, 21 Jun 2023 15:07:11 +0000 (11:07 -0400)
commita8be2356c55617eb959d38c8c249b454e1280a18
tree43738f378a37424c6ba13d02a8a1f4d4bee4022f
parent6973868c9dbc8d6242a7419c0807d301dd6d5fe3
Avoid Assert failure when processing empty statement in aborted xact.

exec_parse_message() wants to create a cached plan in all cases,
including for empty input.  The empty-input path does not have
a test for being in an aborted transaction, making it possible
that plancache.c will fail due to trying to do database lookups
even though there's no real work to do.

One solution would be to throw an aborted-transaction error in
this path too, but it's not entirely clear whether the lack of
such an error was intentional or whether some clients might be
relying on non-error behavior.  Instead, let's hack plancache.c
so that it treats empty statements with the same logic it
already had for transaction control commands, ensuring that it
can soldier through even in an already-aborted transaction.

Per bug #17983 from Alexander Lakhin.  Back-patch to all
supported branches.

Discussion: https://postgr.es/m/17983-da4569fcb878672e@postgresql.org
src/backend/utils/cache/plancache.c
src/test/regress/expected/psql.out
src/test/regress/sql/psql.sql