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

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