From: | Magnus Hagander <magnus(at)hagander(dot)net> |
---|---|
To: | Dave Page <dpage(at)postgresql(dot)org> |
Cc: | pgsql-committers(at)postgresql(dot)org |
Subject: | Re: pgsql: Remove unsafe calling of WSAStartup and WSA Cleanup from DllMain. |
Date: | 2007-03-08 20:52:19 |
Message-ID: | 45F07783.9020300@hagander.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers pgsql-hackers |
>> No, it shouldn't.
>>
>> First, when on mingw, the file with WSAStartup() in it wasn't even
>> linked in. And this is the DLL that we've been distributing in the MSI.
>
> It wasn't? Ok... Anything else we've missed? :-p
Probably :-) Let me know when you find it.
>> Second, they really shouldn't rely on that anyway - I don't think we've
>> documented anywhere that libpq does this ;-) Because I assume you are
>> referring to applications that use *other* winsock functions, but don't
>> call WSAStartup() themselves?
>
> Yeah - I'm not saying it's right having now seen the arguments
> against, but it happens. For example, do you recall us being confused
> when we found we needed to call it in slon.exe?
Eh, no, actually not. Sorry.
>> Because libpq still calls wsastartup on
>> the first attempt to open a connection.
>
> Hmm, ok - did it always?
Seems it was added in 1.255 which was back in July 2003. For 7.4. So not
always, but it's not exactly new.
//Magnus
From | Date | Subject | |
---|---|---|---|
Next Message | Dave Page | 2007-03-09 08:16:12 | Re: pgsql: Remove unsafe calling of WSAStartup and WSA Cleanup from DllMain. |
Previous Message | Dave Page | 2007-03-08 20:50:00 | Re: pgsql: Remove unsafe calling of WSAStartup and WSA Cleanup from DllMain. |
From | Date | Subject | |
---|---|---|---|
Next Message | Jim C. Nasby | 2007-03-08 21:09:13 | Re: Estimating seq_page_fetch and random_page_fetch |
Previous Message | Dave Page | 2007-03-08 20:50:00 | Re: pgsql: Remove unsafe calling of WSAStartup and WSA Cleanup from DllMain. |