Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

General hardening #7764

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

alexey-tikhonov
Copy link
Member

@alexey-tikhonov alexey-tikhonov commented Dec 13, 2024

Clear env of privileged 'sssd_pam' as a security hardening measure.

@alexey-tikhonov
Copy link
Member Author

Looks like clearing 'LDB_MODULES_PATH' breaks a lot of integration tests:

sssd/Makefile.am

Line 2050 in af65c00

TESTS_ENVIRONMENT = LDB_MODULES_PATH=$(abs_top_builddir)/ldb_mod_test_dir \

so I've split this PR into two (see #7774)

This one will be probably blocked until we get rid of intg-tests...

@jengelh
Copy link
Contributor

jengelh commented Dec 19, 2024

Ironically, I did spot that previously and applied it in cc675eb in part.

@eslerm
Copy link

eslerm commented Dec 19, 2024

This PR is mentioned in a vulnerability report. By not assigning CVEs, downstream maintainers (like distros) will not know which security patches are required for backporting to make downstream users of SSSD secure.

https://www.openwall.com/lists/oss-security/2024/12/19/1

@alexey-tikhonov
Copy link
Member Author

This PR is mentioned in a vulnerability report.

Not a "vulnerability" but a "weaknesses".
No way to exploit it was presented thus no CVEs were assigned.

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