Skip to content

Commit

Permalink
Commit fb71f0d introduced to possibility to add options behind permis…
Browse files Browse the repository at this point in the history
…sions

in fstab. Unfortunately it breaks scenarios where no options are provided
as the current regex expects the comma with the options always to be present.

This patch fixes the regex to handle the options as group.
  • Loading branch information
MatthiasPetermann committed Jan 17, 2025
1 parent fb876b4 commit ce0ab9d
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion usr/local/share/bastille/mount.sh
Original file line number Diff line number Diff line change
Expand Up @@ -92,7 +92,7 @@ elif [ ! -e "${_hostpath}" ] || [ "${_type}" != "nullfs" ]; then
fi

# Mount permissions,options need to start with "ro" or "rw"
if ! echo "${_perms}" | grep -Eq 'r[w|o],.*$'; then
if ! echo "${_perms}" | grep -Eq 'r[w|o](,.*)?$'; then
error_notify "Detected invalid mount permissions in FSTAB."
warn "Format: /host/path /jail/path nullfs ro 0 0"
warn "Read: ${_fstab}"
Expand Down

0 comments on commit ce0ab9d

Please sign in to comment.