guix-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v3] daemon: Set ownership of kept build directories to the ca


From: Danny Milosavljevic
Subject: Re: [PATCH v3] daemon: Set ownership of kept build directories to the calling user.
Date: Tue, 6 Dec 2016 21:41:33 +0100

Hi Hartmut,

> +#if defined(SO_PEERCRED)
...
> +#else
> +                /* Setting these to -1 means: do not change */
> +                settings.clientUid = -1;
> +                settings.clientGid = -1;
> +#endif

I think you also have to cast them there, so

               settings.clientUid = (uid_t) -1;
               settings.clientGid = (gid_t) -1;

The reason is because (-1) is a signed integer and clientUid isn't - and 
neither is clientGid.



reply via email to

[Prev in Thread] Current Thread [Next in Thread]