Fix for InitPlan-s: have to copy subselect' result tuple.

diff --git a/src/backend/executor/nodeSubplan.c b/src/backend/executor/nodeSubplan.c
index 610f0a09643f2c499bb7d84e7c2cdee54e599525..3c5560d13bd20690a526432e1101864beff055f9 100644 (file)
--- a/src/backend/executor/nodeSubplan.c
+++ b/src/backend/executor/nodeSubplan.c
@@ -95,6 +95,9 @@ ExecSubPlan(SubPlan *node, List *pvar, ExprContext *econtext)
                        break;
        }
        
+       if ( !found && sublink->subLinkType == ALL_SUBLINK )
+               return ((Datum) true);
+       
        return ((Datum) result);
 }
 
@@ -193,6 +196,15 @@ ExecSetParamPlan (SubPlan *node)
                        break;
                }
                
+               /* 
+                * If this is uncorrelated subquery then its plan will be closed
+                * (see below) and this tuple will be free-ed - bad for not byval
+                * types... But is free-ing possible in the next ExecProcNode in
+                * this loop ? Who knows... Someday we'll keep track of saved 
+                * tuples...
+                */
+               tup = heap_copytuple (tup);
+               
                foreach (lst, node->setParam)
                {
                        ParamExecData   *prm = &(plan->state->es_param_exec_vals[lfirsti(lst)]);