From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id D0AC2C4332F for ; Wed, 2 Feb 2022 15:44:50 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 369388D0111; Wed, 2 Feb 2022 10:44:50 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 2F0E78D00F9; Wed, 2 Feb 2022 10:44:50 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 16BB18D0111; Wed, 2 Feb 2022 10:44:50 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0200.hostedemail.com [216.40.44.200]) by kanga.kvack.org (Postfix) with ESMTP id 05D708D00F9 for ; Wed, 2 Feb 2022 10:44:50 -0500 (EST) Received: from smtpin08.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay05.hostedemail.com (Postfix) with ESMTP id B446A180AA9BF for ; Wed, 2 Feb 2022 15:44:49 +0000 (UTC) X-FDA: 79098262698.08.DFE9347 Received: from mail-out-4.itc.rwth-aachen.de (mail-out-4.itc.rwth-aachen.de [134.130.5.49]) by imf10.hostedemail.com (Postfix) with ESMTP id 3AF22C0002 for ; Wed, 2 Feb 2022 15:44:46 +0000 (UTC) X-IPAS-Result: =?us-ascii?q?A2AEAABmpvph/6QagoZaGgEBAQEBAQEBAQEDAQEBARIBA?= =?us-ascii?q?QEBAgIBAQEBQIFGBQEBAQELAYFRgixqhEmII4hznFOBfAsBAQEBAQEBAQEIA?= =?us-ascii?q?T8CBAEBhQUCg1sCJTQJDgECBAEBAQEDAgMBAQEBAQEDAQEGAQEBAQEBBQSBH?= =?us-ascii?q?IUvRoZDAQUjVhALDgoCAiYCAhAEMxYOBYYbAa5qgTGBAYhWgScJAYEGKgGHK?= =?us-ascii?q?4cugimEPz6HXYJlBJJDhDKRWoMvRpdAkmEHghBUZaARg3KSQAKRVJZKgkeje?= =?us-ascii?q?wIEAgQFAhaBYYIVMz6DOFEXAg+ccEEyOAIGCwEBAwmCOoMKJhOHTwEB?= IronPort-Data: A9a23:uKBOEaBAp+D6WxVW//ziw5YqxClBgxIJ4kV8jS/XYbTApDoggjIDn WtNWj+Gaf2PM2b3fdxzPNjn9BhXvpOAnINqOVdlrnsFo1CmCCbmLYnDch2gb3v6wunrFh8PA xA2M4GYRCwMZiaA4E/ra9ANlFEkvU2ybuOU5NXsZ2YhFWeIdA970Ug5w7Rh2NYy6TSEK1rlV e3a8pW31GCNhmYc3lI8s8pvfzs24ZweEBtB1rAPTagjUG32zhH5P7pDTU2FFEYUd6EPdgKMb 7uZkOvprjuxEyAFUbtJmp6jGqEDryW70QKm0hK6UID66vROS7BbPqsTbJIhhUlrZzqhlfc21 vBtuI6MEBoLL/2diPkRWAdZKnQrVUFG0OevzXmXq9OPz0DWNmCwhvwoFl4qPcgR9qB7DAmi9 9RBc2xLN0vbwbjohuvlFoGAhex6RCXvFIYWtXd91nfWF/E9WrjZXLnKoNZR1zc9gIZCEJ4yY uJAMmY/PUSdM0In1lE/LJR5gb7voUPGTCxihleqtLMIwTjs9VkkuFTqGJ+PEjCQfu1ckkGSv GPX9mLRDRQTNdjZwj2AmlqlhffKtSf6Xp8CUbO/6/hmiUGSwWpVDwcZPXO4rPSigUm5WPpUK 1YT/yszqO417kPDZsf8RRqQsnOCvwBaX9tWDv187xuCjLfXiy6UDGkJQjNbbfQ2sc4tXj0t0 BmCmNaBLThutqCFDHyG+rqKoDeaJycYNykBaDUCQA9D5MPsyKk3jxTSXpNmFYa2kNT+Gnf32 T/ihCw/gagDyM4Czam2+1HBjBqyqZXTCA04/APaWiSi9AwRWWK+T5a39VjW/bNbcMOQCEOeo HhBksTY4O1m4YyxqRFhid4lRNmBj8tp+hWF6bKzN/HNLwiQxkM= IronPort-HdrOrdr: A9a23:V2VN5KBGBMWHopDlHemx55DYdb4zR+YMi2TDGXoedfUzSL38qy nOpoV46faaslsssR0b9exoW5PwIk80l6QV3WB5B97LN2PbUQCTTL2Kg7GM/wHd X-IronPort-Anti-Spam-Filtered: true X-IronPort-AV: E=Sophos;i="5.88,337,1635199200"; d="scan'208";a="134898587" Received: from rwthex-s4-a.rwth-ad.de ([134.130.26.164]) by mail-in-4.itc.rwth-aachen.de with ESMTP; 02 Feb 2022 16:44:45 +0100 Received: from localhost (2a02:908:1069:d8e0:9e36:ff87:655a:5f05) by rwthex-s4-a.rwth-ad.de (2a00:8a60:1:e500::26:164) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.986.15; Wed, 2 Feb 2022 16:44:44 +0100 Date: Wed, 2 Feb 2022 16:44:44 +0100 From: Magnus =?utf-8?B?R3Jvw58=?= To: Alexey Dobriyan CC: Andrew Morton , Kees Cook , Alexander Viro , Eric Biederman , , , Subject: Re: [PATCH] elf: Relax assumptions about vaddr ordering Message-ID: References: <202201260845.FCBC0B5A06@keescook> <202201262230.E16DF58B@keescook> <202201281347.F36AEA5B61@keescook> <20220201144816.f84bafcf45c21d01fbc3880a@linux-foundation.org> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Disposition: inline In-Reply-To: User-Agent: Mutt/2.1.5 (31b18ae9) (2021-12-30) X-Originating-IP: [2a02:908:1069:d8e0:9e36:ff87:655a:5f05] X-ClientProxiedBy: RWTHEX-S2-B.rwth-ad.de (2a00:8a60:1:e500::26:155) To rwthex-s4-a.rwth-ad.de (2a00:8a60:1:e500::26:164) X-Stat-Signature: rjba5gbhu3imfob9ndxdox5w87md78at X-Rspam-User: nil Authentication-Results: imf10.hostedemail.com; dkim=none; dmarc=none; spf=pass (imf10.hostedemail.com: domain of magnus.gross@rwth-aachen.de designates 134.130.5.49 as permitted sender) smtp.mailfrom=magnus.gross@rwth-aachen.de X-Rspamd-Server: rspam01 X-Rspamd-Queue-Id: 3AF22C0002 X-HE-Tag: 1643816686-420781 X-Bogosity: Ham, tests=bogofilter, spamicity=0.000196, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: Am Wed, Feb 02, 2022 at 06:15:58PM +0300 schrieb Alexey Dobriyan: > On Tue, Feb 01, 2022 at 02:48:16PM -0800, Andrew Morton wrote: > > On Fri, 28 Jan 2022 14:30:12 -0800 Kees Cook wrote: > > > > > Andrew, can you update elf-fix-overflow-in-total-mapping-size-calculation.patch > > > to include: > > > > > > Fixes: 5f501d555653 ("binfmt_elf: reintroduce using MAP_FIXED_NOREPLACE") > > > Cc: stable@vger.kernel.org > > > Acked-by: Kees Cook > > > > Done. > > > > I'm taking it that we can omit this patch ("elf: Relax assumptions > > about vaddr ordering") and that Alexey's "ELF: fix overflow in total > > mapping size calculation" will suffice? > > Yes, it is same patch conceptually. > It should work, but those who can't play Bioshock are better test it. Yes it works. Although the change from unsigned int to int is not necessary in the first place, as you can avoid the -1 initialization for min_addr by simply using ULONG_MAX, as can be seen in my patch.