Check SELECT permissions in subqueries.

vadim4o · vadim4o · commit 1a6de0760d5e · 1998-02-27T08:43:53.000Z
diff --git a/src/backend/executor/execMain.c b/src/backend/executor/execMain.c
@@ -26,7 +26,7 @@
  *
  *
  * IDENTIFICATION
- *	  $Header: /cvsroot/pgsql/src/backend/executor/execMain.c,v 1.44 1998/02/26 04:31:09 momjian Exp $
+ *	  $Header: /cvsroot/pgsql/src/backend/executor/execMain.c,v 1.45 1998/02/27 08:43:52 vadim Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -54,12 +54,12 @@
 #include "catalog/heap.h"
 #include "commands/trigger.h"
 
+void
+ExecCheckPerms(CmdType operation, int resultRelation, List *rangeTable,
+			   Query *parseTree);
 
 
 /* decls for local routines only used within this module */
-static void
-ExecCheckPerms(CmdType operation, int resultRelation, List *rangeTable,
-			   Query *parseTree);
 static TupleDesc
 InitPlan(CmdType operation, Query *parseTree,
 		 Plan *plan, EState *estate);
@@ -273,14 +273,7 @@ ExecutorEnd(QueryDesc *queryDesc, EState *estate)
 	BufferRefCountRestore(estate->es_refcount);
 }
 
-/* ===============================================================
- * ===============================================================
-						 static routines follow
- * ===============================================================
- * ===============================================================
- */
-
-static void
+void
 ExecCheckPerms(CmdType operation,
 			   int resultRelation,
 			   List *rangeTable,
@@ -376,6 +369,13 @@ ExecCheckPerms(CmdType operation,
 	}
 }
 
+/* ===============================================================
+ * ===============================================================
+						 static routines follow
+ * ===============================================================
+ * ===============================================================
+ */
+
 
 /* ----------------------------------------------------------------
  *		InitPlan
diff --git a/src/backend/executor/nodeSubplan.c b/src/backend/executor/nodeSubplan.c
@@ -106,11 +106,14 @@ ExecSubPlan(SubPlan *node, List *pvar, ExprContext *econtext)
  *
  * ----------------------------------------------------------------
  */
+extern void ExecCheckPerms(CmdType op, int resRel, List *rtable, Query *q);
 bool
 ExecInitSubPlan(SubPlan *node, EState *estate, Plan *parent)
 {
 	EState	   *sp_estate = CreateExecutorState();
 
+	ExecCheckPerms (CMD_SELECT, 0, node->rtable, (Query*) NULL);
+	
 	sp_estate->es_range_table = node->rtable;
 	sp_estate->es_param_list_info = estate->es_param_list_info;
 	sp_estate->es_param_exec_vals = estate->es_param_exec_vals;

Original file line number	Diff line number	Diff line change
`@@ -106,11 +106,14 @@ ExecSubPlan(SubPlan node, List pvar, ExprContext *econtext)`
`106`	`106`	`*`
`107`	`107`	`* ----------------------------------------------------------------`
`108`	`108`	`*/`
	`109`	`+extern void ExecCheckPerms(CmdType op, int resRel, List rtable, Query q);`
`109`	`110`	`bool`
`110`	`111`	`ExecInitSubPlan(SubPlan node, EState estate, Plan *parent)`
`111`	`112`	`{`
`112`	`113`	`EState *sp_estate = CreateExecutorState();`
`113`	`114`
	`115`	`+ ExecCheckPerms (CMD_SELECT, 0, node->rtable, (Query*) NULL);`
	`116`	`+`
`114`	`117`	`sp_estate->es_range_table = node->rtable;`
`115`	`118`	`sp_estate->es_param_list_info = estate->es_param_list_info;`
`116`	`119`	`sp_estate->es_param_exec_vals = estate->es_param_exec_vals;`