linux-ext4 - Re: [PATCH] ext4: fix #11321: create /proc/ext4/*/stats et al more carefully

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: <20080907162447.GB32429@mit.edu>
Date:	Sun, 7 Sep 2008 12:24:47 -0400
From:	Theodore Tso <tytso@....EDU>
To:	Alexey Dobriyan <adobriyan@...il.com>,
	Ralf Hildebrandt <Ralf.Hildebrandt@...rite.de>
Cc:	Andreas Dilger <adilger@....com>, akpm@...l.org,
	linux-ext4@...r.kernel.org
Subject: Re: [PATCH] ext4: fix #11321: create /proc/ext4/*/stats et al more
	carefully

On Sun, Sep 07, 2008 at 04:15:57PM +0400, Alexey Dobriyan wrote:
> > Why not use strchr(), which is normally optimized assembly:
> > 
> > 	char *p = devname;
> > 	while ((p = strchr(p, '/'))
> > 		*p = '_';
> > 
> > Using '!' as the separator makes it harder to use from shells I suspect,
> > so I'd suggest '_' instead.
> 
> bdevname is only 32 bytes and done once per mount, so nobody cares.

Bloat gets inserted into the kernel, 32 bytes at a time.  :-)

> '!' is what other code does in this situation (reiserfs, md, ...).

I'm not convinced that the consistency is as important in this case as
making it easy for people using shells typing the pathname...

On Sun, Sep 07, 2008 at 06:04:14PM +0200, Ralf Hildebrandt wrote:
> 
> Anyway, the first patch fixes the issue. Will it go into mainline soon?
> 

I'll queue the patch (with fixups) for the 2.6.27 merge window.  This
isn't a regression, and it's getting rather late in the 2.6.27-rc
series.  I've bent the rules about submitting non-regression bug fixes
back when most ext4 users were testers or developers; but now that we
are getting real users, and Linus has started yelling at developers
for ignoring the merge window rules, I'm going to be much more of a
stickler about only pushing regression bug-fixes after the merge
window closes.

						- Ted
--
To unsubscribe from this list: send the line "unsubscribe linux-ext4" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html