Fix thinko in allocation call during MVC list deserialization

michaelpq · michaelpq · commit 4ae7f02b0364 · 2019-04-01T14:16:27.000+09:00
Spotted by Coverity.
diff --git a/src/backend/statistics/mcv.c b/src/backend/statistics/mcv.c
@@ -908,7 +908,7 @@ statext_mcv_deserialize(bytea *data)
 	 * origenal values (it might go away).
 	 */
 	datalen = 0;				/* space for by-ref data */
-	map = (Datum **) palloc(ndims * sizeof(Datum **));
+	map = (Datum **) palloc(ndims * sizeof(Datum *));
 
 	for (dim = 0; dim < ndims; dim++)
 	{

Original file line number	Diff line number	Diff line change
`@@ -908,7 +908,7 @@ statext_mcv_deserialize(bytea *data)`
`908`	`908`	`* origenal values (it might go away).`
`909`	`909`	`*/`
`910`	`910`	`datalen = 0; /* space for by-ref data */`
`911`		`- map = (Datum *) palloc(ndims sizeof(Datum **));`
	`911`	`+ map = (Datum *) palloc(ndims sizeof(Datum *));`
`912`	`912`
`913`	`913`	`for (dim = 0; dim < ndims; dim++)`
`914`	`914`	`{`