Re: pg15.3: dereference null *plan in CachedPlanIsSimplyValid/plpgsql - Mailing list pgsql-hackers

From Tom Lane
Subject Re: pg15.3: dereference null *plan in CachedPlanIsSimplyValid/plpgsql
Date
Msg-id 584104.1689868532@sss.pgh.pa.us
Whole thread Raw
In response to pg15.3: dereference null *plan in CachedPlanIsSimplyValid/plpgsql  (Justin Pryzby <pryzby@telsasoft.com>)
List pgsql-hackers
Justin Pryzby <pryzby@telsasoft.com> writes:
> A production instance crashed like so.

Oh, interesting.  Apparently we got past the "Has cache invalidation fired
on this plan?" check:

    if (!plansource->is_valid || plan != plansource->gplan || !plan->is_valid)
        return false;

because "plan" and "plansource->gplan" were *both* null, which is
a case the code wasn't expecting.  plansource.c seems to be paranoid
about gplan possibly being null everywhere else but here :-(

> Note that the prior query seems to have timed out in the same / similar plpgsql
> statement:

Hm.  Perhaps that helps explain how we got into this state.  It must
be some weird race condition, given the lack of prior reports.
Anyway, adding a check for plan not being null seems like the
obvious fix.

            regards, tom lane



pgsql-hackers by date:

Previous
From: Amit Langote
Date:
Subject: Re: remaining sql/json patches
Next
From: Alvaro Herrera
Date:
Subject: Re: remaining sql/json patches