From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Miller Subject: Re: [PATCH] rtnetlink: Remove VLA usage Date: Thu, 31 May 2018 22:49:26 -0400 (EDT) Message-ID: <20180531.224926.26417889797461444.davem@davemloft.net> References: <20180530222052.GA30622@beast> Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit Cc: fw@strlen.de, dsahern@gmail.com, netdev@vger.kernel.org, linux-kernel@vger.kernel.org To: keescook@chromium.org Return-path: In-Reply-To: <20180530222052.GA30622@beast> Sender: linux-kernel-owner@vger.kernel.org List-Id: netdev.vger.kernel.org From: Kees Cook Date: Wed, 30 May 2018 15:20:52 -0700 > In the quest to remove all stack VLA usage from the kernel[1], this > allocates the maximum size expected for all possible types and adds > sanity-checks at both registration and usage to make sure nothing gets > out of sync. This matches the proposed VLA solution for nfnetlink[2]. The > values chosen here were based on finding assignments for .maxtype and > .slave_maxtype and manually counting the enums: > > slave_maxtype (max 33): ... > maxtype (max 45): ... > > This additionally changes maxtype and slave_maxtype fields to unsigned, > since they're only ever using positive values. > > [1] https://lkml.kernel.org/r/CA+55aFzCG-zNmZwX4A2FQpadafLfEzK6CC=qPXydAacU1RqZWA@mail.gmail.com > [2] https://patchwork.kernel.org/patch/10439647/ > > Signed-off-by: Kees Cook Looks good, applied, thanks.