linux-kernel - Re: [PATCH 2/3] mmap: round mmap hint address above mmap_min

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: <1195249793.2924.99.camel@localhost.localdomain>
Date:	Fri, 16 Nov 2007 16:49:53 -0500
From:	Eric Paris <eparis@...hat.com>
To:	James Morris <jmorris@...ei.org>
Cc:	linux-kernel@...r.kernel.org, sds@...ho.nsa.gov,
	selinux@...ho.nsa.gov, alan@...hat.com, chrisw@...hat.com,
	hpa@...or.com, akpm@...ux-foundation.org
Subject: Re: [PATCH 2/3] mmap: round mmap hint address above mmap_min_addr

On Sat, 2007-11-17 at 08:42 +1100, James Morris wrote:
> On Fri, 16 Nov 2007, Eric Paris wrote:
> 
> > +#ifdef CONFIG_SECURITY
> > +	/*
> > +	 * If a hint addr is less than mmap_min_addr change addr to be as
> > +	 * low as possible but still greater than mmap_min_addr
> > +	 */
> > +	if (!(flags & MAP_FIXED) && ((void *)addr != NULL) &&
> > +	    (addr < mmap_min_addr))
> > +		addr = PAGE_ALIGN(mmap_min_addr);
> > +#endif
> 
> Please make this a static inline which is optimized away with 
> !CONFIG_SECURITY.

Where do you think these should go?  You think I should instead have a
static inline function in both mmap.c and nommu.c so the CONFIG check
isn't scattered in this code but still in this file?  Or did you
actually want something in security.h so I only have the code once?

-Eric

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