URL: https://github.com/SSSD/sssd/pull/5827 Title: #5827: A number of fixes around `strto*()` usage
alexey-tikhonov commented: """ > > src/providers/ldap/ldap_id_services.c > > 266: port = strtouint16(state->name, NULL, 10); > > Again, internal stuff. But ok, to be consistent I will add checks here... > > > src/providers/ldap/sdap_access.c > > 1859: duration = strtouint32(pwdAccountLockedDurationTime, NULL, > > 0); > > This value is from LDAP - "pwdLockoutDuration" . I just wasn't sure what > weird format it might have... But it looks > https://ldapwiki.com/wiki/PwdLockoutDuration says "Integer" so ok, I will > update this as well. Done. """ See the full comment at https://github.com/SSSD/sssd/pull/5827#issuecomment-946133323
_______________________________________________ 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 Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure