URL: https://github.com/SSSD/sssd/pull/5361 Title: #5361: negcache: multiple fixes
sumit-bose commented: """ > https://github.com/sumit-bose/sssd/blob/f095a01b804d97a46149c9f6593e7e99a63b7195/src/responder/common/negcache.c#L1009 > -- I think code should `continue` in case of any error, including `EAGAIN` > (there are 3 similar spots in this function) Hi, this missing `continue` for the users is in fact correct and in the following code the name is treated a generic UPN. I changed the debug message to make this more clear. For groups there are no UPNs should here we should continue on all errors. I fixed the code accordingly and comments. bye, Sumit """ See the full comment at https://github.com/SSSD/sssd/pull/5361#issuecomment-724641235
_______________________________________________ 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://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org
