RE: [HACKERS] Win32 pЭort of libpq - Mailing list pgsql-hackers

From Vince Vielhaber
Subject RE: [HACKERS] Win32 pЭort of libpq
Date
Msg-id Pine.BSF.4.05.9909301204580.13045-100000@paprika.michvhf.com
Whole thread Raw
In response to RE: [HACKERS] Win32 pЭort of libpq  (Dmitry Samersoff <dms@wplus.net>)
List pgsql-hackers
On Thu, 30 Sep 1999, Dmitry Samersoff wrote:

>
> On 30-Sep-99 Vince Vielhaber wrote:
> > On Thu, 30 Sep 1999, Dmitry Samersoff wrote:
> >
> >>
> >> On 30-Sep-99 Edmund Mergl wrote:
> >> > Hi,
> >> >
> >> > in the last few days I compiled libpq on Windows NT
> >> > using MS Visual Studio 6.0. I followed the instructions
> >> > given by Bob Kline <bkline@rksystems.com> in his mail from
> >> > Fri, 3 Sep 1999.
> >> > Unfortuanetely he sent his mail only to dbi-users, so I would
> >> > like to repeat one major problem on this list.
> >> >
> >> > Here is an excerpt from his mail:
> >> >
> >> > 4. The DllMain function in src/interfaces/libpq/libpqdll.c of the
> >> > PostgreSQL 6.5 sources, in which WSAStartup is invoked, is never called,
> >> > which causes gethostbyname calls to fail.  Solution (more properly,
> >> > "kludge" -- I know there's a cleaner fix somewhere, but this works for
> >> > now): immediately after the local declarations for the connectDB function
> >> > in src/interfaces/libpq/fe-connect.c:
> >> >
> >> >#ifdef WIN32
> >> >     static int WeHaveCalledWSAStartup;
> >> >
> >> >     if (!WeHaveCalledWSAStartup) {
> >> >         WSADATA wsaData;
> >> >         if (WSAStartup(MAKEWORD(1, 1), &wsaData)) {
> >> >             sprintf(conn->errorMessage,
> >> >                     "WSAStartup failed: errno=%d\n", h_errno);
> >> >             goto connect_errReturn;
> >> >         }
> >> >         WeHaveCalledWSAStartup = 1;
> >> >     }
> >> >#endif
> >>
> >> You need not to take care wether WSAStartup is alredy called or not.
> >> Windows handle it automatically.
> >
> > By calling it yourself you have more control over which minimum version
> > will be loaded.
>
> Yes, but you can just call
>
>    WSADATA wsaData;
>    if (WSAStartup(MAKEWORD(1, 1), &wsaData)) {
>             sprintf(conn->errorMessage,
>                      "WSAStartup failed: errno=%d\n", h_errno);
>             goto connect_errReturn;
>    }
>
>
> without WeHaveCalledWSAStartup at all.

Accroding to the 1.1 spec, you must call WSACleanup() for EVERY WSAStartup
call made.  So if you call WSAStartup() three times, you must call
WSACleanup() three times - the first two only decrement the internal
counter, the last one does the cleanup.  This may have changed in versions
of Winsock after 1.1 and my spec is a bit old (20 Jan 1993).

Vince.
--
==========================================================================
Vince Vielhaber -- KA8CSH   email: vev@michvhf.com   flame-mail: /dev/null
  # include <std/disclaimers.h>       Have you seen http://www.pop4.net?
        Online Campground Directory    http://www.camping-usa.com
       Online Giftshop Superstore    http://www.cloudninegifts.com
==========================================================================





pgsql-hackers by date:

Previous
From: Lamar Owen
Date:
Subject: Re: [HACKERS] RI status report #2
Next
From: Bruce Momjian
Date:
Subject: Re: [HACKERS] postmaster dead on startup from unportable SSL patch