[SSSD] Only create the SELinux login file if there are mappings on the server

Jakub Hrozek jhrozek at redhat.com
Thu Aug 16 11:33:54 UTC 2012


On Thu, Aug 16, 2012 at 11:34:46AM +0200, Pavel Březina wrote:
> On 08/15/2012 06:27 PM, Jakub Hrozek wrote:
> >On Tue, Aug 14, 2012 at 11:11:01AM +0200, Pavel Březina wrote:
> >>Nack.
> >>
> >>User cannot login when the loginfile does not exist because you
> >>return !EOK here:
> >>
> >>     if (!file_content) {
> >>         err = remove_selinux_login_file(pd->user);
> >>         /* Don't overwrite original error condition if there was one */
> >>         if (ret == EOK) ret = err;
> >>     }
> >>     talloc_free(tmp_ctx);
> >>     return ret;
> >
> >Thank you. I think I should just ignore ENOENT in
> >remove_selinux_login_file()
> >
> >New patches are attached.
> 
> Works fine. Ack.
> 

Pushed to master after resolving a trivial conflict (a three-way merge
was all it took)



More information about the sssd-devel mailing list