Verify Groupownership of Files in /var/log/landscape

Description

To properly set the group owner of /var/log/landscape/*, run the command:

$ sudo chgrp root /var/log/landscape/*

or

$ sudo chgrp landscape /var/log/landscape/*

Rationale

The /var/log/landscape directory contains information about the landscape-client and should only be accessed by authorized personnel.

Remediation

Shell script

The following script can be run on the host to remediate the issue.

#!/bin/bash

newgroup=""
if getent group "root" >/dev/null 2>&1; then
  newgroup="root"
elif getent group "landscape" >/dev/null 2>&1; then
  newgroup="landscape"
fi

if [[ -z "${newgroup}" ]]; then
  >&2 echo "root and landscape is not a defined group on the system"
else
find -P /var/log/landscape/ -maxdepth 1 -type f  ! -group root ! -group landscape -regextype posix-extended -regex '^.*$' -exec chgrp --no-dereference "$newgroup" {} \;

fi

Ansible playbook

The following playbook can be run with Ansible to remediate the issue.

- name: Check that the root group is defined
  ansible.builtin.getent:
    database: group
    key: root
  ignore_errors: true
  when: file_groupownerships_var_log_landscape_newgroup is undefined
  tags:
  - configure_strategy
  - file_groupownerships_var_log_landscape
  - low_complexity
  - low_disruption
  - medium_severity
  - no_reboot_needed

- name: Set the file_groupownerships_var_log_landscape_newgroup variable if root found
  ansible.builtin.set_fact:
    file_groupownerships_var_log_landscape_newgroup: root
  when: ansible_facts.getent_group["root"] is defined
  tags:
  - configure_strategy
  - file_groupownerships_var_log_landscape
  - low_complexity
  - low_disruption
  - medium_severity
  - no_reboot_needed

- name: Check that the landscape group is defined
  ansible.builtin.getent:
    database: group
    key: landscape
  ignore_errors: true
  when: file_groupownerships_var_log_landscape_newgroup is undefined
  tags:
  - configure_strategy
  - file_groupownerships_var_log_landscape
  - low_complexity
  - low_disruption
  - medium_severity
  - no_reboot_needed

- name: Set the file_groupownerships_var_log_landscape_newgroup variable if landscape
    found
  ansible.builtin.set_fact:
    file_groupownerships_var_log_landscape_newgroup: landscape
  when: ansible_facts.getent_group["landscape"] is defined
  tags:
  - configure_strategy
  - file_groupownerships_var_log_landscape
  - low_complexity
  - low_disruption
  - medium_severity
  - no_reboot_needed

- name: Find /var/log/landscape/ file(s) matching ^.*$
  ansible.builtin.command: find -P /var/log/landscape/ -maxdepth 1 -type f  ! -group
    root ! -group landscape -regextype posix-extended -regex "^.*$"
  register: files_found
  changed_when: false
  failed_when: false
  check_mode: false
  tags:
  - configure_strategy
  - file_groupownerships_var_log_landscape
  - low_complexity
  - low_disruption
  - medium_severity
  - no_reboot_needed

- name: Ensure group owner on /var/log/landscape/ file(s) matching ^.*$
  ansible.builtin.file:
    path: '{{ item }}'
    follow: false
    group: '{{ file_groupownerships_var_log_landscape_newgroup }}'
    state: file
  with_items:
  - '{{ files_found.stdout_lines }}'
  tags:
  - configure_strategy
  - file_groupownerships_var_log_landscape
  - low_complexity
  - low_disruption
  - medium_severity
  - no_reboot_needed