Message ID | SG2PR06MB37436071F353CBAE29BEC0DABD2FA@SG2PR06MB3743.apcprd06.prod.outlook.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show
Return-Path: <linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org> X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 56FADEB64D9 for <linux-arm-kernel@archiver.kernel.org>; Wed, 5 Jul 2023 01:21:11 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:In-Reply-To:References: Message-ID:Date:Subject:CC:To:From:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=vj2xi5ZeAwl6LS6wNnk1/PMXuq9DG+2nWhXCIepofYQ=; b=kEBt6BLZ5jXpjw TgVntw2MOektcFulFvye5ToxOblrZG1+AoC9g77piu6yDzGRWwQB+p+9vNJyva8jwiQbaAdAoTq9/ NBwGSBzBBQl/eXP9iyt0avelE99DIhF8dFm9yD6UwXL67SdGilnBkY8JG9FFWPWvvSGbEcttPBmMn 2sO7eAwumBA+RVpyWUU8F8DVXIoJdKN/sJerNgqs1grGEYjeIBHPmEwQNE1arpk3Rw6b6b4XKwxYG bs42sPkToGzJ3m0WgoqyQ9OrXvllHFn8BKYzUGB8NSYJH+n+Wz8R3r+G3PNlOIUiGcd2V91FYK0z2 FcnbJzb5PcmkNxm+FRLg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qGrCD-00EWYz-10; Wed, 05 Jul 2023 01:20:41 +0000 Received: from mail-psaapc01on20729.outbound.protection.outlook.com ([2a01:111:f400:feae::729] helo=APC01-PSA-obe.outbound.protection.outlook.com) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1qGrCA-00EWYA-0t for linux-arm-kernel@lists.infradead.org; Wed, 05 Jul 2023 01:20:40 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=lPY6v2+5Qeuwxkuf7N6iHNa3wMNtRb6G+SOPregYiGSDIlbBZZFXvocAdQoDWL3oJ+973/6aqVRXfBcYd691K6uMl39JSqjUSELe/CO56EEFjNDo+58LR8H7smB9hV1jbtBjpsSit99xReWKVgH2x705hrmvMv5nK8TabJxEy32wIR+pQEOm7jYZOukeeABQXwepDhBVt58r+hqiaPCIC82OchF4VL/cGbTuc5G7fWJXssmIVt8PcsJVmsqwrUPWTdH/XkcNlv1HOH/3mZV9rX5N9V74J0cWAWvwCAdJ6r5YINIJyFdLPUmsfonzMlPvGM8b7bVGeRiPTfHLaZhCmQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=PV9TQaSpVPeibm9UlnYPYLTl3OdV724WdCHkC46DrkQ=; b=AkGnPT3yTMp0swCxLXafXm4tLXcmvd1JOYAsGAJH8VNofiLjs/s8Nw8qnggeU32a55bOAtzw8OMzJB1UByI32JnjsbPCdMbDIUQKvyNK8f/VMehqBpf2sv+RF0HLL8lWQZt5mxQCZ8S8nJCSK479eHDUTeANPrBMjI/Zvi9cemREqL9U85NvtyT7P41VxSNipiFgQdM4aVK0EYnhDpAba3uZ++XBdP1rAbiaHkDrSvCd55tewAr9qwgajBWlCWwVexiWkHi3DnWTt5eAhqeYKGkSf0Pg6otRBkDQDyo/0DR/RA9uhjxcA9k5VAncBYKE1ceobqK7s5DLlBd6jKb8eg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=vivo.com; dmarc=pass action=none header.from=vivo.com; dkim=pass header.d=vivo.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=vivo.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=PV9TQaSpVPeibm9UlnYPYLTl3OdV724WdCHkC46DrkQ=; b=dYtb1ZwCiWlg/4inptB6GZ2AyOMQkAjf9oGS0nYxt8xtjp7fSkTIy9p2/dw8pGkH3USkwpL73MuIbpJt6H7ud0hZ5PqRKSUxxnZg+RIzbr0qmb2p5234KyLzcuBXFs/YLawGseAA7krqw1iiF1nMet4nlcbuubUMdg+8B3yJuawzCSX2yccqRYtg1mBVyEHjZunVkgFG3+5LvdjyYjTi8DKYacBA6fdYVsYHyT04gtVWU8AThOLI0e7bFHLGT9mv/4Ouc6B6FEkwnHbqo+E5GCiJgxIPvLsdK61NM+UapfnWo4g0xtUumoULfIrv+7OUIC+GL5iChaGbIx4KrXKGjA== Received: from SG2PR06MB3743.apcprd06.prod.outlook.com (2603:1096:4:d0::18) by SEYPR06MB6132.apcprd06.prod.outlook.com (2603:1096:101:dd::6) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6544.24; Wed, 5 Jul 2023 01:20:23 +0000 Received: from SG2PR06MB3743.apcprd06.prod.outlook.com ([fe80::7dfd:a3ed:33ca:9cc8]) by SG2PR06MB3743.apcprd06.prod.outlook.com ([fe80::7dfd:a3ed:33ca:9cc8%6]) with mapi id 15.20.6544.024; Wed, 5 Jul 2023 01:20:23 +0000 From: =?eucgb2312_cn?b?zfXD9y3I7bz+tdey47y8yvWyvw==?= <machel@vivo.com> To: Simon Horman <simon.horman@corigine.com> CC: Sunil Goutham <sgoutham@marvell.com>, "David S. Miller" <davem@davemloft.net>, Eric Dumazet <edumazet@google.com>, Jakub Kicinski <kuba@kernel.org>, Paolo Abeni <pabeni@redhat.com>, "linux-arm-kernel@lists.infradead.org" <linux-arm-kernel@lists.infradead.org>, "netdev@vger.kernel.org" <netdev@vger.kernel.org>, "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>, opensource.kernel <opensource.kernel@vivo.com> Subject: =?eucgb2312_cn?b?u9i4tDogW1BBVENIIHYxXSBuZXQ6dGh1bmRlcl9iZ3g6Rml4IHJlc291?= =?eucgb2312_cn?b?cmNlIGxlYWtzIGluIGRldmljZV9mb3JfZWFjaF9jaGlsZF9ub2RlKCkgbG9vcHM=?= Thread-Topic: [PATCH v1] net:thunder_bgx:Fix resource leaks in device_for_each_child_node() loops Thread-Index: AQHZroHx9RfUdxaHXkuLRfcrmV6dGa+qD7qAgABQE5A= Date: Wed, 5 Jul 2023 01:20:23 +0000 Message-ID: <SG2PR06MB37436071F353CBAE29BEC0DABD2FA@SG2PR06MB3743.apcprd06.prod.outlook.com> References: <20230704141457.4844-1-machel@vivo.com> <ZKSBZW1a/f2H7H41@corigine.com> In-Reply-To: <ZKSBZW1a/f2H7H41@corigine.com> Accept-Language: zh-CN, en-US Content-Language: zh-CN X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=vivo.com; x-ms-publictraffictype: Email x-ms-traffictypediagnostic: SG2PR06MB3743:EE_|SEYPR06MB6132:EE_ x-ms-office365-filtering-correlation-id: 0c9419d1-f0f0-455a-261a-08db7cf60191 x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: UgKhNfgu/y9+QLnufem2jaDkgJxU30UP3ldOGzv9R8/7s3OYOc0j7KL+IPj5a9tfY6zgtVl9kAxp535gh4sbMgp/JdWZlU+ZEJgQND8as10oUfGGA6hH4/BmNCAhCvUtICQKpPe86LFwI3JeC8P2kOCF4o0V0he1zxbL3kR4GLyjVaDOYTaFa5huL8WcmEdLXNjDIGIGNIPFrbhJbXACsgDfbDYTmDpaaeXTs3o/rzNk0tNCxsRLH7alE4Pp3kD1RWkFdCt5jUUviclcpEkxdgYkLhSfVWdcVox6RQh1S4aeoLGrcEbsWEbzq9PUhHZOn1kFMTp7F2VUgvSCXSg534PMCYrfZohlsB9a8yov4QIHzTlZ4vVIgnvgwl6gjihJLOaoQp+4eleti//vBOA5U18pGIA/BMlpFijJk72G96QRuGhNfSgACyRYp+Yzpepf7YuSCTaUHV0rf25QByk3LooOwLaWgcWM4PId2VmpYfiPcYI+mAgMGcOCcHgL3n3XVyWCOtY9swKF8n1YJaOacQ6qftpFbOYl/FWGj8x3PuIf+gf4Ai3rxT7xPcEKwfk2bXxrEtA4RhvP2qoTpVJW97KYXWI+97yuFEBZUZ7Zs7o= x-forefront-antispam-report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:SG2PR06MB3743.apcprd06.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230028)(4636009)(366004)(136003)(39860400002)(396003)(376002)(346002)(451199021)(66556008)(64756008)(6506007)(76116006)(6916009)(66476007)(55016003)(4326008)(66946007)(66446008)(478600001)(41300700001)(107886003)(38100700002)(38070700005)(316002)(83380400001)(122000001)(52536014)(186003)(26005)(9686003)(54906003)(966005)(8936002)(224303003)(85182001)(2906002)(5660300002)(7696005)(86362001)(71200400001)(33656002);DIR:OUT;SFP:1102; x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?eucgb2312_cn?b?aHVDWWpNTWlnTThlaEwxOHQ3?= =?eucgb2312_cn?b?N2FjN2dwd0JKcDFmbTVoU0xyQjBxOSs0ZmU2Tlh1cU8zUUo0NWdsU2JUSURjVmRW?= =?eucgb2312_cn?b?VzM0WXZlN0NEcFRENkd4amRHUXJWRWRLSmxCQTUzcnBqN1p3b1FQb3JtZFJLaEtn?= =?eucgb2312_cn?b?QVBNYnBYTjltU2hiTkNQQnBCUkRaZDJVbFBXbG5iQ0ZERG0rUlVXZWN0QzVWNDMz?= =?eucgb2312_cn?b?SDZTWk9ZM1FvN3FtMFViU0llMXhONGd1cW1IbllUU1JzbDR5YndQZ3MwQjF0QXBn?= =?eucgb2312_cn?b?ZmxhZXZkVVZuZWhjUTk5cXBQQXEzLzRORlhxYmJTeC8yQ3JJckkrNkd6RFBhYmxO?= =?eucgb2312_cn?b?Q3BjWjVUWEh5OEpNNEtreEdyeUdxQnQwUlNTREFmNTJRenJrWXZMbXEyeXVwQWJk?= =?eucgb2312_cn?b?UWgrNUhUU1Zya2d2QVE5dTZHTnJYbmdOTHg0VkZlenhqVmllM2FPbloyZmkzQUs1?= =?eucgb2312_cn?b?R0swQkJRb2FRRnZJNmpzcHNkcmdiMnAxcVliYUxGZEMwWDdpSE9XSFhKUGZTdVVI?= =?eucgb2312_cn?b?MXh2MExKV3k2OGpCWm1kWG82WTlyN09SMXk2RFFaeHg4UWpzZHZvRm5obms3d2pR?= =?eucgb2312_cn?b?MDRnaXpFVUVNNUJKb01LTFkveEhVRThmMTEwRU9SdkNwNWpibThiN0o0OFlwcXVu?= =?eucgb2312_cn?b?SEFTTHo4QjM5bXFnMVZmNnl6MU5hQ1JScG5FMVNTZUVGUFlaTndiYW8zNkNuQU5x?= =?eucgb2312_cn?b?QjVVVTZYdURwVU1ra2t2RjFJYURVTVN0d1F2MkgzaUJ6R3FaK1dTeU9pRWozSngy?= =?eucgb2312_cn?b?emNFbUlnNFlRU2kybUNWM0dSZDFGVmxUZUYwckZrNTFPOWh4U20yVjN3cnVnejZ1?= =?eucgb2312_cn?b?MUw1UVdZNXBiTXRLTEJyMXpndCtoR2VkWUVDcTF2QnZpcnlQWTdnb1VFa28vVGly?= =?eucgb2312_cn?b?Ym5HWENFSWNVVkdPdFY5dXVpbWRZams2N0xLb2dCV0R0TU1iSHp6UnZqS1VHcHlM?= =?eucgb2312_cn?b?SU15dVFXN1hSdW9tbk5xTnVGOGduYnFDYmNWQ1ZqaFFCWkt3LzgrNmF0WWYxN2Fk?= =?eucgb2312_cn?b?V3NGNXU2ZitLUjUyUGRINDNTeGtiTWpEaGlleXMwMjUwQlNPOXhPNVZ5VUhRVTNZ?= =?eucgb2312_cn?b?R2ZTdktMUWV5SEZrbm1NT2ViZ1JWRjF2SVMraS9qTVZOb1cxc0tvWEw5ck9vcnBw?= =?eucgb2312_cn?b?QlVMcWFSSVU4UlF0UjRsRUlXYklaa0dyRVlGYXZXTHl1TDVaenNlSzU5MlBycXdk?= =?eucgb2312_cn?b?L21wYmdpaFpORWdUdzFyQzd6UkVGN1FZTFQwNG94aWNHc0pMQTRxK2ZiN0ROd0E4?= =?eucgb2312_cn?b?U2dUQWdUTTZUZGl0Q05JWHl3QnFra3pkQ3JGMVlLdU94dGpxY1RLVGk4eFJtckdM?= =?eucgb2312_cn?b?L0ZRTi9zU3pHa1pRZlhScDRMQmpPdGY2YXB6aWkzaEFvMzlFbkt5SHFqRDZhVTNB?= =?eucgb2312_cn?b?MTl6US9Paks2MTB3VVBlNVk4dGxvRUY5UlM4Q1hrMUM3czZTMmxaSUZDdk1CN1dy?= =?eucgb2312_cn?b?K3Y4c0htR0RubnJ5Y3A1bVh3V3BDbUV6dTF0L1VoTUhZRFdBSklFdUR5R2psRXlH?= =?eucgb2312_cn?b?Rkc2bmptTXB0V00ycmJBVjFjRXF3ekNLN05mek5BT3JnenRETFI5OGk0eUpmRkgx?= =?eucgb2312_cn?b?My9sVm9wOEhYeXI3MnZtU2F1NEd0bXhGSkZYNUhPbnMyUm56dEtYek5JbUJJM1Vi?= =?eucgb2312_cn?b?aWlrdHBIdmhEajBUakhiUFJVU3Brc29wZ3RSRzNseTU5djhVNjQzUlRKK2FQeWpr?= =?eucgb2312_cn?b?aW9tdkpyWFVJWFdIMmphZW1ra3hHUnVvSlZ6Rmcxc05DUUdCdk1kRVk1ZWw5d0Rl?= =?eucgb2312_cn?b?dFpkNnlZNVViZU83MDEzc2YxelNXUTR5TnIwRHRKenlSb216K3pnblFjRFJpSDBO?= =?eucgb2312_cn?b?V3M0Vy9ZL1Q5MjhpYzZySHVOWWZ2eHBGS0NZU3QwKzg4dFNEVU42T2w1bTd6bW1y?= =?eucgb2312_cn?b?cDJZV1pNekhhY3NJanNMTEdlODk3TjRrR2VmQk1hYy85TDlCQ1hxL0x5STVvR21C?= =?eucgb2312_cn?b?N3hQQTFXTHlvQ3krZENrNDd5TTZKOGowclJsUDZyVXBha1hSWjZWRjArMFFaeith?= =?eucgb2312_cn?b?NDl1bkw2b0tiVVNpU0JUVjl3bHNPZWxpUmpWNmpLamdPMDVzNE0rejVqU3FMcQ==?= MIME-Version: 1.0 X-OriginatorOrg: vivo.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: SG2PR06MB3743.apcprd06.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 0c9419d1-f0f0-455a-261a-08db7cf60191 X-MS-Exchange-CrossTenant-originalarrivaltime: 05 Jul 2023 01:20:23.2899 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 923e42dc-48d5-4cbe-b582-1a797a6412ed X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: uRILX02Rk+8olQyKW3nSVn0umwfHD1iAUXgPjgWaGyYYdRXs3QyO4LdIouz/JoEC0/rv3CnBib/gUg/NSzVd7w== X-MS-Exchange-Transport-CrossTenantHeadersStamped: SEYPR06MB6132 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230704_182038_439118_80DD1087 X-CRM114-Status: GOOD ( 24.03 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: <linux-arm-kernel.lists.infradead.org> List-Unsubscribe: <http://lists.infradead.org/mailman/options/linux-arm-kernel>, <mailto:linux-arm-kernel-request@lists.infradead.org?subject=unsubscribe> List-Archive: <http://lists.infradead.org/pipermail/linux-arm-kernel/> List-Post: <mailto:linux-arm-kernel@lists.infradead.org> List-Help: <mailto:linux-arm-kernel-request@lists.infradead.org?subject=help> List-Subscribe: <http://lists.infradead.org/mailman/listinfo/linux-arm-kernel>, <mailto:linux-arm-kernel-request@lists.infradead.org?subject=subscribe> Content-Type: text/plain; charset="gb2312" Content-Transfer-Encoding: base64 Sender: "linux-arm-kernel" <linux-arm-kernel-bounces@lists.infradead.org> Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org |
Series |
回复: [PATCH v1] net:thunder_bgx:Fix resource leaks in device_for_each_child_node() loops
|
expand
|
diff --git a/drivers/net/ethernet/cavium/thunder/thunder_bgx.c b/drivers/net/ethernet/cavium/thunder/thunder_bgx.c index a317feb8decb..3091c96134e4 100644 --- a/drivers/net/ethernet/cavium/thunder/thunder_bgx.c +++ b/drivers/net/ethernet/cavium/thunder/thunder_bgx.c @@ -1469,6 +1469,7 @@ static int bgx_init_of_phy(struct bgx *bgx) struct fwnode_handle *fwn; struct device_node *node = NULL; u8 lmac = 0; + int err = 0; device_for_each_child_node(&bgx->pdev->dev, fwn) { struct phy_device *pd; @@ -1479,7 +1480,7 @@ static int bgx_init_of_phy(struct bgx *bgx) */ node = to_of_node(fwn); if (!node) - break; + goto out_handle_put; of_get_mac_address(node, bgx->lmac[lmac].mac); @@ -1501,10 +1502,8 @@ static int bgx_init_of_phy(struct bgx *bgx) } lmac++; - if (lmac == bgx->max_lmac) { - of_node_put(node); - break; - } + if (lmac == bgx->max_lmac) + goto out_node_put; } return 0; @@ -1519,8 +1518,12 @@ static int bgx_init_of_phy(struct bgx *bgx) } lmac--; } + err = -EPROBE_DEFER; +out_node_put: of_node_put(node); - return -EPROBE_DEFER; +out_handle_put: + fwnode_handle_put(fwn); + return err; } #else