Blame SOURCES/0047-NSS-Use-enum_ctx-as-memory_context-in-_setnetgrent_s.patch

9f2ebf
From 3b4479b63ae812c1ef355c1c697caddd882b1b8f Mon Sep 17 00:00:00 2001
9f2ebf
From: Sumit Bose <sbose@redhat.com>
9f2ebf
Date: Mon, 6 Nov 2017 17:03:19 +0100
9f2ebf
Subject: [PATCH 47/47] NSS: Use enum_ctx as memory_context in
9f2ebf
 _setnetgrent_set_timeout()
9f2ebf
MIME-Version: 1.0
9f2ebf
Content-Type: text/plain; charset=UTF-8
9f2ebf
Content-Transfer-Encoding: 8bit
9f2ebf
9f2ebf
We've noticed some crashes that happened because enum_ctx is already
9f2ebf
freed, but the timeout handler is still called. In order to avoid that,
9f2ebf
let's remove the timeout handler when enum_ctx is freed at other places.
9f2ebf
9f2ebf
Resolves: https://pagure.io/SSSD/sssd/issue/3523
9f2ebf
9f2ebf
Signed-off-by: Fabiano Fidêncio <fidencio@redhat.com>
9f2ebf
9f2ebf
Reviewed-by: Pavel Březina <pbrezina@redhat.com>
9f2ebf
(cherry picked from commit f6a1cef87abdd983d6b5349cd341c9a249826577)
9f2ebf
---
9f2ebf
 src/responder/nss/nss_enum.c | 2 +-
9f2ebf
 1 file changed, 1 insertion(+), 1 deletion(-)
9f2ebf
9f2ebf
diff --git a/src/responder/nss/nss_enum.c b/src/responder/nss/nss_enum.c
9f2ebf
index aa7d8428f37e943a6b5904495c40ad4b8011b767..da844fbced529f606a3e98669fb7b95e0696ce00 100644
9f2ebf
--- a/src/responder/nss/nss_enum.c
9f2ebf
+++ b/src/responder/nss/nss_enum.c
9f2ebf
@@ -283,7 +283,7 @@ nss_setnetgrent_set_timeout(struct tevent_context *ev,
9f2ebf
     timeout = enum_ctx->result[0]->domain->netgroup_timeout;
9f2ebf
 
9f2ebf
     tv = tevent_timeval_current_ofs(timeout, 0);
9f2ebf
-    te = tevent_add_timer(ev, nss_ctx, tv, nss_setnetgrent_timeout, enum_ctx);
9f2ebf
+    te = tevent_add_timer(ev, enum_ctx, tv, nss_setnetgrent_timeout, enum_ctx);
9f2ebf
     if (te == NULL) {
9f2ebf
         DEBUG(SSSDBG_CRIT_FAILURE,
9f2ebf
               "Could not set up life timer for enumeration object.\n");
9f2ebf
-- 
9f2ebf
2.13.6
9f2ebf