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)]);