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

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

In response to

Responses

Browse pgsql-committers by date

  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.

Browse pgsql-hackers by date

  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.