Skip to content

[autobackport: sssd-2-9] ldap: pwd_expire_policy_warn does not overwrite errors - #9443

Merged
alexey-tikhonov merged 4 commits into
SSSD:sssd-2-9from
sssd-bot:SSSD-sssd-backport-pr9393-to-sssd-2-9
Oct 13, 2026
Merged

alexey-tikhonov merged 4 commits into
SSSD:sssd-2-9from
sssd-bot:SSSD-sssd-backport-pr9393-to-sssd-2-9

Conversation

@sssd-bot

Copy link
Copy Markdown
Contributor

This is an automatic backport of PR#9393 ldap: pwd_expire_policy_warn does not overwrite errors to branch sssd-2-9, created by @sumit-bose.

Caution

@sumit-bose The patches did not apply cleanly. It is necessary to resolve conflicts before merging this pull request. Commits that introduced conflict are marked with CONFLICT!.

You can push changes to this pull request

git remote add sssd-bot git@github.com:sssd-bot/sssd.git
git fetch sssd-bot refs/heads/SSSD-sssd-backport-pr9393-to-sssd-2-9
git checkout SSSD-sssd-backport-pr9393-to-sssd-2-9
git push sssd-bot SSSD-sssd-backport-pr9393-to-sssd-2-9 --force

Original commits
e83a29c - ldap: pwd_expire_policy_warn does not overwrite errors
57d6818 - test: pwd_expire_policy_warn does not overwrite errors (IPA)
1a258a4 - ldap: show expired message if shadow lastChange is missing or 0
aaaea88 - test: pwd_expire_policy_warn does not overwrite errors (LDAP)

Backported commits

  • 10af43b - ldap: pwd_expire_policy_warn does not overwrite errors
  • 48c3081 - CONFLICT! test: pwd_expire_policy_warn does not overwrite errors (IPA)
  • 0fc9b84 - ldap: show expired message if shadow lastChange is missing or 0
  • 0f97e06 - CONFLICT! test: pwd_expire_policy_warn does not overwrite errors (LDAP)

Conflicting Files Information (check for deleted and re-added files)

  • CONFLICT! test: pwd_expire_policy_warn does not overwrite errors (IPA)
On branch SSSD-sssd-backport-pr9393-to-sssd-2-9
You are currently cherry-picking commit 57d68184a.
  (fix conflicts and run "git cherry-pick --continue")
  (use "git cherry-pick --skip" to skip this patch)
  (use "git cherry-pick --abort" to cancel the cherry-pick operation)

Unmerged paths:
  (use "git add <file>..." to mark resolution)
	both modified:   src/tests/system/tests/test_ipa.py

no changes added to commit (use "git add" and/or "git commit -a")
```* CONFLICT! test: pwd_expire_policy_warn does not overwrite errors (LDAP)

On branch SSSD-sssd-backport-pr9393-to-sssd-2-9
You are currently cherry-picking commit aaaea88.
(fix conflicts and run "git cherry-pick --continue")
(use "git cherry-pick --skip" to skip this patch)
(use "git cherry-pick --abort" to cancel the cherry-pick operation)

Unmerged paths:
(use "git add ..." to mark resolution)
both modified: src/tests/system/tests/test_ldap.py

no changes added to commit (use "git add" and/or "git commit -a")


---

**Original Pull Request Body**

Make sure the 'pwd_expire_policy_warn' access check does not overwrite
any access denied or other errors.

:fixes: CVE-2026-92821

Comment thread src/tests/system/tests/test_ipa.py Outdated
time.sleep(3)
log2 = client.fs.read(client.sssd.logs.pam)
assert "indicators: 0" in log2, "String `indicators: 0` not found in logs!"
<<<<<<< HEAD
Comment thread src/tests/system/tests/test_ldap.py Outdated
rc, _, stdout, _ = client.auth.parametrize(method).password_with_output("user1", "Secret123")
assert rc == 0, "User 'user1' login failed!"
assert "Your password will expire in " in stdout, "Password expiration warning not generated!"
<<<<<<< HEAD
@sssd-bot
sssd-bot force-pushed the SSSD-sssd-backport-pr9393-to-sssd-2-9 branch from 0f97e06 to 75c3f74 Compare October 12, 2026 17:37
@alexey-tikhonov alexey-tikhonov added no-backport This should go to target branch only. Changes requested labels Oct 12, 2026
@sssd-bot
sssd-bot force-pushed the SSSD-sssd-backport-pr9393-to-sssd-2-9 branch from 75c3f74 to ff3b98d Compare October 12, 2026 17:39
@sumit-bose
sumit-bose force-pushed the SSSD-sssd-backport-pr9393-to-sssd-2-9 branch 2 times, most recently from a3c66a2 to c16c56d Compare October 13, 2026 07:53
@sumit-bose
sumit-bose marked this pull request as ready for review October 13, 2026 08:38
Make sure the 'pwd_expire_policy_warn' access check does not overwrite
any access denied or other errors.

:fixes: CVE-2026-92821

Reviewed-by: Alexey Tikhonov <atikhono@redhat.com>
Reviewed-by: Dan Lavu <dlavu@redhat.com>
(cherry picked from commit e83a29c)
Verify fix for CVE-2026-92821 and make sure the 'pwd_expire_policy_warn'
access check does not overwrite any access denied or other errors.

Reviewed-by: Alexey Tikhonov <atikhono@redhat.com>
Reviewed-by: Dan Lavu <dlavu@redhat.com>
(cherry picked from commit 57d6818)
If the shadow scheme is used to determine if a password is expired or
not the user currently does not see a message that the password is
expired if the shadow attribute sp_lstchg is missing or 0. With this
patch the message is shown in this case as well.

Reviewed-by: Alexey Tikhonov <atikhono@redhat.com>
Reviewed-by: Dan Lavu <dlavu@redhat.com>
(cherry picked from commit 1a258a4)
Verify fix for CVE-2026-92821 and make sure the 'pwd_expire_policy_warn'
access check does not overwrite any access denied or other errors.

Reviewed-by: Alexey Tikhonov <atikhono@redhat.com>
Reviewed-by: Dan Lavu <dlavu@redhat.com>
(cherry picked from commit aaaea88)
@sssd-bot

Copy link
Copy Markdown
Contributor Author

The pull request was accepted by @alexey-tikhonov with the following PR CI status:


🟢 rpm-build:centos-stream-9-x86_64:upstream (success)
🟢 testing-farm:centos-stream-9-x86_64:upstream (success)
🟢 Build / make-distcheck (success)
🟢 ci / prepare (success)
🟢 ci / system (centos-9) (success)
🟢 Static code analysis / files-changed (success)
🟢 Static code analysis / pre-commit (success)
🟢 Static code analysis / python-system-tests (success)


There are unsuccessful or unfinished checks. Make sure that the failures are not related to this pull request before merging.

@sssd-bot
sssd-bot force-pushed the SSSD-sssd-backport-pr9393-to-sssd-2-9 branch from c16c56d to 6641224 Compare October 13, 2026 10:21
@alexey-tikhonov
alexey-tikhonov merged commit 865b654 into SSSD:sssd-2-9 Oct 13, 2026
7 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Accepted no-backport This should go to target branch only.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants