[Freeipa-devel] Fix removal of ipa-kdc-proxy.conf symlink

Christian Heimes cheimes at redhat.com
Mon Jun 29 08:54:50 UTC 2015


Hello,

the attached patch fixes the first bug, that was reported by Fraser
today. installutils.remove_file() uses os.path.exists() to check if the
file still exists, which in turn uses stat(2). I have modified the
function to use os.path.lexists() instead. It doesn't follow symlinks.

Because httpinstance first removes the target file
/etc/ipa/kdcproxy/ipa-kdc-proxy.conf before it tries to remove the
symlink /etc/httpd/conf.d/ipa-kdc-proxy.conf, the
installutils.remove_file() ignores the dangling symlink.

Christian
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-cheimes-0002-Fix-removal-of-ipa-kdc-proxy.conf.patch
Type: text/x-patch
Size: 1881 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20150629/36d509ef/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 455 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20150629/36d509ef/attachment.sig>


More information about the Freeipa-devel mailing list