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

sumit-bose commented:
"""
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.

bye.,
Sumit

> Error: COMPILER_WARNING:
> sssd-1.14.2/src/responder/nss/nsssrv_cmd.c: scope_hint: In function 
> 'nss_cmd_getsidby_search'
> sssd-1.14.2/src/responder/nss/nsssrv_cmd.c:4590:21: warning: 'neg_cache_name' 
> may be used uninitialized in this function [-Wmaybe-uninitialized]
> #                 ret = sss_ncache_set_group(nctx->rctx->ncache, false,
> #                     ^
> # 4588|                   }
> # 4589|   
> # 4590|->                 ret = sss_ncache_set_group(nctx->rctx->ncache, 
> false,
> # 4591|                                              dom, neg_cache_name);
> # 4592|                   if (ret != EOK) {
> ```
> 
> -- 
> You are receiving this because you authored the thread.
> Reply to this email directly or view it on GitHub:
> https://github.com/SSSD/sssd/pull/31#issuecomment-252281331

"""

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

Reply via email to