On Mon, 24 May 2021 at 02:14, Tom Lane <[email protected]> wrote: > Coverity is not happy with this:
> This complaint is triggered because elsewhere in the same function, > you're careful to check for inner_path->param_info being null before > trying to dereference it: > > if ((inner_path->param_info == NULL || > inner_path->param_info->ppi_clauses == NIL) && > innerrel->lateral_vars == NIL) > return NULL; > > Coverity reasons that it's probably a bug that you didn't do the same > here; and I think it's right. Yeah, I think that should be fixed. Thanks for reporting. I propose the attached. I think if the restrictlist is empty then we needn't bother trying Result Cache. David
resultcache_fixup.patch
Description: Binary data
