From: | "Dave Page" <dpage(at)postgresql(dot)org> |
---|---|
To: | "Magnus Hagander" <magnus(at)hagander(dot)net> |
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:50:00 |
Message-ID: | 200703082050000000@263091613 |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers pgsql-hackers |
> ------- Original Message -------
> From: Magnus Hagander <magnus(at)hagander(dot)net>
> To: Dave Page <dpage(at)postgresql(dot)org>
> Sent: 08/03/07, 20:37:33
> Subject: Re: [COMMITTERS] pgsql: Remove unsafe calling of WSAStartup and WSACleanup from DllMain.
>
> 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
> 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?
> Because libpq still calls wsastartup on
> the first attempt to open a connection.
Hmm, ok - did it always?
/D
From | Date | Subject | |
---|---|---|---|
Next Message | Magnus Hagander | 2007-03-08 20:52:19 | Re: pgsql: Remove unsafe calling of WSAStartup and WSA Cleanup from DllMain. |
Previous Message | Magnus Hagander | 2007-03-08 20:43:57 | Re: pgsql: Remove unsafe calling of WSAStartup and WSA Cleanup from DllMain. |
From | Date | Subject | |
---|---|---|---|
Next Message | Magnus Hagander | 2007-03-08 20:52:19 | Re: pgsql: Remove unsafe calling of WSAStartup and WSA Cleanup from DllMain. |
Previous Message | Christian Bird | 2007-03-08 20:11:43 | Re: who gets paid for this |