Received: by 2002:a25:1985:0:0:0:0:0 with SMTP id 127csp929216ybz; Wed, 22 Apr 2020 10:24:24 -0700 (PDT) X-Google-Smtp-Source: APiQypIo4FwSA5btDDBs52oWl0ZlqtWIFiMXs9lGrmJZxlPgMWVZxhWnNNLRgUGfmHV/LpziTOck X-Received: by 2002:a05:6402:688:: with SMTP id f8mr18151278edy.233.1587576263837; Wed, 22 Apr 2020 10:24:23 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1587576263; cv=none; d=google.com; s=arc-20160816; b=XKHVPbiT2JrshjEgThRsBCOgGnk2po8GNCP9ZwJM4dEAn+RVqxZLhJ815gHi/8wKHK q3KpN4ZuW1rGmwLg3vTQnc1f0ZRdvGpJTvmWouL6gMvjuU9P+BZ0eOONjnUg7jE3CWvb IEMqQ21bwXG/BNrlzBAv0OVeQYbfiKi9xd4b4lzFwq6CCylWgh1Wnr9VPr4fH8duqBVo LuycX3If3TijYxTMoiHJYBUrKFmuRmMRM8busuQGYUqA7eJeEfod5hUbKLUx3llUcnO6 LiWYcmpFBWp8GigfA5Llo7mm9e1Y/SeC9PEA2zrnPRLw1ufNGDHd4JVra0K7ziWuabqh HHRA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-disposition:mime-version:references:message-id:subject:cc :to:from:date; bh=G8dJ9x/l2pOK3f+lSdYk6E8cTAoJQruDcO+5ViRUU70=; b=tBCLdqDZAWC2m+yMpdAdx0cKsu0y2fSIzFRGCsR8g95JZyfE9OxKTbHglBUmnct1Dv wR0vivRWib6i9KtE2Afq0YtuMIGkcxmqbNStYlgEEaj+7s9MvY7Jg0neoUAqYLF4HW8g lFKNxo5Fn3wA/D1bKx/evtCgaHLEEiF0x3HwoV1DUOYd09W1XNzx4wqH+GPvJqbppQv/ 20o9h5YcUOhD9aM4YxWH2txCbw5iZnxyw6XktwvJcn3PNhfIKSNYMa8wUz6atKSqFiww ejHAVcMUF7aFo+b2Y9XmDZ1t2Of+ldAvjjtDJJubrcPHREsnb8N99dGjzzbXV40/It3j ovGA== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id w15si1145088edi.597.2020.04.22.10.23.59; Wed, 22 Apr 2020 10:24:23 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726854AbgDVRW6 (ORCPT + 99 others); Wed, 22 Apr 2020 13:22:58 -0400 Received: from verein.lst.de ([213.95.11.211]:53789 "EHLO verein.lst.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726057AbgDVRW6 (ORCPT ); Wed, 22 Apr 2020 13:22:58 -0400 Received: by verein.lst.de (Postfix, from userid 2407) id 7BAAA68C4E; Wed, 22 Apr 2020 19:22:54 +0200 (CEST) Date: Wed, 22 Apr 2020 19:22:54 +0200 From: Christoph Hellwig To: Andrey Ignatov Cc: Christoph Hellwig , Kees Cook , Iurii Zaikin , Alexei Starovoitov , Daniel Borkmann , linux-kernel@vger.kernel.org, linux-mm@kvack.org, linux-fsdevel@vger.kernel.org, netdev@vger.kernel.org, bpf@vger.kernel.org Subject: Re: [PATCH 5/5] sysctl: pass kernel pointers to ->proc_handler Message-ID: <20200422172254.GA30102@lst.de> References: <20200421171539.288622-1-hch@lst.de> <20200421171539.288622-6-hch@lst.de> <20200421192330.GA60879@rdna-mbp.dhcp.thefacebook.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20200421192330.GA60879@rdna-mbp.dhcp.thefacebook.com> User-Agent: Mutt/1.5.17 (2007-11-01) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Apr 21, 2020 at 12:23:30PM -0700, Andrey Ignatov wrote: > > if (ret == 1 && ctx.new_updated) { > > - *new_buf = ctx.new_val; > > + *buf = ctx.new_val; > > Original value of *buf should be freed before overriding it here > otherwise it's lost/leaked unless I missed something. > > Other than this BPF part of this patch looks good to me. Feel free to > add my Ack on the next iteration with this fix. Thanks, fixed. Can you also comment on "bpf-cgroup: remove unused exports" ?