0bcf677ee4
Resolves: rhbz#2109451 - virsh command will hang after the host run several auto test cases Resolves: rhbz#2098654 - cache_req_data_set_hybrid_lookup: cache_req_data should never be NULL Resolves: rhbz#2106685 - [regression] sssctl analyze fails to parse PAM related sssd logs
156 lines
5.3 KiB
Diff
156 lines
5.3 KiB
Diff
From 03142f8de42faf4f75465d24d3be9a49c2dd86f7 Mon Sep 17 00:00:00 2001
|
|
From: Alexey Tikhonov <atikhono@redhat.com>
|
|
Date: Fri, 29 Jul 2022 14:57:20 +0200
|
|
Subject: [PATCH] CLIENT:MC: store context mutex outside of context as it
|
|
should survive context destruction / re-initialization
|
|
MIME-Version: 1.0
|
|
Content-Type: text/plain; charset=UTF-8
|
|
Content-Transfer-Encoding: 8bit
|
|
|
|
Reviewed-by: Iker Pedrosa <ipedrosa@redhat.com>
|
|
Reviewed-by: Pavel Březina <pbrezina@redhat.com>
|
|
(cherry picked from commit 0f3a761ed9d654a61f8caed8eae3863c518b9911)
|
|
---
|
|
src/sss_client/nss_mc.h | 4 ++--
|
|
src/sss_client/nss_mc_common.c | 10 ++++++++--
|
|
src/sss_client/nss_mc_group.c | 5 +++++
|
|
src/sss_client/nss_mc_initgr.c | 5 +++++
|
|
src/sss_client/nss_mc_passwd.c | 5 +++++
|
|
src/sss_client/nss_mc_sid.c | 5 +++++
|
|
6 files changed, 30 insertions(+), 4 deletions(-)
|
|
|
|
diff --git a/src/sss_client/nss_mc.h b/src/sss_client/nss_mc.h
|
|
index b66e8f09f..de1496ccc 100644
|
|
--- a/src/sss_client/nss_mc.h
|
|
+++ b/src/sss_client/nss_mc.h
|
|
@@ -48,7 +48,7 @@ enum sss_mc_state {
|
|
struct sss_cli_mc_ctx {
|
|
enum sss_mc_state initialized;
|
|
#if HAVE_PTHREAD
|
|
- pthread_mutex_t mutex;
|
|
+ pthread_mutex_t *mutex;
|
|
#endif
|
|
int fd;
|
|
|
|
@@ -67,7 +67,7 @@ struct sss_cli_mc_ctx {
|
|
};
|
|
|
|
#if HAVE_PTHREAD
|
|
-#define SSS_CLI_MC_CTX_INITIALIZER {UNINITIALIZED, PTHREAD_MUTEX_INITIALIZER, 1, 0, NULL, 0, NULL, 0, NULL, 0, 0}
|
|
+#define SSS_CLI_MC_CTX_INITIALIZER(mtx) {UNINITIALIZED, (mtx), 1, 0, NULL, 0, NULL, 0, NULL, 0, 0}
|
|
#else
|
|
#define SSS_CLI_MC_CTX_INITIALIZER {UNINITIALIZED, 1, 0, NULL, 0, NULL, 0, NULL, 0, 0}
|
|
#endif
|
|
diff --git a/src/sss_client/nss_mc_common.c b/src/sss_client/nss_mc_common.c
|
|
index c73a93a9a..f38a4a85a 100644
|
|
--- a/src/sss_client/nss_mc_common.c
|
|
+++ b/src/sss_client/nss_mc_common.c
|
|
@@ -58,14 +58,14 @@ do { \
|
|
static void sss_mt_lock(struct sss_cli_mc_ctx *ctx)
|
|
{
|
|
#if HAVE_PTHREAD
|
|
- pthread_mutex_lock(&ctx->mutex);
|
|
+ pthread_mutex_lock(ctx->mutex);
|
|
#endif
|
|
}
|
|
|
|
static void sss_mt_unlock(struct sss_cli_mc_ctx *ctx)
|
|
{
|
|
#if HAVE_PTHREAD
|
|
- pthread_mutex_unlock(&ctx->mutex);
|
|
+ pthread_mutex_unlock(ctx->mutex);
|
|
#endif
|
|
}
|
|
|
|
@@ -131,6 +131,9 @@ errno_t sss_nss_check_header(struct sss_cli_mc_ctx *ctx)
|
|
static void sss_nss_mc_destroy_ctx(struct sss_cli_mc_ctx *ctx)
|
|
{
|
|
uint32_t active_threads = ctx->active_threads;
|
|
+#if HAVE_PTHREAD
|
|
+ pthread_mutex_t *mutex = ctx->mutex;
|
|
+#endif
|
|
|
|
if ((ctx->mmap_base != NULL) && (ctx->mmap_size != 0)) {
|
|
munmap(ctx->mmap_base, ctx->mmap_size);
|
|
@@ -143,6 +146,9 @@ static void sss_nss_mc_destroy_ctx(struct sss_cli_mc_ctx *ctx)
|
|
|
|
/* restore count of active threads */
|
|
ctx->active_threads = active_threads;
|
|
+#if HAVE_PTHREAD
|
|
+ ctx->mutex = mutex;
|
|
+#endif
|
|
}
|
|
|
|
static errno_t sss_nss_mc_init_ctx(const char *name,
|
|
diff --git a/src/sss_client/nss_mc_group.c b/src/sss_client/nss_mc_group.c
|
|
index 2ea40c435..d4f2a82ab 100644
|
|
--- a/src/sss_client/nss_mc_group.c
|
|
+++ b/src/sss_client/nss_mc_group.c
|
|
@@ -29,7 +29,12 @@
|
|
#include "nss_mc.h"
|
|
#include "shared/safealign.h"
|
|
|
|
+#if HAVE_PTHREAD
|
|
+static pthread_mutex_t gr_mc_ctx_mutex = PTHREAD_MUTEX_INITIALIZER;
|
|
+static struct sss_cli_mc_ctx gr_mc_ctx = SSS_CLI_MC_CTX_INITIALIZER(&gr_mc_ctx_mutex);
|
|
+#else
|
|
static struct sss_cli_mc_ctx gr_mc_ctx = SSS_CLI_MC_CTX_INITIALIZER;
|
|
+#endif
|
|
|
|
static errno_t sss_nss_mc_parse_result(struct sss_mc_rec *rec,
|
|
struct group *result,
|
|
diff --git a/src/sss_client/nss_mc_initgr.c b/src/sss_client/nss_mc_initgr.c
|
|
index b05946263..bd7282935 100644
|
|
--- a/src/sss_client/nss_mc_initgr.c
|
|
+++ b/src/sss_client/nss_mc_initgr.c
|
|
@@ -32,7 +32,12 @@
|
|
#include "nss_mc.h"
|
|
#include "shared/safealign.h"
|
|
|
|
+#if HAVE_PTHREAD
|
|
+static pthread_mutex_t initgr_mc_ctx_mutex = PTHREAD_MUTEX_INITIALIZER;
|
|
+static struct sss_cli_mc_ctx initgr_mc_ctx = SSS_CLI_MC_CTX_INITIALIZER(&initgr_mc_ctx_mutex);
|
|
+#else
|
|
static struct sss_cli_mc_ctx initgr_mc_ctx = SSS_CLI_MC_CTX_INITIALIZER;
|
|
+#endif
|
|
|
|
static errno_t sss_nss_mc_parse_result(struct sss_mc_rec *rec,
|
|
long int *start, long int *size,
|
|
diff --git a/src/sss_client/nss_mc_passwd.c b/src/sss_client/nss_mc_passwd.c
|
|
index 01c6801da..256d48444 100644
|
|
--- a/src/sss_client/nss_mc_passwd.c
|
|
+++ b/src/sss_client/nss_mc_passwd.c
|
|
@@ -28,7 +28,12 @@
|
|
#include <time.h>
|
|
#include "nss_mc.h"
|
|
|
|
+#if HAVE_PTHREAD
|
|
+static pthread_mutex_t pw_mc_ctx_mutex = PTHREAD_MUTEX_INITIALIZER;
|
|
+static struct sss_cli_mc_ctx pw_mc_ctx = SSS_CLI_MC_CTX_INITIALIZER(&pw_mc_ctx_mutex);
|
|
+#else
|
|
static struct sss_cli_mc_ctx pw_mc_ctx = SSS_CLI_MC_CTX_INITIALIZER;
|
|
+#endif
|
|
|
|
static errno_t sss_nss_mc_parse_result(struct sss_mc_rec *rec,
|
|
struct passwd *result,
|
|
diff --git a/src/sss_client/nss_mc_sid.c b/src/sss_client/nss_mc_sid.c
|
|
index af7d7bbd5..52e684da5 100644
|
|
--- a/src/sss_client/nss_mc_sid.c
|
|
+++ b/src/sss_client/nss_mc_sid.c
|
|
@@ -30,7 +30,12 @@
|
|
#include "util/mmap_cache.h"
|
|
#include "idmap/sss_nss_idmap.h"
|
|
|
|
+#if HAVE_PTHREAD
|
|
+static pthread_mutex_t sid_mc_ctx_mutex = PTHREAD_MUTEX_INITIALIZER;
|
|
+static struct sss_cli_mc_ctx sid_mc_ctx = SSS_CLI_MC_CTX_INITIALIZER(&sid_mc_ctx_mutex);
|
|
+#else
|
|
static struct sss_cli_mc_ctx sid_mc_ctx = SSS_CLI_MC_CTX_INITIALIZER;
|
|
+#endif
|
|
|
|
static errno_t mc_get_sid_by_typed_id(uint32_t id, enum sss_id_type object_type,
|
|
char **sid, uint32_t *type,
|
|
--
|
|
2.37.1
|
|
|