Re: pg_id.c windows fix - Mailing list pgsql-patches

From Andrew Dunstan
Subject Re: pg_id.c windows fix
Date
Msg-id 002301c38b29$f724cd10$6401a8c0@DUNSLANE
Whole thread Raw
In response to Re: pg_id.c windows fix  (Bruce Momjian <pgman@candle.pha.pa.us>)
List pgsql-patches
That's not the issue. It complained because it wanted a pointer to the size.

The relevant lines in winbase.h files are:

BOOL WINAPI GetUserNameA (LPSTR,PDWORD);
#define GetUserName GetUserNameA

I am using Mingw.

cheers

andrew

----- Original Message -----
From: "Bruce Momjian" <pgman@candle.pha.pa.us>
To: "Andrew Dunstan" <andrew@dunslane.net>
Cc: "PG Patches" <pgsql-patches@postgresql.org>
Sent: Sunday, October 05, 2003 12:17 AM
Subject: Re: [PATCHES] pg_id.c windows fix


>
> Can you tell me what Win32 compile environment can't handle a sizeof()
> as a function parameter?
>
> !       GetUserName(pw->pw_name, sizeof(pw->pw_name)-1);
>
> Does casting to DWORD help?
>
> --------------------------------------------------------------------------
-
>
> Andrew Dunstan wrote:
> > Here's a patch for pg_id.c that lets it compile cleanly and run on
Windows.
> >
> > cheers
> >
> > andrew
> >
> > Index: src/bin/pg_id/pg_id.c
> > ===================================================================
> > RCS file: /projects/cvsroot/pgsql-server/src/bin/pg_id/pg_id.c,v
> > retrieving revision 1.22.2.1
> > diff -c -w -r1.22.2.1 pg_id.c
> > *** src/bin/pg_id/pg_id.c       7 Sep 2003 04:37:04 -0000       1.22.2.1
> > --- src/bin/pg_id/pg_id.c       4 Oct 2003 13:33:30 -0000
> > ***************
> > *** 38,43 ****
> > --- 38,44 ----
> >                 char pw_name[128];
> >         } pass_win32;
> >         struct passwd_win32 *pw = &pass_win32;
> > +       DWORD pwname_size = sizeof(pass_win32.pw_name) - 1;
> >
> >         pw->pw_uid = 1;
> >   #endif
> > ***************
> > *** 113,119 ****
> >                 exit(1);
> >         }
> >
> > !       GetUserName(pw->pw_name, sizeof(pw->pw_name)-1);
> >   #endif
> >
> >         if (!limit_user_info)
> > --- 114,120 ----
> >                 exit(1);
> >         }
> >
> > !       GetUserName(pw->pw_name, &pwname_size);
> >   #endif
> >
> >         if (!limit_user_info)
>
> --
>   Bruce Momjian                        |  http://candle.pha.pa.us
>   pgman@candle.pha.pa.us               |  (610) 359-1001
>   +  If your life is a hard drive,     |  13 Roberts Road
>   +  Christ can be your backup.        |  Newtown Square, Pennsylvania
19073
>
> ---------------------------(end of broadcast)---------------------------
> TIP 4: Don't 'kill -9' the postmaster


pgsql-patches by date:

Previous
From: Stephan Szabo
Date:
Subject: Re: pg_id.c windows fix
Next
From: Peter Eisentraut
Date:
Subject: Re: fix log_min_duration_statement logic error