Hi,

I found this little bug when reviewing Pavel's sudo patch.
>From f0724df7da5029d73c3cd0806beb1a17e9063ec3 Mon Sep 17 00:00:00 2001
From: Jakub Hrozek <jhro...@redhat.com>
Date: Wed, 8 Jun 2016 15:12:14 +0200
Subject: [PATCH] RESPONDER: Fix error check in cache_req.c

---
 src/responder/common/responder_cache_req.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/responder/common/responder_cache_req.c 
b/src/responder/common/responder_cache_req.c
index 
a3253a5efcfea014a01dc38e42ae2a4466dff1a2..c10fe3cb17196d90b0e2cc073a4a2dcdeac0bb24
 100644
--- a/src/responder/common/responder_cache_req.c
+++ b/src/responder/common/responder_cache_req.c
@@ -974,7 +974,9 @@ static errno_t cache_req_cache_search(struct tevent_req 
*req)
 
     /* Verify that the cache is up to date. */
     ret = cache_req_cache_check(req);
-    if (req != EOK) {
+    if (ret != EOK) {
+        CACHE_REQ_DEBUG(SSSDBG_OP_FAILURE, state->cr,
+                        "Cannot find info for [%s]\n", state->cr->debugobj);
         return ret;
     }
 
-- 
2.4.11

_______________________________________________
sssd-devel mailing list
sssd-devel@lists.fedorahosted.org
https://lists.fedorahosted.org/admin/lists/sssd-devel@lists.fedorahosted.org

Reply via email to