Fix bogus sizeof calculations.

tglsfdc · tglsfdc · commit d8122578098d · 2019-09-15T11:51:57.000-04:00
Noted by Coverity. Typo in 27cc7cd, so back-patch to v12 as that was.
diff --git a/src/backend/executor/execMain.c b/src/backend/executor/execMain.c
@@ -2912,7 +2912,7 @@ EvalPlanQualStart(EPQState *epqstate, Plan *planTree)
 	{
 		Assert(epqstate->relsubs_done != NULL);
 		memset(epqstate->relsubs_rowmark, 0,
-			   sizeof(rtsize * sizeof(ExecAuxRowMark *)));
+			   rtsize * sizeof(ExecAuxRowMark *));
 		memset(epqstate->relsubs_done, 0,
 			   rtsize * sizeof(bool));
 	}
@@ -2966,7 +2966,7 @@ EvalPlanQualEnd(EPQState *epqstate)
 	if (epqstate->tuple_table != NIL)
 	{
 		memset(epqstate->relsubs_slot, 0,
-			   sizeof(rtsize * sizeof(TupleTableSlot *)));
+			   rtsize * sizeof(TupleTableSlot *));
 		ExecResetTupleTable(epqstate->tuple_table, true);
 		epqstate->tuple_table = NIL;
 	}

Original file line number	Diff line number	Diff line change
`@@ -2912,7 +2912,7 @@ EvalPlanQualStart(EPQState epqstate, Plan planTree)`
`2912`	`2912`	`{`
`2913`	`2913`	`Assert(epqstate->relsubs_done != NULL);`
`2914`	`2914`	`memset(epqstate->relsubs_rowmark, 0,`
`2915`		`- sizeof(rtsize * sizeof(ExecAuxRowMark *)));`
	`2915`	`+ rtsize * sizeof(ExecAuxRowMark *));`
`2916`	`2916`	`memset(epqstate->relsubs_done, 0,`
`2917`	`2917`	`rtsize * sizeof(bool));`
`2918`	`2918`	`}`
`@@ -2966,7 +2966,7 @@ EvalPlanQualEnd(EPQState *epqstate)`
`2966`	`2966`	`if (epqstate->tuple_table != NIL)`
`2967`	`2967`	`{`
`2968`	`2968`	`memset(epqstate->relsubs_slot, 0,`
`2969`		`- sizeof(rtsize * sizeof(TupleTableSlot *)));`
	`2969`	`+ rtsize * sizeof(TupleTableSlot *));`
`2970`	`2970`	`ExecResetTupleTable(epqstate->tuple_table, true);`
`2971`	`2971`	`epqstate->tuple_table = NIL;`
`2972`	`2972`	`}`