linux-kernel - Re: [PATCH 1/4] Cache xattr security drop check for write v2

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: <20110618180450.GH16236@one.firstfloor.org>
Date:	Sat, 18 Jun 2011 20:04:50 +0200
From:	Andi Kleen <andi@...stfloor.org>
To:	Marco Stornelli <marco.stornelli@...il.com>
Cc:	Al Viro <viro@...IV.linux.org.uk>,
	Steven Whitehouse <swhiteho@...hat.com>,
	Andi Kleen <ak@...ux.intel.com>,
	Andi Kleen <andi@...stfloor.org>, linux-kernel@...r.kernel.org,
	linux-fsdevel@...r.kernel.org, chris.mason@...cle.com,
	josef@...hat.com, agruen@...bit.com,
	"Serge E. Hallyn" <serue@...ibm.com>
Subject: Re: [PATCH 1/4] Cache xattr security drop check for write v2

> several fs now uses MS_NOSEC (because this flag is set in mount_bdev()) 
> but I don't see any user of the function inode_has_no_xattr() in the 
> latest version. If I well understand, a fs that wants to manage this 
> feature has to set MS_NOSEC and calls when needed this function, isn't 
> it? So at this point, why there aren't any user of this function?

Calling the function is just an optimization to avoid the lookup
for the first write.  But most of the benefit you already get
when just the flag is set.

I haven't resent the patches using it to the fs maintainers yet and none
of them took it the first round. 

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