|
 |
ab1ddb |
From 1e2f20f89b1b699e569dfecb7cba98ec8f6fc936 Mon Sep 17 00:00:00 2001
|
|
|
ab1ddb |
From: Sumit Bose <sbose@redhat.com>
|
|
|
ab1ddb |
Date: Thu, 15 Mar 2018 12:43:34 +0100
|
|
|
ab1ddb |
Subject: [PATCH] nss: initialize nss_enum_index in nss_setnetgrent()
|
|
|
ab1ddb |
|
|
|
ab1ddb |
setnetgrent() is the first call when looking up a netgroup and sets the
|
|
|
ab1ddb |
netgroup name for upcoming getnetgrent() and endnetgrent() calls.
|
|
|
ab1ddb |
Currently the state is reset by calling endnetgrent() but it would be
|
|
|
ab1ddb |
more robust to unconditionally reset the state in setnetgrent() as well
|
|
|
ab1ddb |
in case calling endnetgrent() was forgotten.
|
|
|
ab1ddb |
|
|
|
ab1ddb |
Related to https://pagure.io/SSSD/sssd/issue/3679
|
|
|
ab1ddb |
|
|
|
ab1ddb |
Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
|
|
|
ab1ddb |
(cherry picked from commit 37a84285aeb497ed4909d16916bbf934af3f68b3)
|
|
|
ab1ddb |
---
|
|
|
ab1ddb |
src/responder/nss/nss_cmd.c | 3 +++
|
|
|
ab1ddb |
1 file changed, 3 insertions(+)
|
|
|
ab1ddb |
|
|
|
ab1ddb |
diff --git a/src/responder/nss/nss_cmd.c b/src/responder/nss/nss_cmd.c
|
|
|
ab1ddb |
index 956ee53cb88dd24faaa95ac39c8d9540af66cfb2..9f8479b7b350823ee81b5af15199e0dda9acda8b 100644
|
|
|
ab1ddb |
--- a/src/responder/nss/nss_cmd.c
|
|
|
ab1ddb |
+++ b/src/responder/nss/nss_cmd.c
|
|
|
ab1ddb |
@@ -756,6 +756,9 @@ static errno_t nss_setnetgrent(struct cli_ctx *cli_ctx,
|
|
|
ab1ddb |
goto done;
|
|
|
ab1ddb |
}
|
|
|
ab1ddb |
|
|
|
ab1ddb |
+ state_ctx->netgrent.domain = 0;
|
|
|
ab1ddb |
+ state_ctx->netgrent.result = 0;
|
|
|
ab1ddb |
+
|
|
|
ab1ddb |
talloc_zfree(state_ctx->netgroup);
|
|
|
ab1ddb |
state_ctx->netgroup = talloc_strdup(state_ctx, netgroup);
|
|
|
ab1ddb |
if (state_ctx->netgroup == NULL) {
|
|
|
ab1ddb |
--
|
|
|
ab1ddb |
2.17.0
|
|
|
ab1ddb |
|