diff options
| author | Jakub Hrozek <jhrozek@redhat.com> | 2017-06-14 18:25:21 +0200 |
|---|---|---|
| committer | Jakub Hrozek <jhrozek@redhat.com> | 2017-06-15 10:25:50 +0200 |
| commit | 15a76bb7bd9791a3ed1ae416f70753d32c6ff599 (patch) | |
| tree | 0e8f0df4f846228c60fa2fc522183e9edce9303a /src/responder | |
| parent | 818d01b4a0d332fff06db33c0c985b8c0f1417c7 (diff) | |
| download | sssd-15a76bb7bd9791a3ed1ae416f70753d32c6ff599.tar.gz sssd-15a76bb7bd9791a3ed1ae416f70753d32c6ff599.tar.xz sssd-15a76bb7bd9791a3ed1ae416f70753d32c6ff599.zip | |
IFP: Fix error handling in ifp_user_get_attr_handle_reply()
This bug was introduced in 37d2194cc9ea4d0254c88a3419e2376572562bab
Reviewed-by: Fabiano Fidêncio <fidencio@redhat.com>
Diffstat (limited to 'src/responder')
| -rw-r--r-- | src/responder/ifp/ifpsrv_cmd.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/src/responder/ifp/ifpsrv_cmd.c b/src/responder/ifp/ifpsrv_cmd.c index d86aed572..fc9161e82 100644 --- a/src/responder/ifp/ifpsrv_cmd.c +++ b/src/responder/ifp/ifpsrv_cmd.c @@ -236,9 +236,11 @@ ifp_user_get_attr_handle_reply(struct sss_domain_info *domain, if (strcmp(attrs[ai], "domainname") == 0) { ret = ifp_add_value_to_dict(&iter_dict, "domainname", domain->name); - DEBUG(SSSDBG_MINOR_FAILURE, - "Cannot add attribute %s to message\n", attrs[ai]); - continue; + if (ret != EOK) { + DEBUG(SSSDBG_MINOR_FAILURE, + "Cannot add attribute domainname to message\n"); + continue; + } } el = sss_view_ldb_msg_find_element(domain, res->msgs[0], attrs[ai]); |