[SSSD] [PATCH] LDAP: Check return value

Jakub Hrozek jhrozek at redhat.com
Mon Sep 8 18:31:03 UTC 2014


Reported by Coverity.
-------------- next part --------------
>From e2b5f6e519a9c5ea4c6bb669ac75c712fa42fa40 Mon Sep 17 00:00:00 2001
From: Jakub Hrozek <jhrozek at redhat.com>
Date: Mon, 8 Sep 2014 20:30:25 +0200
Subject: [PATCH] LDAP: Check return value

Reported by Coverity
---
 src/providers/ldap/sdap_access.c | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/src/providers/ldap/sdap_access.c b/src/providers/ldap/sdap_access.c
index 3b2d3970cdc2efa395dfe99f1647d044d2f7393d..dda60fc2886171671f2ce7b511693ab920faa1ec 100644
--- a/src/providers/ldap/sdap_access.c
+++ b/src/providers/ldap/sdap_access.c
@@ -1461,6 +1461,12 @@ static void sdap_access_lock_get_lockout_done(struct tevent_req *subreq)
     state = tevent_req_data(req, struct sdap_access_lock_req_ctx);
 
     ret = sdap_get_generic_recv(subreq, state, &num_results, &results);
+    if (ret != EOK) {
+        DEBUG(SSSDBG_OP_FAILURE, "Cannot retrieve ppolicy\n");
+        ret = ERR_NETWORK_IO;
+        goto done;
+    }
+
     talloc_zfree(subreq);
 
     /* Check the number of responses we got
-- 
1.9.3



More information about the sssd-devel mailing list