Skip to content

ldap: pwd_expire_policy_warn does not overwrite errors - #9393

Merged
alexey-tikhonov merged 4 commits into
SSSD:masterfrom
sumit-bose:CVE-2026-92821
Oct 12, 2026
Merged

alexey-tikhonov merged 4 commits into
SSSD:masterfrom
sumit-bose:CVE-2026-92821

Conversation

@sumit-bose

Copy link
Copy Markdown
Contributor

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_access_control_ldap_filter.py Outdated
@alexey-tikhonov

Copy link
Copy Markdown
Member

Fixes per see look good to me.
There is a test related comment from LLM that looks plausible to me.

@danlavu danlavu left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These tests are great, one nitpick and... these tests should be moved to test_ldap.py, since this file contains parametrized ad_access_filter and ldap_access_filter tests. Would expect any test in here to have that configurable.

I know the name is confusing, because ad_access_filter and ldap_access_filter, is access control with ldap filtering, explains the thought process.

f"-l {user.name} localhost whoami",
raise_on_error=False,
)
assert result.rc == 0, "SSH with pubkey failed"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nitpick, ! at the end of these assert messages, all assert messages.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Exclamations marks added.

@sumit-bose

Copy link
Copy Markdown
Contributor Author

These tests are great, one nitpick and... these tests should be moved to test_ldap.py, since this file contains parametrized ad_access_filter and ldap_access_filter tests. Would expect any test in here to have that configurable.

I know the name is confusing, because ad_access_filter and ldap_access_filter, is access control with ldap filtering, explains the thought process.

Hi,

I move the test to test_ldap.py.

bye,
Sumit

@alexey-tikhonov

Copy link
Copy Markdown
Member

Failing test_sudo__ldap_sudo_search_base_not_set_emits_warning is unrelated to this PR.

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>
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>
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>
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>
@sssd-bot

Copy link
Copy Markdown
Contributor

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


🟢 CodeQL (success)
🟢 osh-diff-scan:fedora-rawhide-x86_64:upstream (success)
🟢 rpm-build:centos-stream-10-x86_64:upstream (success)
🟢 rpm-build:fedora-43-x86_64:upstream (success)
🟢 rpm-build:fedora-44-x86_64:upstream (success)
🟢 rpm-build:fedora-45-x86_64:upstream (success)
🟢 rpm-build:fedora-rawhide-x86_64:upstream (success)
🟢 testing-farm:centos-stream-10-x86_64:upstream (success)
🟢 testing-farm:fedora-43-x86_64:upstream (success)
🟢 testing-farm:fedora-44-x86_64:upstream (success)
🟢 testing-farm:fedora-45-x86_64:upstream (success)
🟢 testing-farm:fedora-rawhide-x86_64:upstream (success)
🟢 Build / freebsd (success)
🟢 Build / make-distcheck (success)
🟢 ci / prepare (success)
🟢 ci / system (centos-10) (success)
🟢 ci / system (fedora-43) (success)
🟢 ci / system (fedora-44) (success)
🔴 ci / system (fedora-45) (failure)
🔴 ci / system (fedora-46) (failure)
➖ Coverity scan / coverity (skipped)
🟢 Static code analysis / codeql (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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants