Commit5a1d1b9

committed

Avoid memcpy() with a NULL source pointer and count == 0

When memcpy() is called on a pointer, the compiler is entitled to assumethat the pointer is not null, which can lead to optimizing nearby codein potentially undesirable ways. We still want such optimizations(gcc's -fdelete-null-pointer-checks) in cases where they're valid.Related: commit13bba02.Backpatch to pg11, where this particular instance appeared.Reported-by: Ranier Vilela <ranier.vf@gmail.com>Reported-by: Zhihong Yu <zyu@yugabyte.com>Discussion:https://postgr.es/m/CAEudQApUndmQkr5fLrCKXQ7+ib44i7S+Kk93pyVThS85PnG3bQ@mail.gmail.comDiscussion:https://postgr.es/m/CALNJ-vSdhwSM5f4tnNn1cdLHvXMVe_S+V3nR5GwNrmCPNB2VtQ@mail.gmail.com

1 parentfd3a75d commit5a1d1b9Copy full SHA for 5a1d1b9

File tree

-2

lines changed

-2

lines changed

Lines changed: 4 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -1140,15 +1140,17 @@ DefineIndex(Oid relationId,`
`1140`	`1140`
`1141`	`1141`	`if (partitioned)`
`1142`	`1142`	`{`
	`1143`	`+PartitionDescpartdesc;`
	`1144`	`+`
`1143`	`1145`	`/*`
`1144`	`1146`	`* Unless caller specified to skip this step (via ONLY), process each`
`1145`	`1147`	`* partition to make sure they all contain a corresponding index.`
`1146`	`1148`	`*`
`1147`	`1149`	`* If we're called internally (no stmt->relation), recurse always.`
`1148`	`1150`	`*/`
`1149`		`-if (!stmt->relation\|\|stmt->relation->inh)`
	`1151`	`+partdesc=RelationGetPartitionDesc(rel);`
	`1152`	`+if ((!stmt->relation\|\|stmt->relation->inh)&&partdesc->nparts>0)`
`1150`	`1153`	`{`
`1151`		`-PartitionDescpartdesc=RelationGetPartitionDesc(rel);`
`1152`	`1154`	`intnparts=partdesc->nparts;`
`1153`	`1155`	`Oidpart_oids=palloc(sizeof(Oid)nparts);`
`1154`	`1156`	`boolinvalidate_parent= false;`

Comments

(0)