Skip to content

initramfs-scripts: require canmount=on for non-bootfs #17995

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.

Sign up for GitHub

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

Closed
wants to merge 4 commits into from
Closed

initramfs-scripts: require canmount=on for non-bootfs #17995

Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
b2b53ad
initramfs-scripts: require canmount=on for non-bootfs
bar-g Nov 30, 2025
8ddd819
improved comment
bar-g Nov 30, 2025
5f5e0e5
improved comment
bar-g Nov 30, 2025
0e2506b
improve comment
bar-g Dec 1, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 5 additions & 3 deletions contrib/initramfs/scripts/zfs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -334,12 +334,14 @@ mount_fs()
# Check that the filesystem exists
"${ZFS}" list -oname -tfilesystem -H "${fs}" > /dev/null 2>&1 || return 1

# Skip filesystems with canmount=off. The root fs should not have
# canmount=off, but ignore it for backwards compatibility just in case.
# Skip filesystems not set to canmount=on (i.e. off or noauto), except if set as bootfs.
# The bootfs should not have canmount=off (not be just a placehlder/inheritance provider),
# but we also ignore 'off' for backwards compatibility just in case (top-level canmount=off
# boot pool).
if [ "$fs" != "${ZFS_BOOTFS}" ]
then
canmount=$(get_fs_value "$fs" canmount)
[ "$canmount" = "off" ] && return 0
[ "$canmount" != "on" ] && return 0
fi

# Need the _original_ datasets mountpoint!
Expand Down
Loading