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

Several of the tests produce false failures #24

Closed
opub opened this issue Dec 10, 2019 · 1 comment
Closed

Several of the tests produce false failures #24

opub opened this issue Dec 10, 2019 · 1 comment
Assignees

Comments

@opub
Copy link

opub commented Dec 10, 2019

1.2.1 - is too restrictive since it assumes the existence of specific repo names (base and updates).

1.7.1.4 - if /etc/motd is a symbolic link then test_perms incorrectly inspects link perms instead of file.

3.6.3 and 3.6.4 - this doesn't take into account the iptables rules potentially having comments in them.

4.2.1.3 - doesn't find FileCreateMode if it is set under a /etc/rsyslog.d/*.conf file.

5.4.4 - doesn't find user mask if it occurs multiple times in file like in an if/else condition.

@opub opub changed the title 4.2.1.3 doesn't find FileCreateMode set under /etc/rsyslog.d/*.conf Several of the tests produce false failures Dec 12, 2019
@finalduty finalduty self-assigned this Jul 14, 2020
@finalduty
Copy link
Owner

Hey @opub, thanks for your report and my apologies for not getting to this sooner.

1.2.1 - I originally wrote this based on the default repo names, but it also doesn't take in to account other repos that may be required. The remediation in the standard says to configure the repos according to site policy, so I will instead skip this check and require the user to manually verify it themselves.

1.7.1.4, 3.6.3, 3.6.4 - I have replicated these issues. I've tested a fix and will push that shortly.

5.4.4 - I've updated this check to test all umasks within /etc/bashrc and /etc/profile to ensure they all meet the criteria.

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

No branches or pull requests

2 participants