Received: by 2002:a05:7412:1703:b0:e2:908c:2ebd with SMTP id dm3csp4127593rdb; Wed, 30 Aug 2023 17:13:12 -0700 (PDT) X-Google-Smtp-Source: AGHT+IHc8Ak+EkelFuCGv1Y6rx+9iqZNxHaDhrGanGxuQ+FKFTg3DjEgBtqQIqeaPRGpUABDgIXK X-Received: by 2002:a50:ed0d:0:b0:523:100b:462b with SMTP id j13-20020a50ed0d000000b00523100b462bmr3123324eds.5.1693440792635; Wed, 30 Aug 2023 17:13:12 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1693440792; cv=none; d=google.com; s=arc-20160816; b=V6sEBSGa64wONPj9Ir6xtDelpaspPsqUA1D//ICnh1tvfCvCdEPBqYCKHG4Izn6umc Vn4cMVNF2ayHyyAUM+2s2eklNbn/tyMOfosgH8+BjCD4mPNKkPdMTY8P8TGQt1KmfQ+f GLO85B9G1YA2JZJ/hODdFIEcUGdtflbsZ7mOanan3MIiCdQOkYDNhsgMmgkviBtvmD9O JeiF5EW7e+3RQgtBSS8yB5s/5/l9WztsTqoR6StIvyzG2XCob/rNAm4PwN9UY2dUWpus QoW3/Ia/0MAeknPSwjQowR8ajMYhacHSsFUu8r/tCfXTQiN6+W0+Jw5oFTqqYQHZTjw3 iuQw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:user-agent:in-reply-to:content-disposition :mime-version:references:message-id:subject:cc:to:from:date; bh=dXTdtQDPTdv1elZWQgI6y1h3Y/22saK2xPy+vo3lcJg=; fh=scKEg9vEpzR23LbUgdWAkEYdBUeexHvBh1pEcj7JV0o=; b=ggBmQTg8bvz0GXWXXv+S2wABG7iAVlAxso813awVm6ZVi3r+sNmpaTsnczUQljzt/Y th9pOQMa1mmBtqQGo512CH8V8lvba3M66eyV3Ecf5NqiRCH3U1/eXtgGBkHhWXgI6eOS kcXjxJzzQKsReCQmJl/CVwPcSeXVIBPapggG5+bHR7JsJl5XGABIGZLDqmLyPQVcihXV q7AlGDP0991jmdctEUo+8AUO1J+ze8UqKul30JY0kWqJ0JvOoPxejQEmXIibclb/nYuJ pztZSbKnPxu+czmEwVT41iC5UHmwRzcUWil/5ejMAC4UOT6q8MayJGqO6TsQRcnPf800 SGXA== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id n19-20020aa7c453000000b00523ae81218csi229382edr.242.2023.08.30.17.12.45; Wed, 30 Aug 2023 17:13:12 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1344288AbjH3XAB (ORCPT + 99 others); Wed, 30 Aug 2023 19:00:01 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:57034 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230350AbjH3XAA (ORCPT ); Wed, 30 Aug 2023 19:00:00 -0400 Received: from Chamillionaire.breakpoint.cc (Chamillionaire.breakpoint.cc [IPv6:2a0a:51c0:0:237:300::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 9D6EDE70; Wed, 30 Aug 2023 15:59:33 -0700 (PDT) Received: from fw by Chamillionaire.breakpoint.cc with local (Exim 4.92) (envelope-from ) id 1qbU9h-0004AB-TW; Thu, 31 Aug 2023 00:59:21 +0200 Date: Thu, 31 Aug 2023 00:59:21 +0200 From: Florian Westphal To: Wander Lairson Costa Cc: Pablo Neira Ayuso , Jozsef Kadlecsik , Florian Westphal , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , "open list:NETFILTER" , "open list:NETFILTER" , "open list:NETWORKING [GENERAL]" , open list , Lucas Leong , stable@kernel.org Subject: Re: [PATCH nf] netfilter/osf: avoid OOB read Message-ID: <20230830225921.GA15759@breakpoint.cc> References: <20230830205554.97083-2-wander@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20230830205554.97083-2-wander@redhat.com> User-Agent: Mutt/1.10.1 (2018-07-13) X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00, RCVD_IN_DNSWL_BLOCKED,SPF_HELO_PASS,SPF_PASS autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Wander Lairson Costa wrote: > The opt_num field is controlled by user mode and is not currently > validated inside the kernel. An attacker can take advantage of this to > trigger an OOB read and potentially leak information. [..] Can you send a v2 that rejects bogus nf_osf_user_finger structs? nfnl_osf_add_callback() seems to be the right place to refuse it.