[SSSD] [PATCH] add utility call check_and_open_readonly

Stephen Gallagher sgallagh at redhat.com
Mon Sep 28 17:52:32 UTC 2009


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 09/28/2009 12:24 PM, 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.
> 

Per conversation on IRC, this patch is approved. I didn't realize at
first that we want to exclude symlinks as well from the SSSD.
_______________________________________________
sssd-devel mailing list
sssd-devel at lists.fedorahosted.org
https://fedorahosted.org/mailman/listinfo/sssd-devel

- -- 
Stephen Gallagher
RHCE 804006346421761

Looking to carve out IT costs?
www.redhat.com/carveoutcosts/
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (GNU/Linux)
Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org/

iEYEARECAAYFAkrA99YACgkQeiVVYja6o6M2vACgqtH9nD0ueiuQedrhH1ykP2i6
h0kAn2y6ZyzgRRMOuBq25ArY3vPIaaL2
=tN7m
-----END PGP SIGNATURE-----



More information about the sssd-devel mailing list