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