linux-kernel - Re: [PATCH] Add CONFIG_VFAT_NO_CREATE_WITH

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <524f69650905012002u190f357ax5fad7f3d8d660bcf@mail.gmail.com>
Date:	Fri, 1 May 2009 22:02:38 -0500
From:	Steve French <smfrench@...il.com>
To:	Matthew Wilcox <matthew@....cx>
Cc:	"Paul E. McKenney" <paulmck@...ux.vnet.ibm.com>,
	Christoph Hellwig <hch@...radead.org>,
	Dave Kleikamp <shaggy@...ux.vnet.ibm.com>,
	Ogawa Hirofumi <hirofumi@...l.parknet.co.jp>,
	linux-fsdevel <linux-fsdevel@...r.kernel.org>,
	Michael Tokarev <mjt@....msk.ru>,
	Andrew Tridgell <tridge@...ba.org>,
	LKML <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] Add CONFIG_VFAT_NO_CREATE_WITH_LONGNAMES option

On Fri, May 1, 2009 at 8:59 PM, Matthew Wilcox <matthew@....cx> wrote:
> On Fri, May 01, 2009 at 06:37:29PM -0700, Paul E. McKenney wrote:
>> However, as far as I know, none of us are lawyers, and LKML is definitely
>> a technical rather than a legal forum, so we really do need to stick to
>> technical topics.  I understand that this might be a bit frustrating
>> to you.  On the other hand, I for one much prefer being in a forum
>> restricted to technical topics than to be in those places designed to
>> handle legal topics!
>
> So what's the purely technical argument for including this patch?
The strength of Tridge's approach over alternatives may be that it is clearer
to only create one name (always) and that it is relatively small.
Its weakness in my opinion is that as a sideeffect it prevents saving mixed
case file names.

There are various ways to disable (only) creating long file names, ie
to remove the ability
to create files longer than 8.3 from this module.   Tridge's approach (which
basically avoids saving the alternate name, the mixed case name) works.
I realize that there are other ways to do this with trivial #ifdefs in
namei_vfat.c (see
vfat_build_slots and vfat_create_shortname), and when I tried other approaches
they worked too, although you may prefer Tridge's approach.



-- 
Thanks,

Steve
--
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/