NotificationsYou must be signed in to change notification settings
Fork28
Star151

Commitf578093

committed

Try and silence spurious Coverity warning.

gset_data (aka gd) in planner.c is always non-null if and only ifparse->groupingSets is non-null, but Coverity doesn't know that andcomplains. Feed it an assertion to see if that keeps it happy.

1 parent9fa6e08 commitf578093Copy full SHA for f578093

File tree

1 file changed

-0

lines changed

src/backend/optimizer/plan
- planner.c

1 file changed

-0

lines changed

`‎src/backend/optimizer/plan/planner.c`

Lines changed: 2 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -3362,6 +3362,8 @@ get_number_of_groups(PlannerInfo *root,`
`3362`	`3362`	`ListCell*lc;`
`3363`	`3363`	`ListCell*lc2;`
`3364`	`3364`
	`3365`	`+Assert(gd);/* keep Coverity happy */`
	`3366`	`+`
`3365`	`3367`	`dNumGroups=0;`
`3366`	`3368`
`3367`	`3369`	`foreach(lc,gd->rollups)`

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!

Commitf578093

File tree

1 file changed

1 file changed

`‎src/backend/optimizer/plan/planner.c`

0 commit comments