pg.ddx.io  pgsql-committers@postgresql.org mailing list archive  
help / color / mirror / Atom feed
From: David Rowley <drowley@postgresql.org>
To: pgsql-committers@lists.postgresql.org
Subject: pgsql: Fix reset of incorrect hash iterator in GROUPING SETS queries
Date: Sat, 18 Oct 2025 03:07:57 +0000
Message-ID: <E1v9xIT-002Eis-0U@gemulon.postgresql.org> (raw)

Fix reset of incorrect hash iterator in GROUPING SETS queries

This fixes an unlikely issue when fetching GROUPING SET results from
their internally stored hash tables.  It was possible in rare cases that
the hash iterator would be set up incorrectly which could result in a
crash.

This was introduced in 4d143509c, so backpatch to v18.

Many thanks to Yuri Zamyatin for reporting and helping to debug this
issue.

Bug: #19078
Reported-by: Yuri Zamyatin <yuri@yrz.am>
Author: David Rowley <dgrowleyml@gmail.com>
Reviewed-by: Jeff Davis <pgsql@j-davis.com>
Discussion: https://postgr.es/m/19078-dfd62f840a2c0766@postgresql.org
Backpatch-through: 18

Branch
------
REL_18_STABLE

Details
-------
https://git.postgresql.org/pg/commitdiff/0b6a02f0355c8950ffe22b52eac86f6261e19caf

Modified Files
--------------
src/backend/executor/nodeAgg.c | 2 +-
src/include/lib/simplehash.h   | 4 ++++
2 files changed, 5 insertions(+), 1 deletion(-)



view thread (2+ messages)

Message-ID: <E1v9xIT-002Eis-0U@gemulon.postgresql.org>
Permalink:  ../E1v9xIT-002Eis-0U@gemulon.postgresql.org/
Also on:    postgresql.org/message-id/E1v9xIT-002Eis-0U@gemulon.postgresql.org

 · 

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: pgsql-committers@postgresql.org
  Cc: drowley@postgresql.org, pgsql-committers@lists.postgresql.org
  Subject: Re: pgsql: Fix reset of incorrect hash iterator in GROUPING SETS queries
  In-Reply-To: <E1v9xIT-002Eis-0U@gemulon.postgresql.org>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox