[prev in list] [next in list] [prev in thread] [next in thread] 

List:       sssd-devel
Subject:    [PATCH 07/18] LDAP: check return value of sysdb_attrs_get_el
From:       jhrozek () redhat ! com (Jakub Hrozek)
Date:       2012-04-25 14:57:06
Message-ID: mailman.15.1335441111.29027.sssd-devel () lists ! fedorahosted ! org
[Download RAW message or body]

---
 src/providers/ldap/sdap_async_groups.c |    7 +++++++
 1 files changed, 7 insertions(+), 0 deletions(-)

diff --git a/src/providers/ldap/sdap_async_groups.c b/src/providers/ldap/sdap_async_groups.c
index c7e4ac2794947f2ca040c644ab94f2ce835026c6..d6877a37484c8a4d1149230786dccd004416cec0 100644
--- a/src/providers/ldap/sdap_async_groups.c
+++ b/src/providers/ldap/sdap_async_groups.c
@@ -1092,6 +1092,13 @@ next:
          */
         ret = sysdb_attrs_get_el(state->group,
                 state->opts->group_map[SDAP_AT_GROUP_MEMBER].sys_name, &el);
+        if (ret != EOK) {
+            DEBUG(SSSDBG_CRIT_FAILURE,
+                  ("Failed to get the group member attribute [%d]: %s\n",
+                  ret, strerror(ret)));
+            tevent_req_error(req, ret);
+            return;
+        }
         el->values = talloc_steal(state->group, state->sysdb_dns->values);
         el->num_values = state->sysdb_dns->num_values;
         DEBUG(9, ("Processed Group - Done\n"));
-- 
1.7.7.6


--BOKacYhQ+x31HxR3
Content-Type: text/plain; charset=us-ascii
Content-Disposition: attachment; filename="0008-RESPONDER-check-return-value-from-confdb_get_int.patch"


[prev in list] [next in list] [prev in thread] [next in thread]