On Mon, 29 Jul 2024 at 04:31, Tom Lane <t...@sss.pgh.pa.us> wrote: > It is not clear to me exactly which of these pointers should be > presumed to be possibly-null, but certainly testing ident after > storing through it is pretty pointless. Maybe what was intended > was > > - if (ident && strcmp(*name, "dynahash") == 0) > + if (*name && strcmp(*name, "dynahash") == 0)
It should be *ident. I just missed adding the pointer dereference when moving that code to a function. Thanks for the report. I'll fix shortly. David
