Re: pgsql: Fix handling of restricted processes for Windows Vista (mainly), - Mailing list pgsql-committers

From Andrew Dunstan
Subject Re: pgsql: Fix handling of restricted processes for Windows Vista (mainly),
Date
Msg-id 47C87F69.4080500@dunslane.net
Whole thread Raw
In response to Re: pgsql: Fix handling of restricted processes for Windows Vista (mainly),  (Magnus Hagander <magnus@hagander.net>)
Responses Re: pgsql: Fix handling of restricted processes for Windows Vista (mainly),
List pgsql-committers
My testing on Vista shows we don't need them for Cygwin.

cheers

andrew

Magnus Hagander wrote:
> AFAIK, we will if we want to run as Administrator on Vista. I'm not
> sure what we're doing there is actually compatible with it though. And
> do we care? I'm perfectly fine saying that we don't support that part
> on cygwin, and just #ifdef the whole thing out there?
>
> //Magnus
>
> Andrew Dunstan wrote:
>>
>> You have broken the Cygwin build (see buildfarm). Will we need those
>> calls to AddUserToDacl() on Cygwin? If not, the simplest thing would
>> be to #ifdef them out, I guess.
>>
>> cheers
>>
>> andrew
>>
>> Magnus Hagander wrote:
>>> Log Message:
>>> -----------
>>> Fix handling of restricted processes for Windows Vista (mainly),
>>> by explicitly adding back the user to the DACL of the new process.
>>> This fixes the failure case when executing as the Administrator
>>> user, which had no permissions left at all after we dropped the
>>> Administrators group.
>>>
>>> Dave Page with some modifications from me
>>>
>>> Modified Files:
>>> --------------
>>>     pgsql/src/bin/initdb:
>>>         initdb.c (r1.153 -> r1.154)
>>>
>>> (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/bin/initdb/initdb.c?r1=1.153&r2=1.154)
>>>
>>>     pgsql/src/bin/pg_ctl:
>>>         pg_ctl.c (r1.94 -> r1.95)
>>>
>>> (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/bin/pg_ctl/pg_ctl.c?r1=1.94&r2=1.95)
>>>
>>>     pgsql/src/include:
>>>         port.h (r1.117 -> r1.118)
>>>
>>> (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/include/port.h?r1=1.117&r2=1.118)
>>>
>>>     pgsql/src/port:
>>>         exec.c (r1.57 -> r1.58)
>>>
>>> (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/port/exec.c?r1=1.57&r2=1.58)
>>>
>>>
>>> ---------------------------(end of
>>> broadcast)---------------------------
>>> TIP 3: Have you checked our extensive FAQ?
>>>
>>>                http://www.postgresql.org/docs/faq
>>>
>>>
>
>

pgsql-committers by date:

Previous
From: alvherre@postgresql.org (Alvaro Herrera)
Date:
Subject: pgsql: Remove long-unused and broken TCL_ARRAYS.
Next
From: adunstan@postgresql.org (Andrew Dunstan)
Date:
Subject: pgsql: Don't call AddUserToDacl on Cygwin