Commite2d29d2

committed

Check for children array size sanity in fill_prel_with_partitions.

... which might be violated if has partition was dropped.

1 parente402512 commite2d29d2Copy full SHA for e2d29d2

File tree

+12

-0

lines changed

+12

-0

lines changed

Lines changed: 12 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -755,6 +755,18 @@ fill_prel_with_partitions(PartRelationInfo *prel,`
`755`	`755`	`switch (prel->parttype)`
`756`	`756`	`{`
`757`	`757`	`casePT_HASH:`
	`758`	`+/*`
	`759`	`+ * This might be the case if hash part was dropped, and thus`
	`760`	`+ * children array alloc'ed smaller than needed, but parts`
	`761`	`+ * bound cache still keeps entries with high indexes.`
	`762`	`+ */`
	`763`	`+if (pbin->part_idx >=PrelChildrenCount(prel))`
	`764`	`+ereport(ERROR, (errmsg("pg_pathman's cache for relation \"%s\" "`
	`765`	`+"has not been properly initialized. "`
	`766`	`+"Looks like one of hash partitions was dropped.",`
	`767`	`+get_rel_name_or_relid(PrelParentRelid(prel))),`
	`768`	`+errhint(INIT_ERROR_HINT)));`
	`769`	`+`
`758`	`770`	`prel->children[pbin->part_idx]=pbin->child_relid;`
`759`	`771`	`break;`
`760`	`772`

Comments

(0)