From nobody Sat Jun 13 21:23:36 2026 Received: from mail-pl1-f173.google.com (mail-pl1-f173.google.com [209.85.214.173]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 749341F4611 for ; Tue, 5 May 2026 12:33:24 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.214.173 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1777984405; cv=none; b=ma2QsFOQ+6lXcabhLQ61H/VzuDbegL5EQEU+0gMkiU7J24nENGFgH9sw1bA3Md7i8myJ9vCBdTs+x7bphn7Ko1wQyT84+GKdHjKMtTyynCtBbANSiaOvHXIbEib6Gc3HEqWhNAOzIEWDfpzReBQ1/yuhf+Nyr5pelSybxkUfOvQ= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1777984405; c=relaxed/simple; bh=y3tW/Xv+dA6fK9zbbtZuPIX3+JTpxgxF57IG/or79zM=; h=From:To:Cc:Subject:Date:Message-Id:MIME-Version; b=eImuVLK56wERqJymhN7OSISCTkHImo3Z9KS/bSB3K+zAbuK1BfoPc37sjdB1HNr2ooS0Gjk2ABDpRNQGocTCM0YYn1Tv/2DoBs5UDi3eeMvmlO5LXJO2cB3MQJ1HJNO+VTntJpkqaIozJIV+IzEVeNxMvifcS7QIFCzOn0P74BI= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=SCL3nz+z; arc=none smtp.client-ip=209.85.214.173 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="SCL3nz+z" Received: by mail-pl1-f173.google.com with SMTP id d9443c01a7336-2b458ca2296so29669735ad.0 for ; Tue, 05 May 2026 05:33:24 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1777984404; x=1778589204; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=DvEm19d6uRu2W3J0XJxAC912UPvSOn8nmJ7XsYfi08Y=; b=SCL3nz+zjo3/UeiHlEDRIj7dDByjBjyN/y79R+tK9kjDxE+BZOKQL+ZgpH3BeX3HWP H+CYp4Mh/ex9cIxCeqCeoX1hwzcPHYwE9k4bJ1Bk/XOAjI0T0+CwHEEXLDvIEGJtpTqJ 6hapmV/bZAiyylRGl3RrlaSlS8cmzFDQY7ayttFVUnQzuRxOuehGJXlGqH7JaXV2bneI WRikYhH798nq/nwWnC67cBmNmthHMTvR/PC+b5OPoPt7ILsjY2P2qnKvCqnR+eFsfUFM sjgL5ZAxNh01T0iRS+PqP9Y3goQm47jyPnmsDmCE6ZihxD9Jw8KDqKriFVMJ3uvz8paZ US8w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1777984404; x=1778589204; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-gg:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=DvEm19d6uRu2W3J0XJxAC912UPvSOn8nmJ7XsYfi08Y=; b=icb2ccgkehsbBd93+nnMDIRoJDijbGyQr1Xsrnd0EI8XBOWSQnQAHGkrKvfGzQoaHt YcuZC0WnghoU6xg5+SXdQD+q56A3QMoEgJWcn7+O9C/9MOseUWSaEWz2CV3m56BhRg4H ep3/scFnRZhgOeE8HdrqN63VxGxt9w2PgBzSxAXyopAdxD9UXdAYULf8+aSrG1Q5BRPE 9MruqEq44QjS7CPnJyteumDThQxaO/nYOXbNtIBuFd+qj8z6Ru/bE73E5UEgcMmxa3bX 9exocbLY9Iq7WcX/wTHiGb4QB1dudAEeyAqD9wHnY3p6I+yFy6LxYIbn/S6878CEzpWZ To8g== X-Forwarded-Encrypted: i=1; AFNElJ/F1TGjrALkQOxkZrFdz4cYkeZVTuzswom9q5hA6orpwmj95Z3sQqk0YUbFuKylnXR3qlok9sbq8+CbkGo=@vger.kernel.org X-Gm-Message-State: AOJu0YxlXpU8XNrqTOzcyqSIP755v0wps7IbQiMIGwl07CyhxhLkFwah TqzUz311jZgsOdPYCtA9WjnJPnNIv33pcX4hZjE/aSWuVN+to9VqwLX5 X-Gm-Gg: AeBDiev1U3/oIacGc4g/mhtw8cL93lHFpIrF/k/RP6XKW/25/zkibIfAM+wV/Qr2eBl N4LhfOB3WZDHIUMJk9HxQJ1vFGZL+QkOaSIEV1ZBFTE1KWm5YkFIp5+VCxGJCcUvbVFE2+LslBp O8Pw3JHxpl5JvA8YPg35hU7dCw0G0BL+Gmz/KGwwRhGfADkciVgAnp2bYY0BGCxfdGoGru0gn3y EXbxSQDvijspv4e12dTp42RdFd++1/ev1Vl/dhfFwRdEdlMFkLv6fREbNxQQy3z+Nrjtbj81EOn HLI1sRGs8cjqucCzyUX+iCm7OfYaBIKDsvILmRiuPBIo4RWUiJqEMAoM+6KdUdGiOhTz0hXtm1y MwTEspTIdT1CK6T/v17xGYIXBgLas3LNle/re0cA+KowbfLcnZX5LMYtIjXauKpPcAmRU8DDVTb 5UuByLWJb8ArrQXjPWB9Vx3zs+vZ/i4HRKna7u5lk7wK+pZLBHxjyt+Ngckp/lBoTrMgy5eOAG X-Received: by 2002:a17:902:ea09:b0:2b9:4941:7f6e with SMTP id d9443c01a7336-2b9f256745fmr144236685ad.2.1777984403595; Tue, 05 May 2026 05:33:23 -0700 (PDT) Received: from localhost.localdomain ([115.110.225.242]) by smtp.gmail.com with ESMTPSA id d9443c01a7336-2b9caaaf0e5sm138853715ad.18.2026.05.05.05.33.19 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 05 May 2026 05:33:23 -0700 (PDT) From: Shitalkumar Gandhi X-Google-Original-From: Shitalkumar Gandhi To: =?UTF-8?q?Niklas=20S=C3=B6derlund?= Cc: Geert Uytterhoeven , Andrew Lunn , Jakub Kicinski , "David S . Miller" , Eric Dumazet , Paolo Abeni , Simon Horman , netdev@vger.kernel.org, linux-renesas-soc@vger.kernel.org, linux-kernel@vger.kernel.org, Shitalkumar Gandhi Subject: [PATCH net v2] net: rtsn: fix mdio_node leak in rtsn_mdio_alloc() Date: Tue, 5 May 2026 18:02:36 +0530 Message-Id: <20260505123236.406000-1-shitalkumar.gandhi@cambiumnetworks.com> X-Mailer: git-send-email 2.25.1 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" of_get_child_by_name() takes a reference. The rtsn_reset() and rtsn_change_mode() failure paths jump to out_free_bus and leak mdio_node. Add out_put_node to drop it before falling through. Fixes: b0d3969d2b4d ("net: ethernet: rtsn: Add support for Renesas Ethernet= -TSN") Signed-off-by: Shitalkumar Gandhi Reviewed-by: Geert Uytterhoeven Reviewed-by: Andrew Lunn Reviewed-by: Niklas S=C3=B6derlund --- Changes in v2: - Restore blank line between `return 0;` and `out_put_node:` label (Geert) - Add Reviewed-by: Geert Uytterhoeven Resent as a new thread (no code changes) so netdev CI picks it up (Andrew). Link to v1: https://lore.kernel.org/netdev/20260504200356.3529873-1-shitalk= umar.gandhi@cambiumnetworks.com/ Link to v2 (mis-threaded): https://lore.kernel.org/netdev/20260505085840.35= 2206-1-shitalkumar.gandhi@cambiumnetworks.com/ drivers/net/ethernet/renesas/rtsn.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/drivers/net/ethernet/renesas/rtsn.c b/drivers/net/ethernet/ren= esas/rtsn.c index 03a2669f0518..ee8381b60b8d 100644 --- a/drivers/net/ethernet/renesas/rtsn.c +++ b/drivers/net/ethernet/renesas/rtsn.c @@ -797,11 +797,11 @@ static int rtsn_mdio_alloc(struct rtsn_private *priv) /* Enter config mode before registering the MDIO bus */ ret =3D rtsn_reset(priv); if (ret) - goto out_free_bus; + goto out_put_node; =20 ret =3D rtsn_change_mode(priv, OCR_OPC_CONFIG); if (ret) - goto out_free_bus; + goto out_put_node; =20 rtsn_modify(priv, MPIC, MPIC_PSMCS_MASK | MPIC_PSMHT_MASK, MPIC_PSMCS_DEFAULT | MPIC_PSMHT_DEFAULT); @@ -824,6 +824,8 @@ static int rtsn_mdio_alloc(struct rtsn_private *priv) =20 return 0; =20 +out_put_node: + of_node_put(mdio_node); out_free_bus: mdiobus_free(mii); return ret; --=20 2.25.1