[prev in list] [next in list] [prev in thread] [next in thread] 

List:       busybox
Subject:    Re: mount doesn't understand the auto mount option
From:       Chris Boot <bootc () debian ! org>
Date:       2020-06-29 14:04:48
Message-ID: b1858b7c-14e6-25dd-9bdb-bbacb4e8dbbe () debian ! org
[Download RAW message or body]

On 23/07/2019 18:13, Chris Boot wrote:
> Package: busybox
> Version: 1.30.1
> 
> We've received a bug in Debian about busybox's mount command not
> supporting the 'auto' option in fstab. This option really is a no-op as
> it is default behaviour that can be disabled with noauto, but it turns
> out that some users add this to their fstab. With util-linux's mount
> this works fine, but busybox mount croaks.
> 
> If a user has a separate /usr filesystem, this is mounted in the
> initramfs after the root filesystem. When a user has a separate /usr
> with the auto mount option set, the filesystem fails to mount and the
> user gets dumped into the initramfs shell.
> 
> The patch below appears to fix the problem:
> 
> --- a/util-linux/mount.c
> +++ b/util-linux/mount.c
> @@ -298,6 +298,7 @@
>  		/* "defaults" */ 0,
>  		/* "quiet" 0 - do not filter out, vfat wants to see it */
>  		/* "noauto" */ MOUNT_NOAUTO,
> +		/* "auto" */ ~MOUNT_NOAUTO,
>  		/* "sw"     */ MOUNT_SWAP,
>  		/* "swap"   */ MOUNT_SWAP,
>  		IF_DESKTOP(/* "user"  */ MOUNT_USERS,)
> @@ -358,6 +359,7 @@
>  		"defaults\0"
>  		// "quiet\0" - do not filter out, vfat wants to see it
>  		"noauto\0"
> +		"auto\0"
>  		"sw\0"
>  		"swap\0"
>  		IF_DESKTOP("user\0")
> 
> For reference, the Debian bug is:
> https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=847424

It's been almost a year since I posted this patch. Is there any chance
it can be merged please?

Cheers,
Chris

-- 
Chris Boot
bootc@debian.org
_______________________________________________
busybox mailing list
busybox@busybox.net
http://lists.busybox.net/mailman/listinfo/busybox
[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic