From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755449AbbCBV1D (ORCPT ); Mon, 2 Mar 2015 16:27:03 -0500 Received: from mail.linuxfoundation.org ([140.211.169.12]:43624 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753862AbbCBV07 (ORCPT ); Mon, 2 Mar 2015 16:26:59 -0500 Date: Mon, 2 Mar 2015 13:26:57 -0800 From: Andrew Morton To: Kees Cook Cc: Russell King , Catalin Marinas , Will Deacon , Ralf Baechle , Benjamin Herrenschmidt , Paul Mackerras , Michael Ellerman , Martin Schwidefsky , Heiko Carstens , linux390@de.ibm.com, x86@kernel.org, Alexander Viro , Oleg Nesterov , Andy Lutomirski , "David A. Long" , Andrey Ryabinin , Arun Chandran , Yann Droneaud , Min-Hua Chen , Paul Burton , Alex Smith , Markos Chandras , Vineeth Vijayan , Jeff Bailey , Michael Holzheu , Ben Hutchings , Behan Webster , Ismael Ripoll , Hector Marco-Gisbert , Jan-Simon =?ISO-8859-1?Q?M=F6ller?= , linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-mips@linux-mips.org, linuxppc-dev@lists.ozlabs.org, linux-s390@vger.kernel.org, linux-fsdevel@vger.kernel.org Subject: Re: [PATCH 0/5] split ET_DYN ASLR from mmap ASLR Message-Id: <20150302132657.5507742399620e70dd0a3926@linux-foundation.org> In-Reply-To: <1425006434-3106-1-git-send-email-keescook@chromium.org> References: <1425006434-3106-1-git-send-email-keescook@chromium.org> X-Mailer: Sylpheed 3.4.1 (GTK+ 2.24.23; x86_64-pc-linux-gnu) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, 26 Feb 2015 19:07:09 -0800 Kees Cook wrote: > This separates ET_DYN ASLR from mmap ASLR, as already done on s390. The > various architectures that are already randomizing mmap (arm, arm64, mips, > powerpc, s390, and x86), have their various forms of arch_mmap_rnd() > made available via the new CONFIG_ARCH_HAS_ELF_RANDOMIZE. For these > architectures, arch_randomize_brk() is collapsed as well. > > This is an alternative to the solutions in: > https://lkml.org/lkml/2015/2/23/442 "504 Gateway Time-out" Hector's original patch had very useful descriptions of the bug, why it occurred, how it was exploited it and how the patch fixes it. Your changelogs contain none of this and can be summarized as "randomly churn code around for no apparent reason". Wanna try again? I guess the [0/5] and [4/5] changelogs are the ones to fix.