On Tue, Jul 26, 2016 at 11:12:34AM +0200, Jakub Hrozek wrote:
On Fri, Jul 22, 2016 at 09:44:33PM +0200, Sumit Bose wrote:
> Hi,
>
> this patch set should fix
https://fedorahosted.org/sssd/ticket/2958
> "Support multiple principals for IPA users" so the IPA users can log in
> with their Kerberos alias as well.
First, thank you very much for splitting the patchset into small
self-contained changes. The review was much easier this way.
A Coverity run just finished and found one more issue:
Error: UNINIT (CWE-457):
sssd-1.14.1/src/providers/ldap/sdap_async_initgroups.c:2685: var_decl: Declaring variable
"search_attr" without initializer.
sssd-1.14.1/src/providers/ldap/sdap_async_initgroups.c:2786: uninit_use: Using
uninitialized value "search_attr".
# 2784| }
# 2785|
# 2786|-> if (search_attr == NULL && state->user_base_filter == NULL) {
# 2787| DEBUG(SSSDBG_OP_FAILURE, "Missing search attribute name or
filter.\n");
# 2788|