diff options
| author | Joe Conway <mail@joeconway.com> | 2017-01-02 14:12:04 -0800 |
|---|---|---|
| committer | Joe Conway <mail@joeconway.com> | 2017-01-02 14:12:04 -0800 |
| commit | 35d4dd82c2e82d2bc0e51174b5cfb5ac30061a81 (patch) | |
| tree | d20c2201dccc173620a66966aef4bb9e2782d02e | |
| parent | 4191b9ece43d28cbf8b1257d1a97896184fc1fdf (diff) | |
| download | postgresql-35d4dd82c2e82d2bc0e51174b5cfb5ac30061a81.tar.gz postgresql-35d4dd82c2e82d2bc0e51174b5cfb5ac30061a81.zip | |
Silence compiler warnings
In GetCachedPlan(), initialize 'plan' to silence a compiler warning, but
also add an Assert() to make sure we don't ever actually fall through
with 'plan' still being set to NULL, since we are about to dereference
it.
Back-patch back to 9.2.
Author: Stephen Frost
Discussion: https://postgr.es/m/20161129152102.GR13284%40tamriel.snowman.net
| -rw-r--r-- | src/backend/utils/cache/plancache.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/src/backend/utils/cache/plancache.c b/src/backend/utils/cache/plancache.c index 5eefa74a61d..ca9ac23937c 100644 --- a/src/backend/utils/cache/plancache.c +++ b/src/backend/utils/cache/plancache.c @@ -1125,7 +1125,7 @@ CachedPlan * GetCachedPlan(CachedPlanSource *plansource, ParamListInfo boundParams, bool useResOwner) { - CachedPlan *plan; + CachedPlan *plan = NULL; List *qlist; bool customplan; @@ -1207,6 +1207,8 @@ GetCachedPlan(CachedPlanSource *plansource, ParamListInfo boundParams, } } + Assert(plan != NULL); + /* Flag the plan as in use by caller */ if (useResOwner) ResourceOwnerEnlargePlanCacheRefs(CurrentResourceOwner); |