Bruce Momjian <pgman@candle.pha.pa.us> writes:
> Tom Lane wrote:
>> unlink (someone *please* tell me that that unlink in getaddrinfo_unix
>> is a brain-dead error...)
> Not sure --- that's why I added it.
After further research I conclude it is at best unnecessary and at worst
dangerous. It is unnecessary because Lock_AF_UNIX() will alreay have
unlink'ed any pre-existing socket file in the postmaster-startup case.
It is dangerous because there is no reason that anyone would expect
getaddrinfo() to have destructive side-effects. Certainly the
discussion of the AI_PASSIVE flag in the Linux manpage for getaddrinfo
doesn't suggest any such thing.
I'm going to remove it.
regards, tom lane
