[SSSD] [PATCH] SELINUX: Set permissive umask for selinux operations

Jakub Hrozek jhrozek at redhat.com
Tue Jan 27 16:08:45 UTC 2015


On Tue, Jan 27, 2015 at 10:41:28AM -0500, Roland Mainz wrote:
> 
> 
> ----- Original Message -----
> > From: "Jakub Hrozek" <jhrozek at redhat.com>
> > To: sssd-devel at lists.fedorahosted.org
> > Sent: Tuesday, January 27, 2015 11:31:58 AM
> > Subject: Re: [SSSD] [PATCH] SELINUX: Set permissive umask for selinux operations
> > 
> > On Tue, Jan 27, 2015 at 10:51:24AM +0100, Jakub Hrozek wrote:
> > > Alternatively, we could only set and reset the umask in the caller. That
> > > way, we would know that a short-lived selinux_child is changing umask.
> > > 
> > > That is safer, but risks that we forget next time..so at least we need
> > > to add a comment to the declaration in header.
> > 
> > Attached is an alternate patch that only touches the umask in the
> > short-lived process.
> 
> Which syscall exactly (strace dump, please) causes the original issue ?

mkdir(2)

> Maybe there is a way to do this without having to rely on |umask()| (using the |*at()| APIs or a later modification of the permissions) ...

We have no control over what libsemanage does, unfortunately.



More information about the sssd-devel mailing list