Re: pgsql: Remove unsafe calling of WSAStartup and WSA Cleanup from DllMain.

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

Responses

Browse pgsql-committers by date

  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.

Browse pgsql-hackers by date

  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