Skip to content

Commit 384c097

Browse files
authored
feat(os_hardening): extend file permission tasks to cover more files (#489)
The tasks `Change shadow ownership to root and mode to 0600` and `Change passwd ownership to root and mode to 0644` only handle `/etc/shadow` and `/etc/passwd` respectively. But there multiple adjacent files that should be handled with these rules as well: - `/etc/gshadow` - `/etc/shadow-` - `/etc/gshadow-` - `/etc/group` - `/etc/shadow-` - `/etc/group-` This change adds those files to the rules, so that permissions are handled in the same way. Closes: #488 Signed-off-by: Claudius Heine <[email protected]>
1 parent 999e5fa commit 384c097

File tree

1 file changed

+26
-2
lines changed

1 file changed

+26
-2
lines changed

roles/os_hardening/tasks/minimize_access.yml

Lines changed: 26 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27,19 +27,43 @@
2727
- "{{ minimize_access_directories.results }}"
2828
- stdout_lines
2929

30+
- name: Find shadow files
31+
stat:
32+
path: "{{ item }}"
33+
loop:
34+
- '/etc/shadow'
35+
- '/etc/gshadow'
36+
- '/etc/shadow-'
37+
- '/etc/gshadow-'
38+
register: minimize_access_shadow_files
39+
3040
- name: Change shadow ownership to root and mode to 0600 | os-02
3141
file:
32-
dest: '/etc/shadow'
42+
dest: "{{ item.item }}"
3343
owner: '{{ os_shadow_perms.owner }}'
3444
group: '{{ os_shadow_perms.group }}'
3545
mode: '{{ os_shadow_perms.mode }}'
46+
when: item.stat.exists
47+
loop: "{{ minimize_access_shadow_files.results }}"
48+
49+
- name: Find passwd files
50+
stat:
51+
path: "{{ item }}"
52+
loop:
53+
- '/etc/passwd'
54+
- '/etc/group'
55+
- '/etc/passwd-'
56+
- '/etc/group-'
57+
register: minimize_access_passwd_files
3658

3759
- name: Change passwd ownership to root and mode to 0644 | os-03
3860
file:
39-
dest: '/etc/passwd'
61+
dest: "{{ item.item }}"
4062
owner: '{{ os_passwd_perms.owner }}'
4163
group: '{{ os_passwd_perms.group }}'
4264
mode: '{{ os_passwd_perms.mode }}'
65+
when: item.stat.exists
66+
loop: "{{ minimize_access_passwd_files.results }}"
4367

4468
- name: Change su-binary to only be accessible to user and group root
4569
file:

0 commit comments

Comments
 (0)