On 11/1/21, 10:34 AM, "Tom Lane" <t...@sss.pgh.pa.us> wrote: > Thanks for looking! Here's an expanded patch that also takes care > of the other two easy-to-fix cases, nodeAgg.c and llvmjit.c. > AFAICS, llvm_release_context is like StandbyReleaseLockList > in that we don't need to worry about whether the data structure > is valid after an error partway through. (Maybe we should be > worrying, but I think the callers would need work as well if > that's to be the standard.)
LGTM. Nathan
