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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 084BAC07E97 for ; Fri, 1 Dec 2023 07:08:44 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1377710AbjLAHIf (ORCPT ); Fri, 1 Dec 2023 02:08:35 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:41462 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232478AbjLAHIW (ORCPT ); Fri, 1 Dec 2023 02:08:22 -0500 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 219FDF2 for ; Thu, 30 Nov 2023 23:08:14 -0800 (PST) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 4141FC433C8; Fri, 1 Dec 2023 07:08:13 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1701414493; bh=kkDlSi8LK9wnO0BxtoNklTwo0Po3lzBpozTfDh5KYnQ=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=DcvyVC/azyDijmRL8how/2ktMXVEwKIFt5n4L6TFDZ9/YbVs6t4DBQJGb3xWXQNw7 CThYeS4obQ4V9I/mInL+2uX5bMRuYkx1C7EXpUoQjE5zAtYb96Wu6svE1ppqEayKOK kH+frqVB83WDfjMVmYTBQNs/I6VCXLdIP0I7dagdGgNMI01jHBUT4dzH3uQ04Aoepz n0jYCcIWo0WdcN3ZnK/8NM7yCBX0LcKDOCd941LyLvKpELhfV5R3VmGFBZLLOrIaU+ LbI1B6Y3cByx5dO/oKqPn2iRH921YFJOnqF0QMeDde6/4n3skcm9FvT/OIFJT53KlL ZMxfwsEVqJa0A== Date: Thu, 30 Nov 2023 23:08:12 -0800 From: Jakub Kicinski To: Bert Karwatzki Cc: almasrymina@google.com, edumazet@google.com, hawk@kernel.org, pabeni@redhat.com, netdev@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] net: page_pool: Fix NULL pointer dereference in page_pool_unlist() Message-ID: <20231130230812.16461d0f@kernel.org> In-Reply-To: <20231130192542.4503-1-spasswolf@web.de> References: <20231130192542.4503-1-spasswolf@web.de> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, 30 Nov 2023 20:25:42 +0100 Bert Karwatzki wrote: > When the the hlist_node pool->user.list is in an in an unhashed state, > calling hlist_del() leads to a NULL pointer dereference. This happens > e. g. when rmmod'ing the mt7921e (mediatek wifi driver) kernel module. > An additional check fixes the issue. > > Fixes: 083772c9f972dc ("net: page_pool: record pools per netdev") > > Signed-off-by: Bert Karwatzki Thanks for the patch, we got a couple of similar submissions, and merged the first one to be posted.. should be fixed in net-next now. -- pw-bot: na