Received: by 2002:a05:6358:e9c4:b0:b2:91dc:71ab with SMTP id hc4csp4467658rwb; Mon, 8 Aug 2022 01:07:55 -0700 (PDT) X-Google-Smtp-Source: AA6agR5eujfElKCijMKEJslknKmOLH+g9G7FhumPYAHpN/IHWNrfxtUPSNrCsdf13h/MuqvpZr7G X-Received: by 2002:a05:6402:90d:b0:43b:9ee5:f7eb with SMTP id g13-20020a056402090d00b0043b9ee5f7ebmr16600866edz.71.1659946074844; Mon, 08 Aug 2022 01:07:54 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1659946074; cv=none; d=google.com; s=arc-20160816; b=kMYMxKfQb8RyBW9LSJ6/YsK+fbFX1+ePcKljkQNcxe2dcf1odxAFVtBNhaMmHi/seC lp9znCLK8e3cAJyJWC66QjJHYxXj0sRSsWhzAe+ajZ3Fmju/yFrB9A70cK4OlQOt4xCw 0S3dRcIra2Fa/6SltSNYMhWtj2t1tq/SGWYJMW0L+7ekVUUTXAtMZk2y9vTujOVdvYNi o4RGFfNqvL17Lt3b5QHKeel3YcOiGGQcJSNsRsgTw6/ZUftBEtpk4rkukT4YJO1k8Whm SkjIAtQljdq80KxnCTW0cPx8GodpMuh8/tRicZN6n2EAuppTXA7GdiybO0mby1Jjl2KL SitQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:in-reply-to:content-disposition:mime-version :references:message-id:subject:cc:to:from:date; bh=CvkdoaphDIh2kAjFOwdBsGPuww5bCPL2sf/T/Zjnnx8=; b=LqyGHWpPTMDj754aWtdm1DXMRoI2+gKGGlnhLrQVDvXRCe/q2dw+VdfV/Q1X9/k8F0 OC6zlWnB8ZlDEhZM2l4Uxn1yi+ulruDhVXAP9f2s2zIV/8OGjLZcj9cYGfsRnHaAjMFa pqTueohiRaDFKgv+KsupeZNcRmPnEBv9GHJtemTP1Npdr4fqT1EQnniK89eYuBlqzpn8 graTYSPYynsJ427PEzIh+21nmLozwWBf2V4e7XU0IeASoyo0W7cDEWCHt3a3I8qVZFo/ bUPT1q8tbDJ32PMUyjrvhKDTlI2+Y06qQpU9iFGxbzOmmeN7KlHYqrSL2zTuh8B7GETW MjXg== 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 12-20020a170906308c00b0072f66504cc1si9226859ejv.334.2022.08.08.01.07.27; Mon, 08 Aug 2022 01:07:54 -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 S237712AbiHHH5a (ORCPT + 99 others); Mon, 8 Aug 2022 03:57:30 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:59968 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S236762AbiHHH52 (ORCPT ); Mon, 8 Aug 2022 03:57:28 -0400 Received: from metis.ext.pengutronix.de (metis.ext.pengutronix.de [IPv6:2001:67c:670:201:290:27ff:fe1d:cc33]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 96AF413CFD for ; Mon, 8 Aug 2022 00:57:27 -0700 (PDT) Received: from gallifrey.ext.pengutronix.de ([2001:67c:670:201:5054:ff:fe8d:eefb] helo=bjornoya.blackshift.org) by metis.ext.pengutronix.de with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1oKxd6-0008Ov-7v; Mon, 08 Aug 2022 09:56:52 +0200 Received: from pengutronix.de (unknown [IPv6:2a01:4f8:1c1c:29e9:22:41ff:fe00:1400]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange ECDHE (P-256) server-signature RSA-PSS (4096 bits) server-digest SHA256) (Client did not present a certificate) (Authenticated sender: mkl-all@blackshift.org) by smtp.blackshift.org (Postfix) with ESMTPSA id 87AA1C45A6; Mon, 8 Aug 2022 07:56:46 +0000 (UTC) Date: Mon, 8 Aug 2022 09:56:45 +0200 From: Marc Kleine-Budde To: Fedor Pchelkin Cc: Oleksij Rempel , Robin van der Gracht , kernel@pengutronix.de, Oliver Hartkopp , "David S . Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , linux-can@vger.kernel.org, netdev@vger.kernel.org, linux-kernel@vger.kernel.org, ldv-project@linuxtesting.org, Alexey Khoroshilov Subject: Re: [PATCH] can: j1939: fix memory leak of skbs Message-ID: <20220808075645.qtgwu64mjc2rxnuc@pengutronix.de> References: <20220805150216.66313-1-pchelkin@ispras.ru> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="hiqec7dmmpfnqmy6" Content-Disposition: inline In-Reply-To: <20220805150216.66313-1-pchelkin@ispras.ru> X-SA-Exim-Connect-IP: 2001:67c:670:201:5054:ff:fe8d:eefb X-SA-Exim-Mail-From: mkl@pengutronix.de X-SA-Exim-Scanned: No (on metis.ext.pengutronix.de); SAEximRunCond expanded to false X-PTX-Original-Recipient: linux-kernel@vger.kernel.org X-Spam-Status: No, score=-4.2 required=5.0 tests=BAYES_00,RCVD_IN_DNSWL_MED, SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=unavailable 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 --hiqec7dmmpfnqmy6 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On 05.08.2022 18:02:16, Fedor Pchelkin wrote: > We need to drop skb references taken in j1939_session_skb_queue() when > destroying a session in j1939_session_destroy(). Otherwise those skbs > would be lost. >=20 > Link to Syzkaller info and repro: https://forge.ispras.ru/issues/11743. >=20 > Found by Linux Verification Center (linuxtesting.org) with Syzkaller. >=20 > Fixes: 9d71dd0c7009 ("can: add support of SAE J1939 protocol") > Suggested-by: Oleksij Rempel > Signed-off-by: Fedor Pchelkin > Signed-off-by: Alexey Khoroshilov Added to can/master Thanks, Marc --=20 Pengutronix e.K. | Marc Kleine-Budde | Embedded Linux | https://www.pengutronix.de | Vertretung West/Dortmund | Phone: +49-231-2826-924 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | --hiqec7dmmpfnqmy6 Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQEzBAABCgAdFiEEBsvAIBsPu6mG7thcrX5LkNig010FAmLwwboACgkQrX5LkNig 010SAgf8DcRHAVx/USEkzkATwto/JOfRALIMKu8UHkPr+0JyLricXyQYn4NPMUjg 3dP3ZYZY3l5Pg5hCrto9W0T1tgn/9SxRR7ngIgqX46YNphbBgIr07DjWMOrB5Lv+ YnrdkYgWfBH2rx1f6BJfF+9Dmur3vLomVG7IpwemLDeWa4azE2f9gRnJwHLBbfGX RuvaBIgA+F6qAMXdU9RZv8d4/NGlI3gHQdvzrmNIZA79NEIVZMQA3DrZroN/W8t1 Hf/mJncv4eRYt38RcDoihu/CKEbS3C4gq2olzLN4xA1h+jUDJsDNdBHDHfd0w1fo /3KKaSQleo2KY+k8pFkGm3LivhmhQQ== =TBCJ -----END PGP SIGNATURE----- --hiqec7dmmpfnqmy6--