Skip to content

Commit

Permalink
fix: don't run permissions checks when root or docker
Browse files Browse the repository at this point in the history
  • Loading branch information
dgibbs64 committed Oct 29, 2022
1 parent 2257be4 commit bbf56a0
Showing 1 changed file with 3 additions and 5 deletions.
8 changes: 3 additions & 5 deletions lgsm/functions/check_permissions.sh
Original file line number Diff line number Diff line change
Expand Up @@ -181,7 +181,7 @@ fn_sys_perm_fix_manually_msg() {

# Attempt to fix /sys related permission errors if sudo is available, exits otherwise.
fn_sys_perm_errors_fix() {
if sudo -n true > /dev/null 2>&1; then
if sudo -n true >/dev/null 2>&1; then
fn_print_dots "Automatically fixing /sys permissions"
fn_script_log_info "Automatically fixing /sys permissions."
if [ "${sysdirpermerror}" == "1" ]; then
Expand Down Expand Up @@ -223,10 +223,8 @@ fn_sys_perm_error_process() {
fi
}

# Run perm error detect & fix/alert functions on /sys directories.

## Run checks.
if [ "$(whoami)" != "root" ] || [ -f /.dockerenv ]; then
## Run permisions checks when not root or docker.
if [ "$(whoami)" != "root" ] && [ ! -f /.dockerenv ]; then
fn_check_ownership
fn_check_permissions
if [ "${commandname}" == "START" ]; then
Expand Down

0 comments on commit bbf56a0

Please sign in to comment.