linux-kernel - Re: [patch] reiser4: do not allocate struct file on stack

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]

Date:	Fri, 05 Oct 2007 10:22:38 -0600
From:	Zan Lynx <zlynx@....org>
To:	Edward Shishkin <edward@...esys.com>
Cc:	Andrew Morton <akpm@...ux-foundation.org>,
	"Vladimir V. Saveliev" <vs@...sterfs.com>,
	Dave Hansen <haveblue@...ibm.com>,
	Linux Kernel <linux-kernel@...r.kernel.org>,
	reiserfs-devel <reiserfs-devel@...r.kernel.org>,
	hch <hch@...radead.org>
Subject: Re: [patch] reiser4: do not allocate struct file on stack

On Fri, 2007-10-05 at 03:22 +0400, Edward Shishkin wrote:
> Edward Shishkin wrote:
> 
> > Dave Hansen wrote:
> >
> ...
> 
> >>
> >> I think that stack allocation is a pretty nasty trick for a structure
> >> that's supposed to be pretty persistent and dynamically allocated, and
> >> is certainly something that needs to get fixed up in a proper way.
> >>
> >>
> >
> > agreed.
> >
> >> This works around the problem for now, but this could potentially cause
> >> more bugs any time we add some member to 'struct file' and depend on its
> >> state being sane anywhere in the VFS. If there's a list anywhere of
> >> merge-stopper reiser4 bugs around, this should probably go in there.
> >>
> >>
> >
> > will be fixed.
> >
> 
> The promised fixup is attached.
> Andrew, please apply.
> 
> Thanks,
> Edward.

There were several copies of this email, so I picked one. :)

I had the crash with 2.6.23-rc8-mm2, so I applied this patch.  Hunk #1
failed, the rest applied with fuzz.  I manually applied hunk #1 and
recompiled.

I am now running with 2.6.23-rc8-mm2 + this patch and things seem good.

(Except for some issues with ACPI battery info moving out of proc and
confusing the heck out of HAL and the GNOME power management scripts).
-- 
Zan Lynx <zlynx@....org>

Download attachment "signature.asc" of type "application/pgp-signature" (190 bytes)