[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.
Re: [PATCH v3] daemon: Set ownership of kept build directories to the calling user.,
Danny Milosavljevic <=