NotificationsYou must be signed in to change notification settings
Fork28
Star151

Commit7911e78

committed

Silence compiler warnings

In GetCachedPlan(), initialize 'plan' to silence a compiler warning, butalso add an Assert() to make sure we don't ever actually fall throughwith 'plan' still being set to NULL, since we are about to dereferenceit.Back-patch back to 9.2.Author: Stephen FrostDiscussion:https://postgr.es/m/20161129152102.GR13284%40tamriel.snowman.net

1 parentf7e7d6f commit7911e78Copy full SHA for 7911e78

File tree

1 file changed

-1

lines changed

src/backend/utils/cache
- plancache.c

1 file changed

-1

lines changed

`‎src/backend/utils/cache/plancache.c`

Lines changed: 3 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -1141,7 +1141,7 @@ CachedPlan *`
`1141`	`1141`	`GetCachedPlan(CachedPlanSource*plansource,ParamListInfoboundParams,`
`1142`	`1142`	`booluseResOwner)`
`1143`	`1143`	`{`
`1144`		`-CachedPlan*plan;`
	`1144`	`+CachedPlan*plan=NULL;`
`1145`	`1145`	`List*qlist;`
`1146`	`1146`	`boolcustomplan;`
`1147`	`1147`
`@@ -1223,6 +1223,8 @@ GetCachedPlan(CachedPlanSource *plansource, ParamListInfo boundParams,`
`1223`	`1223`	`}`
`1224`	`1224`	`}`
`1225`	`1225`
	`1226`	`+Assert(plan!=NULL);`
	`1227`	`+`
`1226`	`1228`	`/* Flag the plan as in use by caller */`
`1227`	`1229`	`if (useResOwner)`
`1228`	`1230`	`ResourceOwnerEnlargePlanCacheRefs(CurrentResourceOwner);`

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!

Commit7911e78

File tree

1 file changed

1 file changed

`‎src/backend/utils/cache/plancache.c`

0 commit comments