diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2023-07-20 14:23:46 -0400 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2023-07-20 14:23:46 -0400 |
| commit | 10fd061bbc66351bb9cdfc20b65d0c928620e21c (patch) | |
| tree | 5790df8804144e58874bd9b7b7ab76797b6bc306 /src | |
| parent | 8286b432c32dbec8b3109c5b09a902adb2385c52 (diff) | |
| download | postgresql-10fd061bbc66351bb9cdfc20b65d0c928620e21c.tar.gz postgresql-10fd061bbc66351bb9cdfc20b65d0c928620e21c.zip | |
Guard against null plan pointer in CachedPlanIsSimplyValid().
If both the passed-in plan pointer and plansource->gplan are
NULL, CachedPlanIsSimplyValid would think that the plan pointer
is possibly-valid and try to dereference it. For the one extant
call site in plpgsql, this situation doesn't normally happen
which is why we've not noticed. However, it appears to be possible
if the previous use of the cached plan failed, as per report from
Justin Pryzby. Add an extra check to prevent crashing.
Back-patch to v13 where this code was added.
Discussion: https://postgr.es/m/ZLlV+STFz1l/WhAQ@telsasoft.com
Diffstat (limited to 'src')
| -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 e1a873967f0..1355c3a6f1d 100644 --- a/src/backend/utils/cache/plancache.c +++ b/src/backend/utils/cache/plancache.c @@ -1440,7 +1440,9 @@ CachedPlanIsSimplyValid(CachedPlanSource *plansource, CachedPlan *plan, * that here we *do* check plansource->is_valid, so as to force plan * rebuild if that's become false. */ - if (!plansource->is_valid || plan != plansource->gplan || !plan->is_valid) + if (!plansource->is_valid || + plan == NULL || plan != plansource->gplan || + !plan->is_valid) return false; Assert(plan->magic == CACHEDPLAN_MAGIC); |