Re: CVS commit: src

["J. Hannken-Illjes" <hannken%mailbox.org@localhost>]

> On 2. Feb 2024, at 21:27, Christos Zoulas <christos%netbsd.org@localhost> wrote:
> 
> Module Name: src
> Committed By: christos
> Date: Fri Feb  2 20:27:26 UTC 2024
> 
> Modified Files:
> src/sbin/mount_cd9660: Makefile mount_cd9660.8 mount_cd9660.c
> src/sys/fs/cd9660: cd9660_extern.h cd9660_mount.h cd9660_vfsops.c
>    cd9660_vnops.c
> 
> Log Message:
> PR/57897: Ricardo Branco: Add support for mount options mask,dirmask,uid,gid
> 
> 
> To generate a diff of this commit:
> cvs rdiff -u -r1.12 -r1.13 src/sbin/mount_cd9660/Makefile
> cvs rdiff -u -r1.31 -r1.32 src/sbin/mount_cd9660/mount_cd9660.8
> cvs rdiff -u -r1.33 -r1.34 src/sbin/mount_cd9660/mount_cd9660.c
> cvs rdiff -u -r1.27 -r1.28 src/sys/fs/cd9660/cd9660_extern.h
> cvs rdiff -u -r1.6 -r1.7 src/sys/fs/cd9660/cd9660_mount.h
> cvs rdiff -u -r1.97 -r1.98 src/sys/fs/cd9660/cd9660_vfsops.c
> cvs rdiff -u -r1.62 -r1.63 src/sys/fs/cd9660/cd9660_vnops.c

First, this is not backwards compatible.  An old mount_cd9660 and
a new kernel will always return EINVAL as the argument length
check on top of cd9660_mount() fires.

Second, with these new mount arguments an update mount has to update
the masks and ids, currently they are silently ignored.

--
J. Hannken-Illjes - hannken%mailbox.org@localhost