Received: by 10.213.65.68 with SMTP id h4csp17440imn; Mon, 12 Mar 2018 15:44:03 -0700 (PDT) X-Google-Smtp-Source: AG47ELtf7RXfQZmuq00jqV6As9v8R/bpfKY+5ufEn33skIyT+bTpjnHkY7eYcJxQsQcPB9MAZbZH X-Received: by 10.99.116.85 with SMTP id e21mr4893230pgn.218.1520894643815; Mon, 12 Mar 2018 15:44:03 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1520894643; cv=none; d=google.com; s=arc-20160816; b=uJ2qjbL/mTxb1Fd6I5LbLZuESrLglDna3shJhLTNixcXDd6zgkgcQmfiy2bH8o6wa3 z28kVDT6rSPQftUAfzNDq/jOXatDoOh4LowmJDitzZ9Rvkxhtfoecw3rFRa/bAVY6PcL 65j5UhpVXUjuwH3psF7GRgj5sYvw0FBSMPbjZec3Kj89EwRkPXdlO8HMKtfMKPWIUqP2 ujTqfVIhNjzCAJhWFKi6foghG27j8LB0rqtvfidN6IBwYdJy7Ag8GBxDj7JFmlJdHYmf 57dAgCKqWiKKz2KGYfjNn5ieE2cx6/ax1A3QJStIijChApwlKLhef9DkaGXzlytqCsrG yhYw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding:mime-version :references:in-reply-to:message-id:subject:cc:to:from:date :arc-authentication-results; bh=L+GXTJB90MQAQ4WM21U8NbAdYctrDzvMlvQNpPL7zOs=; b=Io2XXWhFnh/ezXhCAhqrKLZByzpzPB/YLHVc1AvqogWPTWa8n54lfnTmOP4K3CQx1H pFcwglEN27/n9MIyAWYV/S/eo4gSwhip3HwtXhgfY0NUoeHmVpgWtRcSAYy8B9mbO12a +ag171WiFxsJzv6R0Orv7yWG50nn8jOWhZB24+3lnpejRp7tSIzTnVMdTtQq6VWFkCkk 7Y2BPrKT84aUBAD8fm9VQfInqrmxb/Lf7HvFmqdl3Tbsr7mLSCVKxB51MOB1Hy8uwGwC AKxcsRFl72thrTwq0txEV0w9TGnKP1f/RrNQyxghPAnTaPBVtfM2ZU2E1BZmwG1Bs7nn E/6g== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id g10-v6si6429520pln.20.2018.03.12.15.43.49; Mon, 12 Mar 2018 15:44:03 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751495AbeCLWmt (ORCPT + 99 others); Mon, 12 Mar 2018 18:42:49 -0400 Received: from mail.linuxfoundation.org ([140.211.169.12]:38358 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751282AbeCLWms (ORCPT ); Mon, 12 Mar 2018 18:42:48 -0400 Received: from akpm3.svl.corp.google.com (unknown [104.133.9.71]) by mail.linuxfoundation.org (Postfix) with ESMTPSA id 76B351172; Mon, 12 Mar 2018 22:42:47 +0000 (UTC) Date: Mon, 12 Mar 2018 15:42:45 -0700 From: Andrew Morton To: Waiman Long Cc: "Luis R. Rodriguez" , Kees Cook , linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org, Al Viro , Matthew Wilcox Subject: Re: [PATCH v4 2/6] proc/sysctl: Check for invalid flags bits Message-Id: <20180312154245.a3b759d2542ac7e09a5a267a@linux-foundation.org> In-Reply-To: <42c1606b-304e-8cd5-d532-68a403a99dce@redhat.com> References: <1520885744-1546-1-git-send-email-longman@redhat.com> <1520885744-1546-3-git-send-email-longman@redhat.com> <20180312135204.d56f7f1e64c96cf20786eb28@linux-foundation.org> <42c1606b-304e-8cd5-d532-68a403a99dce@redhat.com> X-Mailer: Sylpheed 3.6.0 (GTK+ 2.24.31; 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 Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, 12 Mar 2018 18:12:47 -0400 Waiman Long wrote: > On 03/12/2018 04:52 PM, Andrew Morton wrote: > > On Mon, 12 Mar 2018 16:15:40 -0400 Waiman Long wrote: > > > >> Checking code is added to check for invalid flags in the ctl_table > >> and return error if an unknown flag is used. > > Why? What's wrong with the old code, what value does this change add, > > etc. > > This is an additional checking code requested by Luis. Readers of this patch will wish to know why it exists. That doesn't tell us!