linux-kernel - Re: [PATCH] extend_brk and fixes to users of extend

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]

Date:	Tue, 31 Jul 2012 08:49:21 -0700
From:	"H. Peter Anvin" <hpa@...or.com>
To:	Konrad Rzeszutek Wilk <konrad.wilk@...cle.com>
CC:	linux-kernel@...r.kernel.org, xen-devel@...ts.xensource.com,
	Ian.Campbell@...rix.com, x86@...nel.org
Subject: Re: [PATCH] extend_brk and fixes to users of extend_brk (v1).

On 07/31/2012 03:42 AM, Konrad Rzeszutek Wilk wrote:
>
>   [PATCH 3/3] x86: Let extend_brk return a NULL pointer instead of
>
> makes extend_brk not do BUG_ON and instead just return NULL. This allows
> us to handle the bootup issues more gracefully.
>

NAK.  The whole point of the brk allocator is that users specify the 
upper limit on what they may need and stick to it.  Hence it is a fatal 
code bug if that is ever exceeded.  We want to catch those errors, not 
"handle" them.

This means you're either abusing the brk allocator to do something it is 
not meant to do... which may mean you can a failure in *other* code, or 
you have a bug in your code that you haven't fixed.

	-hpa

-- 
H. Peter Anvin, Intel Open Source Technology Center
I work for Intel.  I don't speak on their behalf.

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