NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commite56dd7c

committed

Fix uninitialized-variable bug.

map_partition_varattnos() failed to set its found_whole_row outputparameter if the given expression list was NIL. This seems to bea pre-existing bug that chanced to be exposed by commit6f6b99d.It might be unreachable in v10, but I have little faith in thatproposition, so back-patch.Per buildfarm.

1 parentf25000c commite56dd7cCopy full SHA for e56dd7c

File tree

1 file changed

+15

-13

lines changed

src/backend/catalog
- partition.c

1 file changed

+15

-13

lines changed

`‎src/backend/catalog/partition.c`

Lines changed: 15 additions & 13 deletions

Original file line number	Diff line number	Diff line change
`@@ -1120,21 +1120,23 @@ map_partition_varattnos(List *expr, int target_varno,`
`1120`	`1120`	`Relationpartrel,Relationparent,`
`1121`	`1121`	`bool*found_whole_row)`
`1122`	`1122`	`{`
`1123`		`-AttrNumber*part_attnos;`
`1124`		`-boolmy_found_whole_row;`
	`1123`	`+boolmy_found_whole_row= false;`
`1125`	`1124`
`1126`		`-if (expr==NIL)`
`1127`		`-returnNIL;`
	`1125`	`+if (expr!=NIL)`
	`1126`	`+{`
	`1127`	`+AttrNumber*part_attnos;`
	`1128`	`+`
	`1129`	`+part_attnos=convert_tuples_by_name_map(RelationGetDescr(partrel),`
	`1130`	`+RelationGetDescr(parent),`
	`1131`	`+gettext_noop("could not convert row type"));`
	`1132`	`+expr= (List)map_variable_attnos((Node)expr,`
	`1133`	`+target_varno,0,`
	`1134`	`+part_attnos,`
	`1135`	`+RelationGetDescr(parent)->natts,`
	`1136`	`+RelationGetForm(partrel)->reltype,`
	`1137`	`+&my_found_whole_row);`
	`1138`	`+}`
`1128`	`1139`
`1129`		`-part_attnos=convert_tuples_by_name_map(RelationGetDescr(partrel),`
`1130`		`-RelationGetDescr(parent),`
`1131`		`-gettext_noop("could not convert row type"));`
`1132`		`-expr= (List)map_variable_attnos((Node)expr,`
`1133`		`-target_varno,0,`
`1134`		`-part_attnos,`
`1135`		`-RelationGetDescr(parent)->natts,`
`1136`		`-RelationGetForm(partrel)->reltype,`
`1137`		`-&my_found_whole_row);`
`1138`	`1140`	`if (found_whole_row)`
`1139`	`1141`	`*found_whole_row=my_found_whole_row;`
`1140`	`1142`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commite56dd7c

File tree

1 file changed

1 file changed

`‎src/backend/catalog/partition.c`

0 commit comments