diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2022-12-22 10:35:02 -0500 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2022-12-22 10:35:02 -0500 |
| commit | 97431d673992326f4f85e4f185941444f33e20db (patch) | |
| tree | 192950262a216613d907e72a12c655f7a0ead761 /src/backend/executor | |
| parent | f489b480f4aa8b4db4858a7bef1b42c984992c8a (diff) | |
| download | postgresql-97431d673992326f4f85e4f185941444f33e20db.tar.gz postgresql-97431d673992326f4f85e4f185941444f33e20db.zip | |
Add some recursion and looping defenses in prepjointree.c.
Andrey Lepikhov demonstrated a case where we spend an unreasonable
amount of time in pull_up_subqueries(). Not only is that recursing
with no explicit check for stack overrun, but the code seems not
interruptable by control-C. Let's stick a CHECK_FOR_INTERRUPTS
there, along with sprinkling some stack depth checks.
An actual fix for the excessive time consumption seems a bit
risky to back-patch; but this isn't, so let's do so.
Discussion: https://postgr.es/m/703c09a2-08f3-d2ec-b33d-dbecd62428b8@postgrespro.ru
Diffstat (limited to 'src/backend/executor')
0 files changed, 0 insertions, 0 deletions