URL: https://github.com/SSSD/sssd/pull/31
Title: #31: nss: allow UPNs in SSS_NSS_GETSIDBYNAME and SSS_NSS_GETORIGBYNAME

jhrozek commented:
"""
On Mon, Oct 10, 2016 at 01:07:12AM -0700, sumit-bose wrote:
> On Fri, Oct 07, 2016 at 08:21:21AM -0700, Jakub Hrozek wrote:
> > I'm afraid Coverity found something it doesn't like:
> > ```
> 
> The latest patch should fix this. In general I think it is a false
> positive because it is hard for Coverity to follow the different
> if-blocks. I tried to fix it with some minor code changes but failed, so
> I just initialized neg_cache_name in the latest version.

I agree, but I also prefer slight code-bending if it keep our
code-scans clean.

Thank you, my tests are still passing and Coverity no longer complains.

"""

See the full comment at 
https://github.com/SSSD/sssd/pull/31#issuecomment-252577955
_______________________________________________
sssd-devel mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to