Blame SOURCES/0174-IFP-Fix-error-handling-in-ifp_user_get_attr_handle_r.patch

ecf709
From 271679e7a7c0c50e39c7a0989dbae77385475c60 Mon Sep 17 00:00:00 2001
ecf709
From: Jakub Hrozek <jhrozek@redhat.com>
ecf709
Date: Wed, 14 Jun 2017 18:25:21 +0200
ecf709
Subject: [PATCH 174/181] IFP: Fix error handling in
ecf709
 ifp_user_get_attr_handle_reply()
ecf709
MIME-Version: 1.0
ecf709
Content-Type: text/plain; charset=UTF-8
ecf709
Content-Transfer-Encoding: 8bit
ecf709
ecf709
This bug was introduced in 37d2194cc9ea4d0254c88a3419e2376572562bab
ecf709
ecf709
Reviewed-by: Fabiano FidĂȘncio <fidencio@redhat.com>
ecf709
(cherry picked from commit 15a76bb7bd9791a3ed1ae416f70753d32c6ff599)
ecf709
---
ecf709
 src/responder/ifp/ifpsrv_cmd.c | 8 +++++---
ecf709
 1 file changed, 5 insertions(+), 3 deletions(-)
ecf709
ecf709
diff --git a/src/responder/ifp/ifpsrv_cmd.c b/src/responder/ifp/ifpsrv_cmd.c
ecf709
index d86aed57206ba8f0a6facbd64051fa7c901513f3..fc9161e82e906ac7dde2712ffc7c0cbb58c519b7 100644
ecf709
--- a/src/responder/ifp/ifpsrv_cmd.c
ecf709
+++ b/src/responder/ifp/ifpsrv_cmd.c
ecf709
@@ -236,9 +236,11 @@ ifp_user_get_attr_handle_reply(struct sss_domain_info *domain,
ecf709
             if (strcmp(attrs[ai], "domainname") == 0) {
ecf709
                 ret = ifp_add_value_to_dict(&iter_dict, "domainname",
ecf709
                                             domain->name);
ecf709
-                DEBUG(SSSDBG_MINOR_FAILURE,
ecf709
-                      "Cannot add attribute %s to message\n", attrs[ai]);
ecf709
-                continue;
ecf709
+                if (ret != EOK) {
ecf709
+                    DEBUG(SSSDBG_MINOR_FAILURE,
ecf709
+                          "Cannot add attribute domainname to message\n");
ecf709
+                    continue;
ecf709
+                }
ecf709
             }
ecf709
 
ecf709
             el = sss_view_ldb_msg_find_element(domain, res->msgs[0], attrs[ai]);
ecf709
-- 
ecf709
2.9.4
ecf709