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