URL: https://github.com/SSSD/sssd/pull/827
Title: #827: ldap/sdap_idmap.c: removed unnecessary include

mzidek-rh commented:
"""
Just to give more context I got from Alexey over IRC. He did this as part of 
his FIPS work. It is better to avoid including security related header files or 
files that may look like they enable some security functions to files where 
they are not needed because we may need to explain why they are there.

So the patch makes sense IMO. I will run some checks and ack if everything is 
OK.
"""

See the full comment at 
https://github.com/SSSD/sssd/pull/827#issuecomment-501813518
_______________________________________________
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org

Reply via email to