diff options
author | Tom Lane <tgl@sss.pgh.pa.us> | 2023-03-13 12:40:28 -0400 |
---|---|---|
committer | Tom Lane <tgl@sss.pgh.pa.us> | 2023-03-13 12:40:28 -0400 |
commit | bc0bcce2e74bf777b458e3fd815bb125dbda2ce6 (patch) | |
tree | b024665589d9535f81b30c1dc904ea1d87d8d714 /src/backend/executor/nodeWindowAgg.c | |
parent | 96bef4374e450b2bd387cbfaad9371db3ac0669a (diff) | |
download | postgresql-bc0bcce2e74bf777b458e3fd815bb125dbda2ce6.tar.gz postgresql-bc0bcce2e74bf777b458e3fd815bb125dbda2ce6.zip |
Fix failure to detect some cases of improperly-nested aggregates.
check_agg_arguments_walker() supposed that it needn't descend into
the arguments of a lower-level aggregate function, but this is
just wrong in the presence of multiple levels of sub-select. The
oversight would lead to executor failures on queries that should
be rejected. (Prior to v11, they actually were rejected, thanks
to a "redundant" execution-time check.)
Per bug #17835 from Anban Company. Back-patch to all supported
branches.
Discussion: https://postgr.es/m/17835-4f29f3098b2d0ba4@postgresql.org
Diffstat (limited to 'src/backend/executor/nodeWindowAgg.c')
0 files changed, 0 insertions, 0 deletions