URL:
https://github.com/SSSD/sssd/pull/89
Title: #89: nss: rewrite nss responder so it uses cache_req
lslebodn commented:
"""
Congratulation you broke integration tests on all distributions.
```
ldap_local_override_test.py::test_regr_2790_override FAILED
test_memory_cache.py::test_getgrnam_simple_with_mc FAILED
test_memory_cache.py::test_getgrnam_membership_with_mc FAILED
test_memory_cache.py::test_invalidation_of_gids_after_initgroups FAILED
test_memory_cache.py::test_initgroups_without_change_in_membership FAILED
test_netgroup.py::test_remove_step_by_step FAILED
test_netgroup.py::test_removing_nested_netgroups FAILED
test_netgroup.py::test_offline_netgroups FAILED
```
and uninitialized variable is used.
```
src/responder/nss/nss_protocol_svcent.c: In function ‘nss_protocol_fill_svcent’:
src/responder/nss/nss_protocol_svcent.c:134:15: error: ‘num_aliases’ may be used
uninitialized in this function [-Werror=maybe-uninitialized]
aliases = talloc_zero_array(mem_ctx, struct sized_string, num_aliases + 1);
^~~~~~~~~~~~~~~~~
src/responder/nss/nss_protocol_svcent.c:121:14: note: ‘num_aliases’ was declared here
uint32_t num_aliases;
^~~~~~~~~~~
CC src/responder/nss/nss_iface.o
```
Maybe it's related to failures in integration tests.
"""
See the full comment at
https://github.com/SSSD/sssd/pull/89#issuecomment-262825019