Title: #444: NSS: Use netgr as memory context in set_netgr_lifetime() - Patch for
SSSD-1.14
pbrezina commented:
"""
On 11/13/2017 06:11 PM, lslebodn wrote:
*@lslebodn* commented on this pull request.
------------------------------------------------------------------------
In src/responder/nss/nsssrv_netgroup.c
<
https://github.com/SSSD/sssd/pull/444#discussion_r150604225>:
> @@ -428,7 +428,7 @@ static void set_netgr_lifetime(uint32_t lifetime,
tv = tevent_timeval_current_ofs(lifetime, 0);
te = tevent_add_timer(step_ctx->nctx->rctx->ev,
- step_ctx->nctx->gctx, tv,
+ netgr, tv,
Why did you decide to use |netgr| as a context?
IMHO it is cleaner to use |struct nss_ctx| instead of |struct getent_ctx|
The timer will free `netgr`. IMHO it is correct to cancel the timer when
`netgr` is no longer available therefore using it as memory context is
correct.
"""
See the full comment at