.gitignore

@@ -1 +1 @@
-SOURCES/sssd-2.9.4.tar.gz
+SOURCES/sssd-2.8.2.tar.gz

.sssd.metadata

@@ -1 +1 @@
-574f6cec9ee12dd943e4305286845343ab7bb891 SOURCES/sssd-2.9.4.tar.gz
+4101c2869e8f952fccab841cd2e46fd18f10465d SOURCES/sssd-2.8.2.tar.gz

SOURCES/0001-ldap-update-shadow-last-change-in-sysdb-as-well.patch

@@ -0,0 +1,158 @@
+From d7da2966f5931bac3b17f42e251adbbb7e793619 Mon Sep 17 00:00:00 2001
+From: =?UTF-8?q?Pavel=20B=C5=99ezina?= <pbrezina@redhat.com>
+Date: Thu, 8 Dec 2022 15:14:05 +0100
+Subject: [PATCH] ldap: update shadow last change in sysdb as well
+MIME-Version: 1.0
+Content-Type: text/plain; charset=UTF-8
+Content-Transfer-Encoding: 8bit
+
+Otherwise pam can use the changed information whe id chaching is
+enabled, so next authentication that fits into the id timeout
+(5 seconds by default) will still sees the password as expired.
+
+Resolves: https://github.com/SSSD/sssd/issues/6477
+
+Reviewed-by: Sumit Bose <sbose@redhat.com>
+Reviewed-by: Tomáš Halman <thalman@redhat.com>
+(cherry picked from commit 7e8b97c14b8ef218d6ea23214be28d25dba13886)
+---
+ src/db/sysdb.h                 |  4 ++++
+ src/db/sysdb_ops.c             | 32 ++++++++++++++++++++++++++++++++
+ src/providers/ldap/ldap_auth.c | 21 ++++++++++++++++-----
+ 3 files changed, 52 insertions(+), 5 deletions(-)
+
+diff --git a/src/db/sysdb.h b/src/db/sysdb.h
+index 7c666f5c4..06b44f5ba 100644
+--- a/src/db/sysdb.h
++++ b/src/db/sysdb.h
+@@ -1061,6 +1061,10 @@ int sysdb_set_user_attr(struct sss_domain_info *domain,
+                         struct sysdb_attrs *attrs,
+                         int mod_op);
+ 
++errno_t sysdb_update_user_shadow_last_change(struct sss_domain_info *domain,
++                                             const char *name,
++                                             const char *attrname);
++
+ /* Replace group attrs */
+ int sysdb_set_group_attr(struct sss_domain_info *domain,
+                          const char *name,
+diff --git a/src/db/sysdb_ops.c b/src/db/sysdb_ops.c
+index 0d6f2d5cd..ed0df9872 100644
+--- a/src/db/sysdb_ops.c
++++ b/src/db/sysdb_ops.c
+@@ -1485,6 +1485,38 @@ done:
+     return ret;
+ }
+ 
++errno_t sysdb_update_user_shadow_last_change(struct sss_domain_info *domain,
++                                             const char *name,
++                                             const char *attrname)
++{
++    struct sysdb_attrs *attrs;
++    char *value;
++    errno_t ret;
++
++    attrs = sysdb_new_attrs(NULL);
++    if (attrs == NULL) {
++        return ENOMEM;
++    }
++
++    /* The attribute contains number of days since the epoch */
++    value = talloc_asprintf(attrs, "%ld", (long)time(NULL)/86400);
++    if (value == NULL) {
++        ret = ENOMEM;
++        goto done;
++    }
++
++    ret = sysdb_attrs_add_string(attrs, attrname, value);
++    if (ret != EOK) {
++        goto done;
++    }
++
++    ret = sysdb_set_user_attr(domain, name, attrs, SYSDB_MOD_REP);
++
++done:
++    talloc_free(attrs);
++    return ret;
++}
++
+ /* =Replace-Attributes-On-Group=========================================== */
+ 
+ int sysdb_set_group_attr(struct sss_domain_info *domain,
+diff --git a/src/providers/ldap/ldap_auth.c b/src/providers/ldap/ldap_auth.c
+index 6404a9d3a..96b9d6df4 100644
+--- a/src/providers/ldap/ldap_auth.c
++++ b/src/providers/ldap/ldap_auth.c
+@@ -1240,6 +1240,7 @@ struct sdap_pam_chpass_handler_state {
+     struct pam_data *pd;
+     struct sdap_handle *sh;
+     char *dn;
++    enum pwexpire pw_expire_type;
+ };
+ 
+ static void sdap_pam_chpass_handler_auth_done(struct tevent_req *subreq);
+@@ -1339,7 +1340,6 @@ static void sdap_pam_chpass_handler_auth_done(struct tevent_req *subreq)
+ {
+     struct sdap_pam_chpass_handler_state *state;
+     struct tevent_req *req;
+-    enum pwexpire pw_expire_type;
+     void *pw_expire_data;
+     size_t msg_len;
+     uint8_t *msg;
+@@ -1349,7 +1349,7 @@ static void sdap_pam_chpass_handler_auth_done(struct tevent_req *subreq)
+     state = tevent_req_data(req, struct sdap_pam_chpass_handler_state);
+ 
+     ret = auth_recv(subreq, state, &state->sh, &state->dn,
+-                    &pw_expire_type, &pw_expire_data);
++                    &state->pw_expire_type, &pw_expire_data);
+     talloc_free(subreq);
+ 
+     if ((ret == EOK || ret == ERR_PASSWORD_EXPIRED) &&
+@@ -1361,7 +1361,7 @@ static void sdap_pam_chpass_handler_auth_done(struct tevent_req *subreq)
+     }
+ 
+     if (ret == EOK) {
+-        switch (pw_expire_type) {
++        switch (state->pw_expire_type) {
+         case PWEXPIRE_SHADOW:
+             ret = check_pwexpire_shadow(pw_expire_data, time(NULL), NULL);
+             break;
+@@ -1381,7 +1381,8 @@ static void sdap_pam_chpass_handler_auth_done(struct tevent_req *subreq)
+             break;
+         default:
+             DEBUG(SSSDBG_CRIT_FAILURE,
+-                  "Unknown password expiration type %d.\n", pw_expire_type);
++                  "Unknown password expiration type %d.\n",
++                  state->pw_expire_type);
+             state->pd->pam_status = PAM_SYSTEM_ERR;
+             goto done;
+         }
+@@ -1392,7 +1393,8 @@ static void sdap_pam_chpass_handler_auth_done(struct tevent_req *subreq)
+         case ERR_PASSWORD_EXPIRED:
+             DEBUG(SSSDBG_TRACE_LIBS,
+                   "user [%s] successfully authenticated.\n", state->dn);
+-            ret = sdap_pam_chpass_handler_change_step(state, req, pw_expire_type);
++            ret = sdap_pam_chpass_handler_change_step(state, req,
++                                                      state->pw_expire_type);
+             if (ret != EOK) {
+                 DEBUG(SSSDBG_OP_FAILURE,
+                       "sdap_pam_chpass_handler_change_step() failed.\n");
+@@ -1506,6 +1508,15 @@ static void sdap_pam_chpass_handler_chpass_done(struct tevent_req *subreq)
+ 
+     switch (ret) {
+     case EOK:
++        if (state->pw_expire_type == PWEXPIRE_SHADOW) {
++            ret = sysdb_update_user_shadow_last_change(state->be_ctx->domain,
++                    state->pd->user, SYSDB_SHADOWPW_LASTCHANGE);
++            if (ret != EOK) {
++                state->pd->pam_status = PAM_SYSTEM_ERR;
++                goto done;
++            }
++        }
++
+         state->pd->pam_status = PAM_SUCCESS;
+         break;
+     case ERR_CHPASS_DENIED:
+-- 
+2.37.3
+

SOURCES/0001-sssd-adding-mail-as-case-insensitive.patch

@@ -1,144 +0,0 @@
-From dd0f63246aa75d5f53b44cbc185e88833e79976e Mon Sep 17 00:00:00 2001
-From: Andre Boscatto <andreboscatto@gmail.com>
-Date: Wed, 7 Feb 2024 12:28:28 +0100
-Subject: [PATCH] sssd: adding mail as case insensitive
-MIME-Version: 1.0
-Content-Type: text/plain; charset=UTF-8
-Content-Transfer-Encoding: 8bit
-
-Resolves: https://github.com/SSSD/sssd/issues/7173
-
-Reviewed-by: Iker Pedrosa <ipedrosa@redhat.com>
-Reviewed-by: Tomáš Halman <thalman@redhat.com>
-(cherry picked from commit 945cebcf72ef53ea0368f19c09e710f7fff11b51)
----
- src/db/sysdb_init.c    |  7 ++++++
- src/db/sysdb_private.h |  5 +++-
- src/db/sysdb_upgrade.c | 56 ++++++++++++++++++++++++++++++++++++++++++
- 3 files changed, 67 insertions(+), 1 deletion(-)
-
-diff --git a/src/db/sysdb_init.c b/src/db/sysdb_init.c
-index c2ea6c369..38a9cd64a 100644
---- a/src/db/sysdb_init.c
-+++ b/src/db/sysdb_init.c
-@@ -603,6 +603,13 @@ static errno_t sysdb_domain_cache_upgrade(TALLOC_CTX *mem_ctx,
-         }
-     }
- 
-+    if (strcmp(version, SYSDB_VERSION_0_23) == 0) {
-+        ret = sysdb_upgrade_23(sysdb, &version);
-+        if (ret != EOK) {
-+            goto done;
-+        }
-+    }
-+
-     ret = EOK;
- done:
-     sysdb->ldb = save_ldb;
-diff --git a/src/db/sysdb_private.h b/src/db/sysdb_private.h
-index 1f55007bc..63f7b5601 100644
---- a/src/db/sysdb_private.h
-+++ b/src/db/sysdb_private.h
-@@ -23,6 +23,7 @@
- #ifndef __INT_SYS_DB_H__
- #define __INT_SYS_DB_H__
- 
-+#define SYSDB_VERSION_0_24 "0.24"
- #define SYSDB_VERSION_0_23 "0.23"
- #define SYSDB_VERSION_0_22 "0.22"
- #define SYSDB_VERSION_0_21 "0.21"
-@@ -47,7 +48,7 @@
- #define SYSDB_VERSION_0_2 "0.2"
- #define SYSDB_VERSION_0_1 "0.1"
- 
--#define SYSDB_VERSION SYSDB_VERSION_0_23
-+#define SYSDB_VERSION SYSDB_VERSION_0_24
- 
- #define SYSDB_BASE_LDIF \
-      "dn: @ATTRIBUTES\n" \
-@@ -60,6 +61,7 @@
-      "objectclass: CASE_INSENSITIVE\n" \
-      "ipHostNumber: CASE_INSENSITIVE\n" \
-      "ipNetworkNumber: CASE_INSENSITIVE\n" \
-+     "mail: CASE_INSENSITIVE\n" \
-      "\n" \
-      "dn: @INDEXLIST\n" \
-      "@IDXATTR: cn\n" \
-@@ -191,6 +193,7 @@ int sysdb_upgrade_19(struct sysdb_ctx *sysdb, const char **ver);
- int sysdb_upgrade_20(struct sysdb_ctx *sysdb, const char **ver);
- int sysdb_upgrade_21(struct sysdb_ctx *sysdb, const char **ver);
- int sysdb_upgrade_22(struct sysdb_ctx *sysdb, const char **ver);
-+int sysdb_upgrade_23(struct sysdb_ctx *sysdb, const char **ver);
- 
- int sysdb_ts_upgrade_01(struct sysdb_ctx *sysdb, const char **ver);
- 
-diff --git a/src/db/sysdb_upgrade.c b/src/db/sysdb_upgrade.c
-index 346a1cb0b..56083e6be 100644
---- a/src/db/sysdb_upgrade.c
-+++ b/src/db/sysdb_upgrade.c
-@@ -2718,6 +2718,62 @@ done:
-     return ret;
- }
- 
-+int sysdb_upgrade_23(struct sysdb_ctx *sysdb, const char **ver)
-+{
-+    TALLOC_CTX *tmp_ctx;
-+    int ret;
-+    struct ldb_message *msg;
-+    struct upgrade_ctx *ctx;
-+
-+    tmp_ctx = talloc_new(NULL);
-+    if (!tmp_ctx) {
-+        return ENOMEM;
-+    }
-+
-+    ret = commence_upgrade(sysdb, sysdb->ldb, SYSDB_VERSION_0_24, &ctx);
-+    if (ret) {
-+        return ret;
-+    }
-+
-+    /* Add new indexes */
-+    msg = ldb_msg_new(tmp_ctx);
-+    if (!msg) {
-+        ret = ENOMEM;
-+        goto done;
-+    }
-+    msg->dn = ldb_dn_new(tmp_ctx, sysdb->ldb, "@ATTRIBUTES");
-+    if (!msg->dn) {
-+        ret = ENOMEM;
-+        goto done;
-+    }
-+
-+    /* Case insensitive search for mail */
-+    ret = ldb_msg_add_empty(msg, SYSDB_USER_EMAIL, LDB_FLAG_MOD_ADD, NULL);
-+    if (ret != LDB_SUCCESS) {
-+        ret = ENOMEM;
-+        goto done;
-+    }
-+    ret = ldb_msg_add_string(msg, SYSDB_USER_EMAIL, "CASE_INSENSITIVE");
-+    if (ret != LDB_SUCCESS) {
-+        ret = ENOMEM;
-+        goto done;
-+    }
-+
-+    ret = ldb_modify(sysdb->ldb, msg);
-+    if (ret != LDB_SUCCESS) {
-+        ret = sysdb_error_to_errno(ret);
-+        goto done;
-+    }
-+
-+    /* conversion done, update version number */
-+    ret = update_version(ctx);
-+
-+done:
-+    ret = finish_upgrade(ret, &ctx, ver);
-+    talloc_free(tmp_ctx);
-+    return ret;
-+}
-+
- int sysdb_ts_upgrade_01(struct sysdb_ctx *sysdb, const char **ver)
- {
-     struct upgrade_ctx *ctx;
--- 
-2.41.0
-

SOURCES/0002-SSS_CLIENT-fix-error-codes-returned-by-common-read-w.patch

@@ -0,0 +1,58 @@
+From f3333b9dbeda33a9344b458accaa4ff372adb660 Mon Sep 17 00:00:00 2001
+From: Alexey Tikhonov <atikhono@redhat.com>
+Date: Fri, 3 Feb 2023 11:35:42 +0100
+Subject: [PATCH 2/4] SSS_CLIENT: fix error codes returned by common
+ read/write/check helpers.
+MIME-Version: 1.0
+Content-Type: text/plain; charset=UTF-8
+Content-Transfer-Encoding: 8bit
+
+It's kind of expected that in case `(POLLERR | POLLHUP | POLLNVAL)`
+error condition is detected, regular `POLLIN/POLLOUT` won't be set.
+Error code set by error condition should have a priority. This enables
+users of this helper to retry attempt (as designed).
+
+Reviewed-by: Pavel Březina <pbrezina@redhat.com>
+Reviewed-by: Sumit Bose <sbose@redhat.com>
+(cherry picked from commit 0b8638d8de435384562f17d041655887b73523cd)
+---
+ src/sss_client/common.c | 9 +++------
+ 1 file changed, 3 insertions(+), 6 deletions(-)
+
+diff --git a/src/sss_client/common.c b/src/sss_client/common.c
+index 2c888faa9..27e09f6f3 100644
+--- a/src/sss_client/common.c
++++ b/src/sss_client/common.c
+@@ -161,8 +161,7 @@ static enum sss_status sss_cli_send_req(enum sss_cli_command cmd,
+         case 1:
+             if (pfd.revents & (POLLERR | POLLHUP | POLLNVAL)) {
+                 *errnop = EPIPE;
+-            }
+-            if (!(pfd.revents & POLLOUT)) {
++            } else if (!(pfd.revents & POLLOUT)) {
+                 *errnop = EBUSY;
+             }
+             break;
+@@ -273,8 +272,7 @@ static enum sss_status sss_cli_recv_rep(enum sss_cli_command cmd,
+             }
+             if (pfd.revents & (POLLERR | POLLNVAL)) {
+                 *errnop = EPIPE;
+-            }
+-            if (!(pfd.revents & POLLIN)) {
++            } else if (!(pfd.revents & POLLIN)) {
+                 *errnop = EBUSY;
+             }
+             break;
+@@ -725,8 +723,7 @@ static enum sss_status sss_cli_check_socket(int *errnop,
+         case 1:
+             if (pfd.revents & (POLLERR | POLLHUP | POLLNVAL)) {
+                 *errnop = EPIPE;
+-            }
+-            if (!(pfd.revents & (POLLIN | POLLOUT))) {
++            } else if (!(pfd.revents & (POLLIN | POLLOUT))) {
+                 *errnop = EBUSY;
+             }
+             break;
+-- 
+2.37.3
+

SOURCES/0002-sdap-add-search_bases-option-to-groups_by_user_send.patch

@@ -1,154 +0,0 @@
-From a7621a5b464af7a3c8409dcbde038b35fee2c895 Mon Sep 17 00:00:00 2001
-From: Sumit Bose <sbose@redhat.com>
-Date: Tue, 23 Jan 2024 13:47:53 +0100
-Subject: [PATCH 2/3] sdap: add search_bases option to groups_by_user_send()
-MIME-Version: 1.0
-Content-Type: text/plain; charset=UTF-8
-Content-Transfer-Encoding: 8bit
-
-AD handles users and computer objects very similar and so does SSSD's
-GPO code when lookup up the host's group-memberships. But users and
-computers might be stored in different sub-tree of the AD LDAP tree and
-if a dedicated user search base is given with the ldap_user_search_base
-option in sssd.conf the host object might be in a different sub-tree. To
-make sure the host can still be found this patch uses the base DN of
-the LDAP tree when searching for hosts in the GPO code.
-
-Resolves: https://github.com/SSSD/sssd/issues/5708
-
-Reviewed-by: Alejandro López <allopez@redhat.com>
-Reviewed-by: Tomáš Halman <thalman@redhat.com>
-(cherry picked from commit 29a77c6e79020d7e8cb474b4d3b394d390eba196)
----
- src/providers/ad/ad_gpo.c                  | 10 ++++++++++
- src/providers/ldap/ldap_common.h           |  1 +
- src/providers/ldap/ldap_id.c               |  6 +++++-
- src/providers/ldap/sdap_async.h            |  1 +
- src/providers/ldap/sdap_async_initgroups.c |  4 +++-
- 5 files changed, 20 insertions(+), 2 deletions(-)
-
-diff --git a/src/providers/ad/ad_gpo.c b/src/providers/ad/ad_gpo.c
-index 94959c36b..b0ee3e616 100644
---- a/src/providers/ad/ad_gpo.c
-+++ b/src/providers/ad/ad_gpo.c
-@@ -2091,6 +2091,7 @@ ad_gpo_connect_done(struct tevent_req *subreq)
-     char *server_uri;
-     LDAPURLDesc *lud;
-     struct sdap_domain *sdom;
-+    struct sdap_search_base **search_bases;
- 
-     req = tevent_req_callback_data(subreq, struct tevent_req);
-     state = tevent_req_data(req, struct ad_gpo_access_state);
-@@ -2184,9 +2185,18 @@ ad_gpo_connect_done(struct tevent_req *subreq)
-         goto done;
-     }
- 
-+    ret = common_parse_search_base(state, sdom->basedn, state->ldb_ctx,
-+                                   "AD_HOSTS", NULL, &search_bases);
-+    if (ret != EOK) {
-+        DEBUG(SSSDBG_OP_FAILURE,
-+              "Failed to create dedicated search base for host lookups, "
-+              "trying with user search base.");
-+    }
-+
-     subreq = groups_by_user_send(state, state->ev,
-                                  state->access_ctx->ad_id_ctx->sdap_id_ctx,
-                                  sdom, state->conn,
-+                                 search_bases,
-                                  state->host_fqdn,
-                                  BE_FILTER_NAME,
-                                  NULL,
-diff --git a/src/providers/ldap/ldap_common.h b/src/providers/ldap/ldap_common.h
-index 7159d6356..2c984ef50 100644
---- a/src/providers/ldap/ldap_common.h
-+++ b/src/providers/ldap/ldap_common.h
-@@ -304,6 +304,7 @@ struct tevent_req *groups_by_user_send(TALLOC_CTX *memctx,
-                                        struct sdap_id_ctx *ctx,
-                                        struct sdap_domain *sdom,
-                                        struct sdap_id_conn_ctx *conn,
-+                                       struct sdap_search_base **search_bases,
-                                        const char *filter_value,
-                                        int filter_type,
-                                        const char *extra_value,
-diff --git a/src/providers/ldap/ldap_id.c b/src/providers/ldap/ldap_id.c
-index da54816bd..b3ea2333f 100644
---- a/src/providers/ldap/ldap_id.c
-+++ b/src/providers/ldap/ldap_id.c
-@@ -1139,6 +1139,7 @@ struct groups_by_user_state {
-     struct sdap_id_op *op;
-     struct sysdb_ctx *sysdb;
-     struct sss_domain_info *domain;
-+    struct sdap_search_base **search_bases;
- 
-     const char *filter_value;
-     int filter_type;
-@@ -1160,6 +1161,7 @@ struct tevent_req *groups_by_user_send(TALLOC_CTX *memctx,
-                                        struct sdap_id_ctx *ctx,
-                                        struct sdap_domain *sdom,
-                                        struct sdap_id_conn_ctx *conn,
-+                                       struct sdap_search_base **search_bases,
-                                        const char *filter_value,
-                                        int filter_type,
-                                        const char *extra_value,
-@@ -1192,6 +1194,7 @@ struct tevent_req *groups_by_user_send(TALLOC_CTX *memctx,
-     state->extra_value = extra_value;
-     state->domain = sdom->dom;
-     state->sysdb = sdom->dom->sysdb;
-+    state->search_bases = search_bases;
- 
-     if (state->domain->type == DOM_TYPE_APPLICATION || set_non_posix) {
-         state->non_posix = true;
-@@ -1254,6 +1257,7 @@ static void groups_by_user_connect_done(struct tevent_req *subreq)
-                                   sdap_id_op_handle(state->op),
-                                   state->ctx,
-                                   state->conn,
-+                                  state->search_bases,
-                                   state->filter_value,
-                                   state->filter_type,
-                                   state->extra_value,
-@@ -1449,7 +1453,7 @@ sdap_handle_acct_req_send(TALLOC_CTX *mem_ctx,
-         }
- 
-         subreq = groups_by_user_send(state, be_ctx->ev, id_ctx,
--                                     sdom, conn,
-+                                     sdom, conn, NULL,
-                                      ar->filter_value,
-                                      ar->filter_type,
-                                      ar->extra_value,
-diff --git a/src/providers/ldap/sdap_async.h b/src/providers/ldap/sdap_async.h
-index 5458d21f1..89245f41f 100644
---- a/src/providers/ldap/sdap_async.h
-+++ b/src/providers/ldap/sdap_async.h
-@@ -158,6 +158,7 @@ struct tevent_req *sdap_get_initgr_send(TALLOC_CTX *memctx,
-                                         struct sdap_handle *sh,
-                                         struct sdap_id_ctx *id_ctx,
-                                         struct sdap_id_conn_ctx *conn,
-+                                        struct sdap_search_base **search_bases,
-                                         const char *name,
-                                         int filter_type,
-                                         const char *extra_value,
-diff --git a/src/providers/ldap/sdap_async_initgroups.c b/src/providers/ldap/sdap_async_initgroups.c
-index 97be594a3..fb3d8fe24 100644
---- a/src/providers/ldap/sdap_async_initgroups.c
-+++ b/src/providers/ldap/sdap_async_initgroups.c
-@@ -2732,6 +2732,7 @@ struct tevent_req *sdap_get_initgr_send(TALLOC_CTX *memctx,
-                                         struct sdap_handle *sh,
-                                         struct sdap_id_ctx *id_ctx,
-                                         struct sdap_id_conn_ctx *conn,
-+                                        struct sdap_search_base **search_bases,
-                                         const char *filter_value,
-                                         int filter_type,
-                                         const char *extra_value,
-@@ -2764,7 +2765,8 @@ struct tevent_req *sdap_get_initgr_send(TALLOC_CTX *memctx,
-     state->orig_user = NULL;
-     state->timeout = dp_opt_get_int(state->opts->basic, SDAP_SEARCH_TIMEOUT);
-     state->user_base_iter = 0;
--    state->user_search_bases = sdom->user_search_bases;
-+    state->user_search_bases = (search_bases == NULL) ? sdom->user_search_bases
-+                                                      : search_bases;
-     if (!state->user_search_bases) {
-         DEBUG(SSSDBG_CRIT_FAILURE,
-               "Initgroups lookup request without a user search base\n");
--- 
-2.41.0
-

SOURCES/0003-SSS_CLIENT-if-poll-returns-POLLNVAL-then-socket-is-a.patch

@@ -0,0 +1,63 @@
+From a40b25a3af29706c058ce5a02dd0ba294dbb6874 Mon Sep 17 00:00:00 2001
+From: Alexey Tikhonov <atikhono@redhat.com>
+Date: Wed, 8 Feb 2023 17:48:52 +0100
+Subject: [PATCH 3/4] SSS_CLIENT: if poll() returns POLLNVAL then socket is
+ alredy closed (or wasn't open) so it shouldn't be closed again. Otherwise
+ there is a risk to close "foreign" socket opened in another thread.
+MIME-Version: 1.0
+Content-Type: text/plain; charset=UTF-8
+Content-Transfer-Encoding: 8bit
+
+Reviewed-by: Pavel Březina <pbrezina@redhat.com>
+Reviewed-by: Sumit Bose <sbose@redhat.com>
+(cherry picked from commit ef93284b5a1f196425d9a61e8e24de8972240eb3)
+---
+ src/sss_client/common.c | 18 +++++++++++++++---
+ 1 file changed, 15 insertions(+), 3 deletions(-)
+
+diff --git a/src/sss_client/common.c b/src/sss_client/common.c
+index 27e09f6f3..c8ade645b 100644
+--- a/src/sss_client/common.c
++++ b/src/sss_client/common.c
+@@ -159,7 +159,11 @@ static enum sss_status sss_cli_send_req(enum sss_cli_command cmd,
+             *errnop = ETIME;
+             break;
+         case 1:
+-            if (pfd.revents & (POLLERR | POLLHUP | POLLNVAL)) {
++            if (pfd.revents & (POLLERR | POLLHUP)) {
++                *errnop = EPIPE;
++            } else if (pfd.revents & POLLNVAL) {
++                /* Invalid request: fd is not opened */
++                sss_cli_sd = -1;
+                 *errnop = EPIPE;
+             } else if (!(pfd.revents & POLLOUT)) {
+                 *errnop = EBUSY;
+@@ -270,7 +274,11 @@ static enum sss_status sss_cli_recv_rep(enum sss_cli_command cmd,
+             if (pfd.revents & (POLLHUP)) {
+                 pollhup = true;
+             }
+-            if (pfd.revents & (POLLERR | POLLNVAL)) {
++            if (pfd.revents & POLLERR) {
++                *errnop = EPIPE;
++            } else if (pfd.revents & POLLNVAL) {
++                /* Invalid request: fd is not opened */
++                sss_cli_sd = -1;
+                 *errnop = EPIPE;
+             } else if (!(pfd.revents & POLLIN)) {
+                 *errnop = EBUSY;
+@@ -721,7 +729,11 @@ static enum sss_status sss_cli_check_socket(int *errnop,
+             *errnop = ETIME;
+             break;
+         case 1:
+-            if (pfd.revents & (POLLERR | POLLHUP | POLLNVAL)) {
++            if (pfd.revents & (POLLERR | POLLHUP)) {
++                *errnop = EPIPE;
++            } else if (pfd.revents & POLLNVAL) {
++                /* Invalid request: fd is not opened */
++                sss_cli_sd = -1;
+                 *errnop = EPIPE;
+             } else if (!(pfd.revents & (POLLIN | POLLOUT))) {
+                 *errnop = EBUSY;
+-- 
+2.37.3
+

SOURCES/0003-sdap-add-naming_context-as-new-member-of-struct-sdap.patch

@@ -1,194 +0,0 @@
-From 6a8e60df84d5d2565bec36be19c2def25a6ece1f Mon Sep 17 00:00:00 2001
-From: Sumit Bose <sbose@redhat.com>
-Date: Wed, 24 Jan 2024 14:21:12 +0100
-Subject: [PATCH 3/3] sdap: add naming_context as new member of struct
- sdap_domain
-MIME-Version: 1.0
-Content-Type: text/plain; charset=UTF-8
-Content-Transfer-Encoding: 8bit
-
-The naming_context could be a more reliable source than basedn for the
-actual base DN because basedn is set very early from the domain name
-given in sssd.conf. Although it is recommended to use the fully
-qualified DNS domain name here it is not required. As a result basedn
-might not reflect the actual based DN of the LDAP server. Also pure LDAP
-server (i.e. not AD or FreeIPA) might use different schemes to set the
-base DN which will not be based on the DNS domain of the LDAP server.
-
-Resolves: https://github.com/SSSD/sssd/issues/5708
-
-Reviewed-by: Alejandro López <allopez@redhat.com>
-Reviewed-by: Tomáš Halman <thalman@redhat.com>
-(cherry picked from commit a153f13f296401247a862df2b99048bb1bbb8e2e)
----
- src/providers/ad/ad_gpo.c |  6 ++++--
- src/providers/ldap/sdap.c | 36 +++++++++++++-----------------------
- src/providers/ldap/sdap.h | 11 +++++++++++
- 3 files changed, 28 insertions(+), 25 deletions(-)
-
-diff --git a/src/providers/ad/ad_gpo.c b/src/providers/ad/ad_gpo.c
-index b0ee3e616..3d1ad39c7 100644
---- a/src/providers/ad/ad_gpo.c
-+++ b/src/providers/ad/ad_gpo.c
-@@ -2185,8 +2185,10 @@ ad_gpo_connect_done(struct tevent_req *subreq)
-         goto done;
-     }
- 
--    ret = common_parse_search_base(state, sdom->basedn, state->ldb_ctx,
--                                   "AD_HOSTS", NULL, &search_bases);
-+    ret = common_parse_search_base(state,
-+                                   sdom->naming_context == NULL ? sdom->basedn
-+                                                                : sdom->naming_context,
-+                                   state->ldb_ctx, "AD_HOSTS", NULL, &search_bases);
-     if (ret != EOK) {
-         DEBUG(SSSDBG_OP_FAILURE,
-               "Failed to create dedicated search base for host lookups, "
-diff --git a/src/providers/ldap/sdap.c b/src/providers/ldap/sdap.c
-index f5637c5fb..956eba93a 100644
---- a/src/providers/ldap/sdap.c
-+++ b/src/providers/ldap/sdap.c
-@@ -1252,19 +1252,10 @@ errno_t sdap_set_config_options_with_rootdse(struct sysdb_attrs *rootdse,
-                                              struct sdap_domain *sdom)
- {
-     int ret;
--    char *naming_context = NULL;
- 
--    if (!sdom->search_bases
--            || !sdom->user_search_bases
--            || !sdom->group_search_bases
--            || !sdom->netgroup_search_bases
--            || !sdom->host_search_bases
--            || !sdom->sudo_search_bases
--            || !sdom->iphost_search_bases
--            || !sdom->ipnetwork_search_bases
--            || !sdom->autofs_search_bases) {
--        naming_context = get_naming_context(opts->basic, rootdse);
--        if (naming_context == NULL) {
-+    if (!sdom->naming_context) {
-+        sdom->naming_context = get_naming_context(sdom, rootdse);
-+        if (sdom->naming_context == NULL) {
-             DEBUG(SSSDBG_CRIT_FAILURE, "get_naming_context failed.\n");
- 
-             /* This has to be non-fatal, since some servers offer
-@@ -1280,7 +1271,7 @@ errno_t sdap_set_config_options_with_rootdse(struct sysdb_attrs *rootdse,
-     if (!sdom->search_bases) {
-         ret = sdap_set_search_base(opts, sdom,
-                                    SDAP_SEARCH_BASE,
--                                   naming_context);
-+                                   sdom->naming_context);
-         if (ret != EOK) goto done;
-     }
- 
-@@ -1288,7 +1279,7 @@ errno_t sdap_set_config_options_with_rootdse(struct sysdb_attrs *rootdse,
-     if (!sdom->user_search_bases) {
-         ret = sdap_set_search_base(opts, sdom,
-                                    SDAP_USER_SEARCH_BASE,
--                                   naming_context);
-+                                   sdom->naming_context);
-         if (ret != EOK) goto done;
-     }
- 
-@@ -1296,7 +1287,7 @@ errno_t sdap_set_config_options_with_rootdse(struct sysdb_attrs *rootdse,
-     if (!sdom->group_search_bases) {
-         ret = sdap_set_search_base(opts, sdom,
-                                    SDAP_GROUP_SEARCH_BASE,
--                                   naming_context);
-+                                   sdom->naming_context);
-         if (ret != EOK) goto done;
-     }
- 
-@@ -1304,7 +1295,7 @@ errno_t sdap_set_config_options_with_rootdse(struct sysdb_attrs *rootdse,
-     if (!sdom->netgroup_search_bases) {
-         ret = sdap_set_search_base(opts, sdom,
-                                    SDAP_NETGROUP_SEARCH_BASE,
--                                   naming_context);
-+                                   sdom->naming_context);
-         if (ret != EOK) goto done;
-     }
- 
-@@ -1312,7 +1303,7 @@ errno_t sdap_set_config_options_with_rootdse(struct sysdb_attrs *rootdse,
-     if (!sdom->host_search_bases) {
-         ret = sdap_set_search_base(opts, sdom,
-                                    SDAP_HOST_SEARCH_BASE,
--                                   naming_context);
-+                                   sdom->naming_context);
-         if (ret != EOK) goto done;
-     }
- 
-@@ -1320,7 +1311,7 @@ errno_t sdap_set_config_options_with_rootdse(struct sysdb_attrs *rootdse,
-     if (!sdom->sudo_search_bases) {
-        ret = sdap_set_search_base(opts, sdom,
-                                    SDAP_SUDO_SEARCH_BASE,
--                                   naming_context);
-+                                   sdom->naming_context);
-         if (ret != EOK) goto done;
-     }
- 
-@@ -1328,7 +1319,7 @@ errno_t sdap_set_config_options_with_rootdse(struct sysdb_attrs *rootdse,
-     if (!sdom->service_search_bases) {
-        ret = sdap_set_search_base(opts, sdom,
-                                   SDAP_SERVICE_SEARCH_BASE,
--                                  naming_context);
-+                                  sdom->naming_context);
-         if (ret != EOK) goto done;
-     }
- 
-@@ -1336,7 +1327,7 @@ errno_t sdap_set_config_options_with_rootdse(struct sysdb_attrs *rootdse,
-     if (!sdom->autofs_search_bases) {
-        ret = sdap_set_search_base(opts, sdom,
-                                   SDAP_AUTOFS_SEARCH_BASE,
--                                  naming_context);
-+                                  sdom->naming_context);
-         if (ret != EOK) goto done;
-     }
- 
-@@ -1344,7 +1335,7 @@ errno_t sdap_set_config_options_with_rootdse(struct sysdb_attrs *rootdse,
-     if (!sdom->iphost_search_bases) {
-         ret = sdap_set_search_base(opts, sdom,
-                                    SDAP_IPHOST_SEARCH_BASE,
--                                   naming_context);
-+                                   sdom->naming_context);
-         if (ret != EOK) goto done;
-     }
- 
-@@ -1352,14 +1343,13 @@ errno_t sdap_set_config_options_with_rootdse(struct sysdb_attrs *rootdse,
-     if (!sdom->ipnetwork_search_bases) {
-         ret = sdap_set_search_base(opts, sdom,
-                                    SDAP_IPNETWORK_SEARCH_BASE,
--                                   naming_context);
-+                                   sdom->naming_context);
-         if (ret != EOK) goto done;
-     }
- 
-     ret = EOK;
- 
- done:
--    talloc_free(naming_context);
-     return ret;
- }
- 
-diff --git a/src/providers/ldap/sdap.h b/src/providers/ldap/sdap.h
-index 161bc5c26..103d50ed4 100644
---- a/src/providers/ldap/sdap.h
-+++ b/src/providers/ldap/sdap.h
-@@ -454,6 +454,17 @@ struct sdap_domain {
- 
-     char *basedn;
- 
-+    /* The naming_context could be a more reliable source than basedn for the
-+     * actual base DN because basedn is set very early from the domain name
-+     * given in sssd.conf. Although it is recommended to use the fully
-+     * qualified DNS domain name here it is not required. As a result basedn
-+     * might not reflect the actual based DN of the LDAP server. Also pure
-+     * LDAP server (i.e. not AD or FreeIPA) might use different schemes to set
-+     * the base DN which will not be based on the DNS domain of the LDAP
-+     * server. naming_context might be NULL even after connection to an LDAP
-+     * server. */
-+    char *naming_context;
-+
-     struct sdap_search_base **search_bases;
-     struct sdap_search_base **user_search_bases;
-     struct sdap_search_base **group_search_bases;
--- 
-2.41.0
-

SOURCES/0004-PAM_SSS-close-sss_cli_sd-should-also-be-protected-wi.patch

@@ -0,0 +1,53 @@
+From 1fd7a5ecb46a02a29ebf42039575b5344307bfbb Mon Sep 17 00:00:00 2001
+From: Alexey Tikhonov <atikhono@redhat.com>
+Date: Wed, 8 Feb 2023 18:58:37 +0100
+Subject: [PATCH 4/4] PAM_SSS: close(sss_cli_sd) should also be protected with
+ mutex. Otherwise a thread calling pam_end() can close socket mid pam
+ transaction in another thread.
+MIME-Version: 1.0
+Content-Type: text/plain; charset=UTF-8
+Content-Transfer-Encoding: 8bit
+
+Bug only manifested on platforms where "lockfree client"
+feature wasn't built.
+
+Reviewed-by: Pavel Březina <pbrezina@redhat.com>
+Reviewed-by: Sumit Bose <sbose@redhat.com>
+(cherry picked from commit bf3f73ea0ee123fe4e7c4bdd2287ac5a5e6d9082)
+---
+ src/sss_client/pam_sss.c     | 3 +++
+ src/sss_client/pam_sss_gss.c | 2 ++
+ 2 files changed, 5 insertions(+)
+
+diff --git a/src/sss_client/pam_sss.c b/src/sss_client/pam_sss.c
+index afbdef59a..39ad17188 100644
+--- a/src/sss_client/pam_sss.c
++++ b/src/sss_client/pam_sss.c
+@@ -117,7 +117,10 @@ static void close_fd(pam_handle_t *pamh, void *ptr, int err)
+ #endif /* PAM_DATA_REPLACE */
+ 
+     D(("Closing the fd"));
++
++    sss_pam_lock();
+     sss_cli_close_socket();
++    sss_pam_unlock();
+ }
+ 
+ struct cert_auth_info {
+diff --git a/src/sss_client/pam_sss_gss.c b/src/sss_client/pam_sss_gss.c
+index 1109ec570..dd578ae5d 100644
+--- a/src/sss_client/pam_sss_gss.c
++++ b/src/sss_client/pam_sss_gss.c
+@@ -581,7 +581,9 @@ int pam_sm_authenticate(pam_handle_t *pamh,
+     }
+ 
+ done:
++    sss_pam_lock();
+     sss_cli_close_socket();
++    sss_pam_unlock();
+     free(username);
+     free(domain);
+     free(target);
+-- 
+2.37.3
+

SOURCES/0004-pam-fix-SC-auth-with-multiple-certs-and-missing-logi.patch

@@ -1,233 +0,0 @@
-From 50077c3255177fe1b01837fbe31a7f8fd47dee74 Mon Sep 17 00:00:00 2001
-From: Sumit Bose <sbose@redhat.com>
-Date: Thu, 18 Jan 2024 13:08:17 +0100
-Subject: [PATCH] pam: fix SC auth with multiple certs and missing login name
-
-While introducing the local_auth_policy option a quite specific use-case
-was not covered correctly. If there are multiple matching certificates
-on the Smartcard, 'local_auth_policy = only' is set and GDM's Smartcard
-mode was used for login, i.e. there is no user name given and the user
-has to be derived from the certificate used for login, authentication
-failed. The main reason for the failure is that in this case the
-Smartcard interaction and the user mapping has to be done first to
-determine the user before local_auth_policy is evaluated. As a result
-when checking if the authentication can be finished the request was in
-an unexpected state because the indicator for local Smartcard
-authentication was not enabled.
-
-Resolves: https://github.com/SSSD/sssd/issues/7109
-
-Reviewed-by: Justin Stephenson <jstephen@redhat.com>
-Reviewed-by: Scott Poore <spoore@redhat.com>
-(cherry picked from commit 44ec3e4638b0c6f7f45a3390a28c2e8745d52bc3)
----
- src/responder/pam/pamsrv.h           | 10 ++++
- src/responder/pam/pamsrv_cmd.c       | 17 +++++--
- src/tests/intg/Makefile.am           |  2 +
- src/tests/intg/test_pam_responder.py | 74 +++++++++++++++++++++++++++-
- 4 files changed, 96 insertions(+), 7 deletions(-)
-
-diff --git a/src/responder/pam/pamsrv.h b/src/responder/pam/pamsrv.h
-index 7013a8edd..618836189 100644
---- a/src/responder/pam/pamsrv.h
-+++ b/src/responder/pam/pamsrv.h
-@@ -93,7 +93,17 @@ struct pam_auth_req {
-     struct ldb_message *user_obj;
-     struct cert_auth_info *cert_list;
-     struct cert_auth_info *current_cert;
-+    /* Switched to 'true' if the backend indicates that it cannot handle
-+     * Smartcard authentication, but Smartcard authentication is
-+     * possible and local Smartcard authentication is allowed. */
-     bool cert_auth_local;
-+    /* Switched to 'true' if authentication (not pre-authentication) was
-+     * started without a login name and the name had to be lookup up with the
-+     * certificate used for authentication. Since reading the certificate from
-+     * the Smartcard already involves the PIN validation in this case there
-+     * would be no need for an additional Smartcard interaction if only local
-+     * Smartcard authentication is possible. */
-+    bool initial_cert_auth_successful;
- 
-     bool passkey_data_exists;
-     uint32_t client_id_num;
-diff --git a/src/responder/pam/pamsrv_cmd.c b/src/responder/pam/pamsrv_cmd.c
-index c23ea7ba4..a7c181733 100644
---- a/src/responder/pam/pamsrv_cmd.c
-+++ b/src/responder/pam/pamsrv_cmd.c
-@@ -2200,8 +2200,8 @@ static void pam_forwarder_lookup_by_cert_done(struct tevent_req *req)
-                 ret = ENOENT;
-                 goto done;
-             }
--
--            if (cert_count > 1) {
-+            /* Multiple certificates are only expected during pre-auth */
-+            if (cert_count > 1 && preq->pd->cmd == SSS_PAM_PREAUTH) {
-                 for (preq->current_cert = preq->cert_list;
-                      preq->current_cert != NULL;
-                      preq->current_cert = sss_cai_get_next(preq->current_cert)) {
-@@ -2285,7 +2285,9 @@ static void pam_forwarder_lookup_by_cert_done(struct tevent_req *req)
-             }
- 
-             /* If logon_name was not given during authentication add a
--             * SSS_PAM_CERT_INFO message to send the name to the caller. */
-+             * SSS_PAM_CERT_INFO message to send the name to the caller.
-+             * Additionally initial_cert_auth_successful is set to
-+             * indicate that the user is already authenticated. */
-             if (preq->pd->cmd == SSS_PAM_AUTHENTICATE
-                     && preq->pd->logon_name == NULL) {
-                 ret = add_pam_cert_response(preq->pd,
-@@ -2297,6 +2299,8 @@ static void pam_forwarder_lookup_by_cert_done(struct tevent_req *req)
-                     preq->pd->pam_status = PAM_AUTHINFO_UNAVAIL;
-                     goto done;
-                 }
-+
-+                preq->initial_cert_auth_successful = true;
-             }
- 
-             /* cert_user will be returned to the PAM client as user name, so
-@@ -2851,12 +2855,15 @@ static void pam_dom_forwarder(struct pam_auth_req *preq)
-         if (found) {
-             if (local_policy != NULL && strcasecmp(local_policy, "only") == 0) {
-                 talloc_free(tmp_ctx);
--                DEBUG(SSSDBG_IMPORTANT_INFO, "Local auth only set, skipping online auth\n");
-+                DEBUG(SSSDBG_IMPORTANT_INFO,
-+                      "Local auth only set and matching certificate was found, "
-+                      "skipping online auth\n");
-                 if (preq->pd->cmd == SSS_PAM_PREAUTH) {
-                     preq->pd->pam_status = PAM_SUCCESS;
-                 } else if (preq->pd->cmd == SSS_PAM_AUTHENTICATE
-                                 && IS_SC_AUTHTOK(preq->pd->authtok)
--                                && preq->cert_auth_local) {
-+                                && (preq->cert_auth_local
-+                                        || preq->initial_cert_auth_successful)) {
-                     preq->pd->pam_status = PAM_SUCCESS;
-                     preq->callback = pam_reply;
-                 }
-diff --git a/src/tests/intg/Makefile.am b/src/tests/intg/Makefile.am
-index 3866d3ca6..0cfd268dc 100644
---- a/src/tests/intg/Makefile.am
-+++ b/src/tests/intg/Makefile.am
-@@ -199,6 +199,7 @@ clean-local:
- 
- PAM_CERT_DB_PATH="$(abs_builddir)/../test_CA/SSSD_test_CA.pem"
- SOFTHSM2_CONF="$(abs_builddir)/../test_CA/softhsm2_one.conf"
-+SOFTHSM2_TWO_CONF="$(abs_builddir)/../test_CA/softhsm2_two.conf"
- 
- intgcheck-installed: config.py passwd group pam_sss_service pam_sss_alt_service pam_sss_sc_required pam_sss_try_sc pam_sss_allow_missing_name pam_sss_domains sss_netgroup_thread_test
- 	pipepath="$(DESTDIR)$(pipepath)"; \
-@@ -233,6 +234,7 @@ intgcheck-installed: config.py passwd group pam_sss_service pam_sss_alt_service
- 	PAM_CERT_DB_PATH=$(PAM_CERT_DB_PATH) \
- 	ABS_SRCDIR=$(abs_srcdir) \
- 	SOFTHSM2_CONF=$(SOFTHSM2_CONF) \
-+	SOFTHSM2_TWO_CONF=$(SOFTHSM2_TWO_CONF) \
- 	KCM_RENEW=$(KCM_RENEW) \
- 	FILES_PROVIDER=$(FILES_PROVIDER) \
- 	DBUS_SOCK_DIR="$(DESTDIR)$(runstatedir)/dbus/" \
-diff --git a/src/tests/intg/test_pam_responder.py b/src/tests/intg/test_pam_responder.py
-index 1fc3937e6..0fbf8065e 100644
---- a/src/tests/intg/test_pam_responder.py
-+++ b/src/tests/intg/test_pam_responder.py
-@@ -168,7 +168,7 @@ def format_pam_cert_auth_conf(config, provider):
-         {provider.p}
- 
-         [certmap/auth_only/user1]
--        matchrule = <SUBJECT>.*CN=SSSD test cert 0001.*
-+        matchrule = <SUBJECT>.*CN=SSSD test cert 000[12].*
-     """).format(**locals())
- 
- 
-@@ -201,7 +201,7 @@ def format_pam_cert_auth_conf_name_format(config, provider):
-         {provider.p}
- 
-         [certmap/auth_only/user1]
--        matchrule = <SUBJECT>.*CN=SSSD test cert 0001.*
-+        matchrule = <SUBJECT>.*CN=SSSD test cert 000[12].*
-     """).format(**locals())
- 
- 
-@@ -380,6 +380,28 @@ def simple_pam_cert_auth_no_cert(request, passwd_ops_setup):
-     return None
- 
- 
-+@pytest.fixture
-+def simple_pam_cert_auth_two_certs(request, passwd_ops_setup):
-+    """Setup SSSD with pam_cert_auth=True"""
-+    config.PAM_CERT_DB_PATH = os.environ['PAM_CERT_DB_PATH']
-+
-+    old_softhsm2_conf = os.environ['SOFTHSM2_CONF']
-+    softhsm2_two_conf = os.environ['SOFTHSM2_TWO_CONF']
-+    os.environ['SOFTHSM2_CONF'] = softhsm2_two_conf
-+
-+    conf = format_pam_cert_auth_conf(config, provider_switch(request.param))
-+    create_conf_fixture(request, conf)
-+    create_sssd_fixture(request)
-+
-+    os.environ['SOFTHSM2_CONF'] = old_softhsm2_conf
-+
-+    passwd_ops_setup.useradd(**USER1)
-+    passwd_ops_setup.useradd(**USER2)
-+    sync_files_provider(USER2['name'])
-+
-+    return None
-+
-+
- @pytest.fixture
- def simple_pam_cert_auth_name_format(request, passwd_ops_setup):
-     """Setup SSSD with pam_cert_auth=True and full_name_format"""
-@@ -522,6 +544,54 @@ def test_sc_auth(simple_pam_cert_auth, env_for_sssctl):
-     assert err.find("pam_authenticate for user [user1]: Success") != -1
- 
- 
-+@pytest.mark.parametrize('simple_pam_cert_auth_two_certs', provider_list(), indirect=True)
-+def test_sc_auth_two(simple_pam_cert_auth_two_certs, env_for_sssctl):
-+
-+    sssctl = subprocess.Popen(["sssctl", "user-checks", "user1",
-+                               "--action=auth", "--service=pam_sss_service"],
-+                              universal_newlines=True,
-+                              env=env_for_sssctl, stdin=subprocess.PIPE,
-+                              stdout=subprocess.PIPE, stderr=subprocess.PIPE)
-+
-+    try:
-+        out, err = sssctl.communicate(input="2\n123456")
-+    except Exception:
-+        sssctl.kill()
-+        out, err = sssctl.communicate()
-+
-+    sssctl.stdin.close()
-+    sssctl.stdout.close()
-+
-+    if sssctl.wait() != 0:
-+        raise Exception("sssctl failed")
-+
-+    assert err.find("pam_authenticate for user [user1]: Success") != -1
-+
-+
-+@pytest.mark.parametrize('simple_pam_cert_auth_two_certs', provider_list(), indirect=True)
-+def test_sc_auth_two_missing_name(simple_pam_cert_auth_two_certs, env_for_sssctl):
-+
-+    sssctl = subprocess.Popen(["sssctl", "user-checks", "",
-+                               "--action=auth", "--service=pam_sss_allow_missing_name"],
-+                              universal_newlines=True,
-+                              env=env_for_sssctl, stdin=subprocess.PIPE,
-+                              stdout=subprocess.PIPE, stderr=subprocess.PIPE)
-+
-+    try:
-+        out, err = sssctl.communicate(input="2\n123456")
-+    except Exception:
-+        sssctl.kill()
-+        out, err = sssctl.communicate()
-+
-+    sssctl.stdin.close()
-+    sssctl.stdout.close()
-+
-+    if sssctl.wait() != 0:
-+        raise Exception("sssctl failed")
-+
-+    assert err.find("pam_authenticate for user [user1]: Success") != -1
-+
-+
- @pytest.mark.parametrize('simple_pam_cert_auth', ['proxy_password'], indirect=True)
- def test_sc_proxy_password_fallback(simple_pam_cert_auth, env_for_sssctl):
-     """
--- 
-2.41.0
-

SOURCES/0005-ad-gpo-use-hash-to-store-intermediate-results.patch

@@ -1,218 +0,0 @@
-From e1bfbc2493c4194988acc3b2413df3dde0735ae3 Mon Sep 17 00:00:00 2001
-From: Sumit Bose <sbose@redhat.com>
-Date: Wed, 8 Nov 2023 14:50:24 +0100
-Subject: [PATCH] ad-gpo: use hash to store intermediate results
-MIME-Version: 1.0
-Content-Type: text/plain; charset=UTF-8
-Content-Transfer-Encoding: 8bit
-
-Currently after the evaluation of a single GPO file the intermediate
-results are stored in the cache and this cache entry is updated until
-all applicable GPO files are evaluated. Finally the data in the cache is
-used to make the decision of access is granted or rejected.
-
-If there are two or more access-control request running in parallel one
-request might overwrite the cache object with intermediate data while
-another request reads the cached data for the access decision and as a
-result will do this decision based on intermediate data.
-
-To avoid this the intermediate results are not stored in the cache
-anymore but in hash tables which are specific to the request. Only the
-final result is written to the cache to have it available for offline
-authentication.
-
-Reviewed-by: Alexey Tikhonov <atikhono@redhat.com>
-Reviewed-by: Tomáš Halman <thalman@redhat.com>
-(cherry picked from commit d7db7971682da2dbf7642ac94940d6b0577ec35a)
----
- src/providers/ad/ad_gpo.c | 116 +++++++++++++++++++++++++++++++++-----
- 1 file changed, 102 insertions(+), 14 deletions(-)
-
-diff --git a/src/providers/ad/ad_gpo.c b/src/providers/ad/ad_gpo.c
-index 3d1ad39c7..b879b0a08 100644
---- a/src/providers/ad/ad_gpo.c
-+++ b/src/providers/ad/ad_gpo.c
-@@ -1431,6 +1431,33 @@ ad_gpo_extract_policy_setting(TALLOC_CTX *mem_ctx,
-     return ret;
- }
- 
-+static errno_t
-+add_result_to_hash(hash_table_t *hash, const char *key, char *value)
-+{
-+    int hret;
-+    hash_key_t k;
-+    hash_value_t v;
-+
-+    if (hash == NULL || key == NULL || value == NULL) {
-+        return EINVAL;
-+    }
-+
-+    k.type = HASH_KEY_CONST_STRING;
-+    k.c_str = key;
-+
-+    v.type = HASH_VALUE_PTR;
-+    v.ptr = value;
-+
-+    hret = hash_enter(hash, &k, &v);
-+    if (hret != HASH_SUCCESS) {
-+        DEBUG(SSSDBG_OP_FAILURE, "Failed to add [%s][%s] to hash: [%s].\n",
-+                                 key, value, hash_error_string(hret));
-+        return EIO;
-+    }
-+
-+    return EOK;
-+}
-+
- /*
-  * This function parses the cse-specific (GP_EXT_GUID_SECURITY) filename,
-  * and stores the allow_key and deny_key of all of the gpo_map_types present
-@@ -1438,6 +1465,7 @@ ad_gpo_extract_policy_setting(TALLOC_CTX *mem_ctx,
-  */
- static errno_t
- ad_gpo_store_policy_settings(struct sss_domain_info *domain,
-+                             hash_table_t *allow_maps, hash_table_t *deny_maps,
-                              const char *filename)
- {
-     struct ini_cfgfile *file_ctx = NULL;
-@@ -1571,14 +1599,14 @@ ad_gpo_store_policy_settings(struct sss_domain_info *domain,
-                 goto done;
-             } else if (ret != ENOENT) {
-                 const char *value = allow_value ? allow_value : empty_val;
--                ret = sysdb_gpo_store_gpo_result_setting(domain,
--                                                         allow_key,
--                                                         value);
-+                ret = add_result_to_hash(allow_maps, allow_key,
-+                                         talloc_strdup(allow_maps, value));
-                 if (ret != EOK) {
--                    DEBUG(SSSDBG_CRIT_FAILURE,
--                          "sysdb_gpo_store_gpo_result_setting failed for key:"
--                          "'%s' value:'%s' [%d][%s]\n", allow_key, allow_value,
--                          ret, sss_strerror(ret));
-+                    DEBUG(SSSDBG_CRIT_FAILURE, "Failed to add key: [%s] "
-+                                               "value: [%s] to allow maps "
-+                                               "[%d][%s].\n",
-+                                               allow_key, value, ret,
-+                                               sss_strerror(ret));
-                     goto done;
-                 }
-             }
-@@ -1598,14 +1626,14 @@ ad_gpo_store_policy_settings(struct sss_domain_info *domain,
-                 goto done;
-             } else if (ret != ENOENT) {
-                 const char *value = deny_value ? deny_value : empty_val;
--                ret = sysdb_gpo_store_gpo_result_setting(domain,
--                                                         deny_key,
--                                                         value);
-+                ret = add_result_to_hash(deny_maps, deny_key,
-+                                         talloc_strdup(deny_maps, value));
-                 if (ret != EOK) {
--                    DEBUG(SSSDBG_CRIT_FAILURE,
--                          "sysdb_gpo_store_gpo_result_setting failed for key:"
--                          "'%s' value:'%s' [%d][%s]\n", deny_key, deny_value,
--                          ret, sss_strerror(ret));
-+                    DEBUG(SSSDBG_CRIT_FAILURE, "Failed to add key: [%s] "
-+                                               "value: [%s] to deny maps "
-+                                               "[%d][%s].\n",
-+                                               deny_key, value, ret,
-+                                               sss_strerror(ret));
-                     goto done;
-                 }
-             }
-@@ -1902,6 +1930,8 @@ struct ad_gpo_access_state {
-     int num_cse_filtered_gpos;
-     int cse_gpo_index;
-     const char *ad_domain;
-+    hash_table_t *allow_maps;
-+    hash_table_t *deny_maps;
- };
- 
- static void ad_gpo_connect_done(struct tevent_req *subreq);
-@@ -2023,6 +2053,19 @@ ad_gpo_access_send(TALLOC_CTX *mem_ctx,
-         goto immediately;
-     }
- 
-+    ret = sss_hash_create(state, 0, &state->allow_maps);
-+    if (ret != EOK) {
-+        DEBUG(SSSDBG_FATAL_FAILURE, "Could not create allow maps "
-+              "hash table [%d]: %s\n", ret, sss_strerror(ret));
-+        goto immediately;
-+    }
-+
-+    ret = sss_hash_create(state, 0, &state->deny_maps);
-+    if (ret != EOK) {
-+        DEBUG(SSSDBG_FATAL_FAILURE, "Could not create deny maps "
-+              "hash table [%d]: %s\n", ret, sss_strerror(ret));
-+        goto immediately;
-+    }
- 
-     subreq = sdap_id_op_connect_send(state->sdap_op, state, &ret);
-     if (subreq == NULL) {
-@@ -2713,6 +2756,43 @@ ad_gpo_cse_step(struct tevent_req *req)
-     return EAGAIN;
- }
- 
-+static errno_t
-+store_hash_maps_in_cache(struct sss_domain_info *domain,
-+                         hash_table_t *allow_maps, hash_table_t *deny_maps)
-+{
-+    int ret;
-+    struct hash_iter_context_t *iter;
-+    hash_entry_t *entry;
-+    size_t c;
-+    hash_table_t *hash_list[] = { allow_maps, deny_maps, NULL};
-+
-+
-+    for (c = 0; hash_list[c] != NULL; c++) {
-+        iter = new_hash_iter_context(hash_list[c]);
-+        if (iter == NULL) {
-+            DEBUG(SSSDBG_OP_FAILURE, "Failed to create hash iterator.\n");
-+            return EINVAL;
-+        }
-+
-+        while ((entry = iter->next(iter)) != NULL) {
-+            ret = sysdb_gpo_store_gpo_result_setting(domain,
-+                                                     entry->key.c_str,
-+                                                     entry->value.ptr);
-+            if (ret != EOK) {
-+                free(iter);
-+                DEBUG(SSSDBG_OP_FAILURE,
-+                      "sysdb_gpo_store_gpo_result_setting failed for key:"
-+                      "[%s] value:[%s] [%d][%s]\n", entry->key.c_str,
-+                      (char *) entry->value.ptr, ret, sss_strerror(ret));
-+                return ret;
-+            }
-+        }
-+        talloc_free(iter);
-+    }
-+
-+    return EOK;
-+}
-+
- /*
-  * This cse-specific function (GP_EXT_GUID_SECURITY) increments the
-  * cse_gpo_index until the policy settings for all applicable GPOs have been
-@@ -2754,6 +2834,7 @@ ad_gpo_cse_done(struct tevent_req *subreq)
-      * (as part of the GPO Result object in the sysdb cache).
-      */
-     ret = ad_gpo_store_policy_settings(state->host_domain,
-+                                       state->allow_maps, state->deny_maps,
-                                        cse_filtered_gpo->policy_filename);
-     if (ret != EOK && ret != ENOENT) {
-         DEBUG(SSSDBG_OP_FAILURE,
-@@ -2767,6 +2848,13 @@ ad_gpo_cse_done(struct tevent_req *subreq)
- 
-     if (ret == EOK) {
-         /* ret is EOK only after all GPO policy files have been downloaded */
-+        ret = store_hash_maps_in_cache(state->host_domain,
-+                                       state->allow_maps, state->deny_maps);
-+        if (ret != EOK) {
-+            DEBUG(SSSDBG_OP_FAILURE, "Failed to store evaluated GPO maps "
-+                                     "[%d][%s].\n", ret, sss_strerror(ret));
-+            goto done;
-+        }
-         ret = ad_gpo_perform_hbac_processing(state,
-                                              state->gpo_mode,
-                                              state->gpo_map_type,
--- 
-2.44.0
-

SPECS/sssd.spec

@@ -18,8 +18,8 @@
 %global enable_systemtap_opt --enable-systemtap
 
 Name: sssd
-Version: 2.9.4
-Release: 3%{?dist}
+Version: 2.8.2
+Release: 2%{?dist}
 Group: Applications/System
 Summary: System Security Services Daemon
 License: GPLv3+
@@ -27,11 +27,10 @@ URL: https://github.com/SSSD/sssd
 Source0: https://github.com/SSSD/sssd/releases/download/%{version}/sssd-%{version}.tar.gz
 
 ### Patches ###
-Patch0001: 0001-sssd-adding-mail-as-case-insensitive.patch
-Patch0002: 0002-sdap-add-search_bases-option-to-groups_by_user_send.patch
-Patch0003: 0003-sdap-add-naming_context-as-new-member-of-struct-sdap.patch
-Patch0004: 0004-pam-fix-SC-auth-with-multiple-certs-and-missing-logi.patch
-Patch0005: 0005-ad-gpo-use-hash-to-store-intermediate-results.patch
+Patch0001: 0001-ldap-update-shadow-last-change-in-sysdb-as-well.patch
+Patch0002: 0002-SSS_CLIENT-fix-error-codes-returned-by-common-read-w.patch
+Patch0003: 0003-SSS_CLIENT-if-poll-returns-POLLNVAL-then-socket-is-a.patch
+Patch0004: 0004-PAM_SSS-close-sss_cli_sd-should-also-be-protected-wi.patch
 
 ### Downstream Patches ###
 
@@ -215,6 +214,7 @@ Summary: Userspace tools for use with the SSSD
 Group: Applications/System
 License: GPLv3+
 Requires: sssd-common = %{version}-%{release}
+Requires: libsss_simpleifp = %{version}-%{release}
 # required by sss_obfuscate
 Requires: python3-sss = %{version}-%{release}
 Requires: python3-sssdconfig = %{version}-%{release}
@@ -355,7 +355,6 @@ Group: Applications/System
 License: GPLv3+
 Conflicts: sssd < 1.10.0-8.beta2
 Requires: sssd-common = %{version}-%{release}
-Requires: libsss_certmap = %{version}-%{release}
 Requires(pre): shadow-utils
 
 %description proxy
@@ -595,8 +594,6 @@ autoreconf -ivf
     --with-initscript=systemd \
     --with-syslog=journald \
     --with-subid \
-    --with-files-provider \
-    --with-libsifp \
     --enable-sss-default-nss-plugin \
     --without-python2-bindings \
     --with-sssd-user=sssd \
@@ -913,7 +910,7 @@ done
 %{_mandir}/man5/sssd-ifp.5*
 %{_unitdir}/sssd-ifp.service
 # InfoPipe DBus plumbing
-%{_datadir}/dbus-1/system.d/org.freedesktop.sssd.infopipe.conf
+%{_sysconfdir}/dbus-1/system.d/org.freedesktop.sssd.infopipe.conf
 %{_datadir}/dbus-1/system-services/org.freedesktop.sssd.infopipe.service
 
 %files -n libsss_simpleifp
@@ -1216,70 +1213,6 @@ fi
 %systemd_postun_with_restart sssd.service
 
 %changelog
-* Thu Apr 18 2024 Alexey Tikhonov <atikhono@redhat.com> - 2.9.4-3
-- Resolves: RHEL-27205 - Race condition during authorization leads to GPO policies functioning inconsistently
-
-* Mon Feb 12 2024 Alexey Tikhonov <atikhono@redhat.com> - 2.9.4-2
-- Resolves: RHEL-25064 - AD users are unable to log in due to case sensitivity of user because the domain is found as an alias to the email address. [rhel-8]
-- Resolves: RHEL-25066 - gdm smartcard login fails with sssd-2.9.3 in case of multiple identities [rhel-8]
-- Resolves: RHEL-25065 - ssh pubkey stored in ldap/AD no longer works to authenticate via sssd [rhel-8]
-
-* Sat Jan 13 2024 Alexey Tikhonov <atikhono@redhat.com> - 2.9.4-1
-- Resolves: RHEL-2630 - Rebase SSSD for RHEL 8.10
-- Resolves: RHEL-1680 - auto_private_groups does not create cache in IPA server SSSD cache
-- Resolves: RHEL-10092 - logfile rotation for sssd_kcm not working properly, sssd_kcm never receives a 'kill -HUP'
-- Resolves: RHEL-17495 - New sssd.conf seems not to be backwards compatible (wrt SmartCard auth of local users using 'files provider')
-- Resolves: RHEL-18431 - Excessive logging to sssd_nss and sssd_be in multi-domain AD forest
-- Resolves: RHEL-5033 - Incorrect IdM product name in man sssd.conf
-- Resolves: RHEL-15368 - SSSD GPO lacks group resolution on hosts [rhel-8]
-- Resolves: RHEL-10721 - very bad performance when requesting service tickets
-- Resolves: RHEL-19011 - Invalid handling groups from child domain
-- Resolves: RHEL-19949 - latest sssd breaks logging in via XDMCP for LDAP/Kerberos users [rhel-8]
-
-* Mon Nov 13 2023 Alexey Tikhonov <atikhono@redhat.com> - 2.9.3-2
-- Resolves: RHEL-2630 - Rebase SSSD for RHEL 8.10
-
-* Mon Nov 13 2023 Alexey Tikhonov <atikhono@redhat.com> - 2.9.3-1
-- Resolves: RHEL-2630 - Rebase SSSD for RHEL 8.10
-- Resolves: RHEL-14070 - sssd-2.9.2-1.el8 breaks smart card authentication
-- Resolves: RHEL-3665 - Unexplainable error "Unable to find primary gid [2]: No such file or directory" when SSSD performs lookup for an AD user
-
-* Mon Sep 11 2023 Alexey Tikhonov <atikhono@redhat.com> - 2.9.2-1
-- Resolves: RHEL-2630 - Rebase SSSD for RHEL 8.10
-- Resolves: rhbz#2226021 - dbus and crond getting terminated with SIGBUS in sss_client code
-- Resolves: rhbz#2237253 - SSSD runs multiples lookup search for each NFS request (SBUS req chaining stopped working in sssd-2.7)
-
-* Mon Jul 10 2023 Alexey Tikhonov <atikhono@redhat.com> - 2.9.1-2
-- Resolves: rhbz#2149241 - [sssd] SSSD enters failed state after heavy load in the system
-
-* Fri Jun 23 2023 Alexey Tikhonov <atikhono@redhat.com> - 2.9.1-1
-- Resolves: rhbz#2167836 - Rebase SSSD for RHEL 8.9
-- Resolves: rhbz#2196521 - [RHEL8] sssd : AD user login problem when modify ldap_user_name= name and restricted by GPO Policy
-- Resolves: rhbz#2195919 - sssd-be tends to run out of system resources, hitting the maximum number of open files
-- Resolves: rhbz#2192708 - [RHEL8] [sssd] User lookup on IPA client fails with 's2n get_fqlist request failed'
-- Resolves: rhbz#2139467 - [RHEL8] sssd attempts LDAP password modify extended op after BIND failure
-- Resolves: rhbz#2054825 - sssd_be segfault at 0 ip 00007f16b5fcab7e sp 00007fffc1cc0988 error 4 in libc-2.28.so[7f16b5e72000+1bc000]
-- Resolves: rhbz#2189583 - [sssd] RHEL 8.9 Tier 0 Localization
-- Resolves: rhbz#2170720 - [RHEL8] When adding attributes in sssd.conf that we have already, the cross-forest query just stop working
-- Resolves: rhbz#2096183 - BE_REQ_USER_AND_GROUP LDAP search filter can inadvertently catch multiple overrides
-- Resolves: rhbz#2151450 - [RHEL8] SSSD missing group membership when evaluating GPO policy with 'auto_private_groups = true'
-
-* Tue May 30 2023 Alexey Tikhonov <atikhono@redhat.com> - 2.9.0-4
-- Related: rhbz#2190417 - Rebase Samba to the latest 4.18.x release
-  Rebuild against rebased Samba libs
-
-* Thu May 25 2023 Alexey Tikhonov <atikhono@redhat.com> - 2.9.0-3
-- Resolves: rhbz#2167836 - Rebase SSSD for RHEL 8.9
-
-* Mon May 15 2023 Alexey Tikhonov <atikhono@redhat.com> - 2.9.0-1
-- Resolves: rhbz#2167836 - Rebase SSSD for RHEL 8.9
-- Resolves: rhbz#2101489 - [sssd] Auth fails if client cannot speak to forest root domain (ldap_sasl_interactive_bind_s failed)
-- Resolves: rhbz#2143925 - kinit switches KCM away from the newly issued ticket
-- Resolves: rhbz#2151403 - AD user is not found on IPA client after upgrading to RHEL8.7
-- Resolves: rhbz#2164805 - man page entry should make clear that a nested group needs a name
-- Resolves: rhbz#2170484 - Unable to lookup AD user from child domain (or "make filtering of the domains more configurable")
-- Resolves: rhbz#2180981 - sss allows extraneous @ characters prefixed to username #
-
 * Mon Feb 13 2023 Alexey Tikhonov <atikhono@redhat.com> - 2.8.2-2
 - Resolves: rhbz#2149091 - Update to sssd-2.7.3-4.el8_7.1.x86_64 resulted in "Request to sssd failed. Device or resource busy"