Commit 9d878d7
committed
Guard against core dump from uninitialized subplan.
If the planner erroneously puts a non-parallel-safe SubPlan into
a parallelized portion of the query tree, nodeSubplan.c will fail
in the worker processes because it finds a null in es_subplanstates,
which it's unable to cope with. It seems worth a test-and-elog to
make that an error case rather than a core dump case.
This probably should have been included in commit 16ebab6, which
was responsible for allowing nulls to appear in es_subplanstates
to begin with. So, back-patch to v10 where that came in.
Discussion: https://postgr.es/m/924226.1604422326@sss.pgh.pa.us1 parent df4405b commit 9d878d7Copy full SHA for 9d878d7
File tree
Expand file treeCollapse file tree
1 file changed
+9
-1
lines changedOpen diff view settings
Filter options
- src/backend/executor
Expand file treeCollapse file tree
1 file changed
+9
-1
lines changedOpen diff view settings
Collapse file
src/backend/executor/nodeSubplan.c
Copy file name to clipboardExpand all lines: src/backend/executor/nodeSubplan.c+9-1Lines changed: 9 additions & 1 deletion
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
710 | 710 | |
711 | 711 | |
712 | 712 | |
713 | | - |
| 713 | + |
| 714 | + |
| 715 | + |
| 716 | + |
| 717 | + |
| 718 | + |
| 719 | + |
| 720 | + |
| 721 | + |
714 | 722 | |
715 | 723 | |
716 | 724 | |
|
0 commit comments