NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit7ba6ee8

committed

Add missed bms_copy() in perform_pruning_combine_step

We were initializing a BMS to merely reference an existing one, whichwould cause a double-free (and a crash) when the recursive algorithmtried to intersect it with an empty one. Fix it by creating a copy atinitialization time.Reported-by: sqlsmith (by way of Andreas Seltenreich)Author: Amit LangoteDiscussion:https://postgr.es/m/87in923lyw.fsf@ansel.ydns.eu

1 parent2c19ea8 commit7ba6ee8Copy full SHA for 7ba6ee8

File tree

1 file changed

-1

lines changed

src/backend/partitioning
- partprune.c

1 file changed

-1

lines changed

`‎src/backend/partitioning/partprune.c`

Lines changed: 2 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -2923,7 +2923,8 @@ perform_pruning_combine_step(PartitionPruneContext *context,`
`2923`	`2923`	`if (firststep)`
`2924`	`2924`	`{`
`2925`	`2925`	`/* Copy step's result the first time. */`
`2926`		`-result->bound_offsets=step_result->bound_offsets;`
	`2926`	`+result->bound_offsets=`
	`2927`	`+bms_copy(step_result->bound_offsets);`
`2927`	`2928`	`result->scan_null=step_result->scan_null;`
`2928`	`2929`	`result->scan_default=step_result->scan_default;`
`2929`	`2930`	`firststep= false;`

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!

Commit7ba6ee8

File tree

1 file changed

1 file changed

`‎src/backend/partitioning/partprune.c`

0 commit comments