linux-kernel - Re: [PATCH] udf: implement mode and dmode mounting options

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20081119191338.GD29820@duck.suse.cz>
Date:	Wed, 19 Nov 2008 20:13:38 +0100
From:	Jan Kara <jack@...e.cz>
To:	Marcin Slusarz <marcin.slusarz@...il.com>
Cc:	Andrew Morton <akpm@...ux-foundation.org>,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH] udf: implement mode and dmode mounting options

On Wed 19-11-08 19:14:57, Marcin Slusarz wrote:
> On Wed, Nov 19, 2008 at 04:16:06PM +0100, Jan Kara wrote:
> > On Tue 18-11-08 16:22:05, Andrew Morton wrote:
> > > On Sun, 16 Nov 2008 20:52:19 +0100
> > > Marcin Slusarz <marcin.slusarz@...il.com> wrote:
> > > 
> > > > "dmode" allows overriding permissions of directories and
> > > > "mode" allows overriding permissions of files.
> > > > 
> > > > Signed-off-by: Marcin Slusarz <marcin.slusarz@...il.com>
> > > > Cc: Jan Kara <jack@...e.cz>
> > > > ---
> > > >  Documentation/filesystems/udf.txt |    2 ++
> > > >  fs/udf/inode.c                    |   11 +++++++++--
> > > >  fs/udf/super.c                    |   31 ++++++++++++++++++++++++++++++-
> > > >  fs/udf/udf_sb.h                   |    2 ++
> > > >  4 files changed, 43 insertions(+), 3 deletions(-)
> > > > 
> > > > diff --git a/Documentation/filesystems/udf.txt b/Documentation/filesystems/udf.txt
> > > > index fde829a..902b95d 100644
> > > > --- a/Documentation/filesystems/udf.txt
> > > > +++ b/Documentation/filesystems/udf.txt
> > > > @@ -24,6 +24,8 @@ The following mount options are supported:
> > > >  
> > > >  	gid=		Set the default group.
> > > >  	umask=		Set the default umask.
> > > > +	mode=		Set the default file permissions.
> > > > +	dmode=		Set the default directory permissions.
> > > 
> > > Can we give these the same names and usage as their fatfs equivalents?
> >   Yes, making these options dmask and fmask would be more consistent I
> > think. Martin?
> 
> You can't add permissions by umask/fmask/dmask. I have one DVD with top
> level dir permissions set to 0000 and there's no way to read it now (as
> an user) and adding mask options wouldn't change it.
> 
> I should mention in a changelog that the names and semantics of these
> options are copied from isofs.
> 
> I can create a second patch which adds mask options but I think modes are
> needed too. What do you think?
  Ah, ok. That makes sence (although such DVD's seem to be really broken).
I'll merge the patch.

									Honza
-- 
Jan Kara <jack@...e.cz>
SUSE Labs, CR
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/