[SSSD] [PATCH] add utility call check_and_open_readonly

Stephen Gallagher sgallagh at redhat.com
Mon Sep 28 16:24:31 UTC 2009


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

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.

- -- 
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/

iEYEARECAAYFAkrA4zkACgkQeiVVYja6o6NOaQCgnSwWeD49iq9Cu93S2i9vrJ7z
4ucAoJTL/wEoCBCYQN+96NNno1UFn4aY
=asr7
-----END PGP SIGNATURE-----



More information about the sssd-devel mailing list