URL: https://github.com/SSSD/sssd/pull/192 Title: #192: Add certificate mapping library
sumit-bose commented: """ Hi, Lukas, thank you for the review. I fixed the invalid read the two NULL RETURNS, the PW.SET_BUT_NOT_USED and the CLANG WARNING. It would be possible to save the result in a variable for the CHECKED_RETURN > # 1003|-> NSS_ShutdownContext(nss_ctx); but there is nothing which can be done with the variable so it would just produce a different warning, so I would suggest to ignore it. Same with the regexec warning, add an otherwise unused variable would just cause a different warning. """ See the full comment at https://github.com/SSSD/sssd/pull/192#issuecomment-286696735
_______________________________________________ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org
