[SSSD] [PATCH] add utility call check_and_open_readonly

Dmitri Pal dpal at redhat.com
Mon Sep 28 17:50:44 UTC 2009


Stephen Gallagher wrote:
> On 09/28/2009 11:49 AM, Sumit Bose wrote:
> > Hi,
>
> > with the patch the config file is only read if it is
> > - a regular file
> > - owner and group are 0 (root)
> > - file permissions are 600
>
> > This patch depends on the config_from_fd patch currently under review.
>
> > bye,
> > Sumit
>
>
> > ------------------------------------------------------------------------
>
> > _______________________________________________
> > sssd-devel mailing list
> > sssd-devel at lists.fedorahosted.org
> > https://fedorahosted.org/mailman/listinfo/sssd-devel
>
> Nack.
>
> As discussed on IRC, the lstat is redundant. All of the necessary
> file-type checks can be performed with the fstat, with no risk of race
> condition.
>
I thought we had some plans to move part of this functionality into ini
interface.
Should we?
I think there was a ticket about this to make ini config interface deal
with special files and permissions.



_______________________________________________
sssd-devel mailing list
sssd-devel at lists.fedorahosted.org
https://fedorahosted.org/mailman/listinfo/sssd-devel

-- 
Thank you,
Dmitri Pal

Engineering Manager IPA project,
Red Hat Inc.


-------------------------------
Looking to carve out IT costs?
www.redhat.com/carveoutcosts/




More information about the sssd-devel mailing list