On Thu, Jun 26, 2014 at 05:33:39PM +0200, Pavel Reichl wrote:
Hello,
please see attached patch.
good catch, obvious ACK.
bye,
Sumit
Thanks,
PR
From 7249e59cd00050a81780d99e7bdfcdeb0a01a144 Mon Sep 17 00:00:00
2001
From: Pavel Reichl <preichl(a)redhat.com>
Date: Thu, 26 Jun 2014 16:21:16 +0100
Subject: [PATCH] sudo: return after tevent_req_error
Don't call tevent_req_done after tevent_req_error (for the same request).
---
src/responder/sudo/sudosrv_query.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/responder/sudo/sudosrv_query.c b/src/responder/sudo/sudosrv_query.c
index 632afa712c9c08af7ed253f77705e35f49b6b039..4d19514cd63f432d7cee2bbfbece237fb56c5dec
100644
--- a/src/responder/sudo/sudosrv_query.c
+++ b/src/responder/sudo/sudosrv_query.c
@@ -364,6 +364,7 @@ static void sudosrv_parse_query_done(struct tevent_req *subreq)
talloc_free(subreq);
if (ret != EOK) {
tevent_req_error(req, ret);
+ return;
}
tevent_req_done(req);
--
1.8.4.2
_______________________________________________
sssd-devel mailing list
sssd-devel(a)lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/sssd-devel