"Magnus Hagander" <mha@sollentuna.net> writes:
>> Actually ... why do we need a WSAStartup in makeEmptyPGconn?
>> If we have one in DLL attach, isn't that sufficient?
> Not if you link libpq as a static lib...
> I *think* that's why it was added. In the beginning it was only in the
> DLL attach code, and if you were using the library as a static lib you
> had to call it yourself. Not sure when that changed, but clearly that is
> when the WSACleanup() call should've gone in.
Ah. Makes sense. So adding it to freePGconn is correct.
Bruce, you should probably back-patch for as far as the WSAStartup call
is there, which looks to be 7.4.
regards, tom lane