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

Integrate 'Permission settings' of atsec into openQA #14962

Merged
merged 1 commit into from May 27, 2022
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
1 change: 1 addition & 0 deletions schedule/security/atsec_tests.yaml
Expand Up @@ -12,6 +12,7 @@ schedule:
- security/atsec/syscall_thrasher
- security/atsec/netlink_message
- security/atsec/chrony_pid_file
- security/atsec/permission_settings
conditional_schedule:
bootloader_zkvm:
ARCH:
Expand Down
37 changes: 37 additions & 0 deletions tests/security/atsec/permission_settings.pm
@@ -0,0 +1,37 @@
# SUSE's openQA tests
#
# Copyright 2022 SUSE LLC
# SPDX-License-Identifier: FSFAP
#
# Summary: Run 'Permission settings of relevant configuration files' test case of ATSec test suite
# Maintainer: xiaojing.liu <xiaojing.liu@suse.com>
# Tags: poo#111518

use base 'consoletest';
use strict;
use warnings;
use testapi;
use utils;

sub run {
my ($self) = shift;

select_console 'root-console';

my $output = script_output('find -L /etc -perm -g+w,o+w');

# This test is to verify that important configuration files are protected
# against access by unauthorized users. The result shows files that are softlinks
# or the random device is allowed exception to the intial result expectation.
foreach my $file (split('\n', $output)) {
my $file_detail = script_output("readlink $file");
if ($file_detail !~ /(\/dev\/null|\/dev\/random)/) {

Copy link
Contributor

Choose a reason for hiding this comment

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

I think it's better to remove this blank line.

# The file is not a softlink or doesn't link to expected device
record_info($file, $file_detail, result => 'fail');
$self->result('fail');
}
}
}

1;