From patchwork Wed Aug 18 06:59:27 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Shubhrajyoti Datta X-Patchwork-Id: 12443431 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-16.8 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER,INCLUDES_PATCH, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_GIT autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id DFF41C432BE for ; Wed, 18 Aug 2021 06:59:43 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id C39B961051 for ; Wed, 18 Aug 2021 06:59:43 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S238103AbhHRHAR (ORCPT ); Wed, 18 Aug 2021 03:00:17 -0400 Received: from mail-sn1anam02on2085.outbound.protection.outlook.com ([40.107.96.85]:30980 "EHLO NAM02-SN1-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S238080AbhHRHAQ (ORCPT ); Wed, 18 Aug 2021 03:00:16 -0400 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=eDtmheBf6fdz7vdFkYCYF2845CafCI1f4ula0lmzpczq0v9wCDCPQbYFdunI1MTP+xJSNDPV7aNZDOm1T6PFEiNZmauFuyQVDUyisdPxS8YXC76DDqoDQkIX0FUhIZHvVbIo9HiPYTaSSFgcsR3OH0eAuWeybjwPVp6H8lW+2KDPXG1PS7B2IR20vHOi6HpO1PgqyvP1+fQbt7jg7aOTlAfRAbAem+TUaf24Dsk9Ws6mRcyc3tx98v0j9IXgZaBhdNv7QFWZg7dHxkSu4t9zC4JJOA1zSirpR3F1IL0fAms+HNNCqZ1sM+MuML4EVlJcj+scm6WZDKehWEiwCsJ3hA== 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-SenderADCheck; bh=qb25QNsNW9YWKZ67Aj0TmgV0keVwo19yuIqq7FWTjwc=; b=edVZhz9gMrkZWxqRiuxHrPwrUMWU390jFA18/CCUHdKlnkIC7DRouSZShmnRpqK0Gzj/maBOxGJydk28j2kS3HTiYOh+EBTQsEA2OeABdxcVsH9uFP1Hl4fygbu8ILsQWQFZn21RA+L0vZXc3lCfc34sroWzTthCo3PL+C8zgeFK0WukYEb+MOlyafr1cblzS7k/YGUUoe+xy+kemReB9UdAZL8ARTtTqXqF7lfVdYiXgDP3aVx9TYVCntUBHcbu3zKYkXmBelSXxlDd/b3MQODMKwTGY4ScCURaZXLeOYoE84GG0fbh8mmBCrGtnwBIbbNc0Z8OYA3gynY48R0qzA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 149.199.62.198) smtp.rcpttodomain=vger.kernel.org smtp.mailfrom=xilinx.com; dmarc=pass (p=none sp=none pct=100) action=none header.from=xilinx.com; dkim=none (message not signed); arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=xilinx.onmicrosoft.com; s=selector2-xilinx-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=qb25QNsNW9YWKZ67Aj0TmgV0keVwo19yuIqq7FWTjwc=; b=fCoj3zWsFxc40sHW0wNfkkgz/CMjsPlIOD/UK/30fCcoZCV90JMuSjxQXsSWuB2eynmu/W4r7tv34lIskLKCF6Fmdue+iT4VuMPtNIf1isc8FmIz7WHS4oBCxpSidM9hlNJ/e8e4XKAK1APm9iKD0ZWUFM69zMzzh0KglG3cMfE= Received: from SN7P222CA0005.NAMP222.PROD.OUTLOOK.COM (2603:10b6:806:124::22) by BYAPR02MB4152.namprd02.prod.outlook.com (2603:10b6:a02:fa::33) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4415.13; Wed, 18 Aug 2021 06:59:39 +0000 Received: from SN1NAM02FT0062.eop-nam02.prod.protection.outlook.com (2603:10b6:806:124:cafe::f9) by SN7P222CA0005.outlook.office365.com (2603:10b6:806:124::22) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4436.19 via Frontend Transport; Wed, 18 Aug 2021 06:59:38 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 149.199.62.198) smtp.mailfrom=xilinx.com; vger.kernel.org; dkim=none (message not signed) header.d=none;vger.kernel.org; dmarc=pass action=none header.from=xilinx.com; Received-SPF: Pass (protection.outlook.com: domain of xilinx.com designates 149.199.62.198 as permitted sender) receiver=protection.outlook.com; client-ip=149.199.62.198; helo=xsj-pvapexch01.xlnx.xilinx.com; Received: from xsj-pvapexch01.xlnx.xilinx.com (149.199.62.198) by SN1NAM02FT0062.mail.protection.outlook.com (10.97.4.69) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.20.4436.19 via Frontend Transport; Wed, 18 Aug 2021 06:59:38 +0000 Received: from xsj-pvapexch02.xlnx.xilinx.com (172.19.86.41) by xsj-pvapexch01.xlnx.xilinx.com (172.19.86.40) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2176.14; Tue, 17 Aug 2021 23:59:38 -0700 Received: from smtp.xilinx.com (172.19.127.96) by xsj-pvapexch02.xlnx.xilinx.com (172.19.86.41) with Microsoft SMTP Server id 15.1.2176.14 via Frontend Transport; Tue, 17 Aug 2021 23:59:38 -0700 Envelope-to: linux-clk@vger.kernel.org, sboyd@kernel.org, mturquette@baylibre.com Received: from [10.140.6.59] (port=53032 helo=xhdshubhraj40.xilinx.com) by smtp.xilinx.com with esmtp (Exim 4.90) (envelope-from ) id 1mGFY0-000GPx-6C; Tue, 17 Aug 2021 23:59:36 -0700 From: Shubhrajyoti Datta To: CC: , , , , Shubhrajyoti Datta Subject: [PATCH 1/3] clk: zynqmp: Check the return type Date: Wed, 18 Aug 2021 12:29:27 +0530 Message-ID: <20210818065929.12835-2-shubhrajyoti.datta@xilinx.com> X-Mailer: git-send-email 2.17.1 In-Reply-To: <20210818065929.12835-1-shubhrajyoti.datta@xilinx.com> References: <20210818065929.12835-1-shubhrajyoti.datta@xilinx.com> MIME-Version: 1.0 X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: a67542b6-c8ce-44e0-e3e4-08d96215bee0 X-MS-TrafficTypeDiagnostic: BYAPR02MB4152: X-Microsoft-Antispam-PRVS: X-Auto-Response-Suppress: DR, RN, NRN, OOF, AutoReply X-MS-Oob-TLC-OOBClassifiers: OLM:669; X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: lbMKSz/uyVjM0W7fa/3okiArgjhzSPAe5w3aMVogd3P9HvpYp/k0cLESEpZ13UoA795MGQwrM4CsYj02uqZIQYvFFp9Pigg5ZQCo4AMIKhB943SHHeu02FFee9T+CR77r8QKJSlrSeBPjzVui0nU5UNTOJLrujlZRTSH49MhtzD8/LGvcvZy9r/vKLWQs6QDaXG2CJUneBUNbImm9l7xZ++zSDMxD0BEq9KLK58wL6qbhBpN3gI0FDV4mMTIJA1E/KRpLIxY26YRunPV3f1wd+wzug8MMzk7YizyCMQMMilDxaoi3HIoaU3N4I0XTQzf07RQoPvWGG8yuNcSakE5fo0vne7zStJ48IsnTpBEn6OrjgZMvztJ9NpUrV9NvylzU6mTa4x1J9LNdMOSk3newU19WQ/xlUfBz4F6H9NK08G9NUdbuI1eB5uc9oS//tGI0U73+O6neaP2P8InqHx+ry6dCzonf7noQnYn0Jh0jkeHOHWVj/mK4g+KBcMXjNHXvKcGOfxsCq5Flr6NPGBFpurinh79LeIxzOEz5g6ixN2QNDo+g7PZ1S4S20fqTBwSLIhpc14YP//LwH3z+gxLTv6OFBhQbq6T5fypdCpWLY1ONtu1K+gPBGl8C3oR8F02Mej8eJ3wTyvoFJqIggQ1OnUgoKlQPIX5ln8r7HUwxbfsWWsFbD755SAa9Bc3cxXQd45EsXaY0LT8WmKr1af2j1NH7aj2OLUTScCgrf0M72Y= X-Forefront-Antispam-Report: CIP:149.199.62.198;CTRY:US;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:xsj-pvapexch01.xlnx.xilinx.com;PTR:unknown-62-198.xilinx.com;CAT:NONE;SFS:(4636009)(39860400002)(346002)(136003)(376002)(396003)(46966006)(36840700001)(70206006)(70586007)(8936002)(426003)(107886003)(36860700001)(6666004)(82740400003)(36906005)(6916009)(336012)(7696005)(8676002)(2616005)(5660300002)(7636003)(2906002)(1076003)(316002)(44832011)(478600001)(36756003)(9786002)(4326008)(356005)(26005)(82310400003)(4744005)(83380400001)(186003)(54906003)(47076005)(102446001);DIR:OUT;SFP:1101; X-OriginatorOrg: xilinx.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 18 Aug 2021 06:59:38.5050 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: a67542b6-c8ce-44e0-e3e4-08d96215bee0 X-MS-Exchange-CrossTenant-Id: 657af505-d5df-48d0-8300-c31994686c5c X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=657af505-d5df-48d0-8300-c31994686c5c;Ip=[149.199.62.198];Helo=[xsj-pvapexch01.xlnx.xilinx.com] X-MS-Exchange-CrossTenant-AuthSource: SN1NAM02FT0062.eop-nam02.prod.protection.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: BYAPR02MB4152 Precedence: bulk List-ID: X-Mailing-List: linux-clk@vger.kernel.org Currently the return value of of_clk_add_hw_provider is ignored. lets check and return value. Signed-off-by: Shubhrajyoti Datta --- drivers/clk/zynqmp/clkc.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/clk/zynqmp/clkc.c b/drivers/clk/zynqmp/clkc.c index 871184e406e1..eb25303eefed 100644 --- a/drivers/clk/zynqmp/clkc.c +++ b/drivers/clk/zynqmp/clkc.c @@ -762,9 +762,7 @@ static int zynqmp_clk_setup(struct device_node *np) zynqmp_register_clocks(np); zynqmp_data->num = clock_max_idx; - of_clk_add_hw_provider(np, of_clk_hw_onecell_get, zynqmp_data); - - return 0; + return of_clk_add_hw_provider(np, of_clk_hw_onecell_get, zynqmp_data); } static int zynqmp_clock_probe(struct platform_device *pdev) From patchwork Wed Aug 18 06:59:28 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Shubhrajyoti Datta X-Patchwork-Id: 12443433 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-16.8 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER,INCLUDES_PATCH, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_GIT autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 5562CC4338F for ; Wed, 18 Aug 2021 06:59:45 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 4152761056 for ; Wed, 18 Aug 2021 06:59:45 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S238109AbhHRHAS (ORCPT ); Wed, 18 Aug 2021 03:00:18 -0400 Received: from mail-bn7nam10on2062.outbound.protection.outlook.com ([40.107.92.62]:4256 "EHLO NAM10-BN7-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S238080AbhHRHAS (ORCPT ); Wed, 18 Aug 2021 03:00:18 -0400 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=MvtzguUkWYm9rjVhKwwfvzTokGr333E+OsdkT30jyDZoICwYX0h99HPVDx2uni6sIlH+tnuuc4mmyzZnyg6mKH3Y2iOg6aODKyLNphMDFrsmmhn4J2eiAZO7U+L//ysHXonC6ENB/BYcCbx3iHzIOc07TqbsgJJjiDwXMIrCaCKTdFSch+PdVSb/lTZtH5C5joSbWjXsX2B1KjO2HxAYnamyjEdYSTxe1QAiPCvGDMQMzyotLfdzBQZJL9qWR17Iz4V1NpmRmJdZdIXKebWXmLjbW3CN0D8TY1UJsR5T9TZA21RHLyHYYqJOWWNKTotCq7fJf+qupXMhq8U8uf917Q== 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-SenderADCheck; bh=cet0Nu5Ylop2oeRaVrV9oUq89/9IpmSGUC6qWjr7+6E=; b=OTBOe3EfcWeQgIXVEMKT+imBbEUqyP/k3b5HkCCC961OiGbee6+FBJ+nGb5kSQMwB6w/0v2eNQEUOApphuc48jrLj1h+IBO5ICAwWf+E7C3WOUzzsQ8LO0TqFZjPL9rlEIqtBkOe0O6YoqLl4RiC4iC6g75q2/hAM+oUgEidgtD3slVmNes7bQSx9iA6G9lM1ptvGx2a09EDPG5HByy1pahQHzra5qhy74uI3YGOlnI3qa2G792JYm0q+VPO0gN5L8dYmviMV6ifHsLLTvY5YwQPyxUP28l4g0td3bMQzmsNFw3x9r/5QIio04QsRyIjFoLHlcJWRry8A8LRPjF8/g== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 149.199.62.198) smtp.rcpttodomain=vger.kernel.org smtp.mailfrom=xilinx.com; dmarc=pass (p=none sp=none pct=100) action=none header.from=xilinx.com; dkim=none (message not signed); arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=xilinx.onmicrosoft.com; s=selector2-xilinx-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=cet0Nu5Ylop2oeRaVrV9oUq89/9IpmSGUC6qWjr7+6E=; b=WbbL9iKn7ASpuizV6W7AZQC3LetQW8Bnun9aqnQA0zA5PXTYoOPpRkg8Q8Zz7q+XWNBYfFRafDPRGbBbPs94mREHkCQHrLwtuIGwR9/+xEIh0RImu01rDFvSZr0F8NOCPBdZb2aTty1oek4n0fvjjJmafIg6JxYbhIZUuDZ4PlQ= Received: from BN0PR02CA0004.namprd02.prod.outlook.com (2603:10b6:408:e4::9) by BYAPR02MB4919.namprd02.prod.outlook.com (2603:10b6:a03:51::20) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4415.19; Wed, 18 Aug 2021 06:59:42 +0000 Received: from BN1NAM02FT010.eop-nam02.prod.protection.outlook.com (2603:10b6:408:e4:cafe::6a) by BN0PR02CA0004.outlook.office365.com (2603:10b6:408:e4::9) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4436.19 via Frontend Transport; Wed, 18 Aug 2021 06:59:41 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 149.199.62.198) smtp.mailfrom=xilinx.com; vger.kernel.org; dkim=none (message not signed) header.d=none;vger.kernel.org; dmarc=pass action=none header.from=xilinx.com; Received-SPF: Pass (protection.outlook.com: domain of xilinx.com designates 149.199.62.198 as permitted sender) receiver=protection.outlook.com; client-ip=149.199.62.198; helo=xsj-pvapexch02.xlnx.xilinx.com; Received: from xsj-pvapexch02.xlnx.xilinx.com (149.199.62.198) by BN1NAM02FT010.mail.protection.outlook.com (10.13.2.128) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.20.4436.19 via Frontend Transport; Wed, 18 Aug 2021 06:59:41 +0000 Received: from xsj-pvapexch02.xlnx.xilinx.com (172.19.86.41) by xsj-pvapexch02.xlnx.xilinx.com (172.19.86.41) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2176.14; Tue, 17 Aug 2021 23:59:40 -0700 Received: from smtp.xilinx.com (172.19.127.96) by xsj-pvapexch02.xlnx.xilinx.com (172.19.86.41) with Microsoft SMTP Server id 15.1.2176.14 via Frontend Transport; Tue, 17 Aug 2021 23:59:40 -0700 Envelope-to: linux-clk@vger.kernel.org, sboyd@kernel.org, mturquette@baylibre.com Received: from [10.140.6.59] (port=53032 helo=xhdshubhraj40.xilinx.com) by smtp.xilinx.com with esmtp (Exim 4.90) (envelope-from ) id 1mGFY2-000GPx-OH; Tue, 17 Aug 2021 23:59:39 -0700 From: Shubhrajyoti Datta To: CC: , , , , Shubhrajyoti Datta Subject: [PATCH 2/3] clk: zynqmp: Fix a memory leak Date: Wed, 18 Aug 2021 12:29:28 +0530 Message-ID: <20210818065929.12835-3-shubhrajyoti.datta@xilinx.com> X-Mailer: git-send-email 2.17.1 In-Reply-To: <20210818065929.12835-1-shubhrajyoti.datta@xilinx.com> References: <20210818065929.12835-1-shubhrajyoti.datta@xilinx.com> MIME-Version: 1.0 X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 1d310938-8ae8-43dc-596a-08d96215c0a7 X-MS-TrafficTypeDiagnostic: BYAPR02MB4919: X-Microsoft-Antispam-PRVS: X-Auto-Response-Suppress: DR, RN, NRN, OOF, AutoReply X-MS-Oob-TLC-OOBClassifiers: OLM:376; X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: sBPXr0Stl1UiPGxARnQXDVb7+oxqQkqJF2ZdNnO5CFEGiMfCvFDG9sGpMPvVOQnXRl0+coqtRwa9mRdGF29q3N4V50Mqh9rj6cD/ZRw8R7l72GO4Qns9uhfhwvYS7MFIn4Fi5IPEaNxz2rvcOQJOwoRezxeivjBaC2uyCx6aE3dX41My1ULsClfen5ltQnJa9teDmvJGA88tpm+HiLhHepS29KL1ghn2Bnu97fRBhODqBNOkafASlycilIQRTYs7015wqcgFh5eNECaXMmkbEUy9+/SB8D9S+VVhSFx82ilHpYj7qkheUXpbQKJyHWtsyoRrNQ6KnFEbmt41eghFM83QwV5/uGJsanDXEaxGNl5rXYf3n8vECRLn8eQSHwWf1VPO1H5NP6PhmODZODw+hjx6UWZtL0FUNNnz2Yw6qjfsG7plOJm039RedxRySVieGcsjJo2s9w9QnkctynNL1GurKLKUonS0wzkIm15/dMjwIJ2/EEpXMjVdjHBe6ncvyO1h4LeVKdUOjyrPbh+7i+R2Re2jFTHaeVliuUu+UleOEvRv0kN4DFkQX2rQq/kN4ZOqjnNSWy0cEVX8yrQ50oFMNK+CR9e7hJ2UeNVA2N6dyeEfEWgoi7dFSln/48eJ+C3fduQ4MAlOf/IiAqeAdcsbwQ9CKw3WdI/LGP4ewjNNf/Ds1C5xciv7YFTBu21/Sebs8f1QfLAOAQi9hETxebWJaTUj4RPzQjT33lFPD2M= X-Forefront-Antispam-Report: CIP:149.199.62.198;CTRY:US;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:xsj-pvapexch02.xlnx.xilinx.com;PTR:unknown-62-198.xilinx.com;CAT:NONE;SFS:(4636009)(376002)(136003)(396003)(346002)(39860400002)(36840700001)(46966006)(4744005)(4326008)(82310400003)(1076003)(83380400001)(36906005)(8676002)(26005)(9786002)(47076005)(6666004)(336012)(8936002)(107886003)(44832011)(2616005)(54906003)(426003)(316002)(82740400003)(356005)(36756003)(7636003)(2906002)(478600001)(5660300002)(70206006)(36860700001)(70586007)(7696005)(6916009)(186003)(102446001);DIR:OUT;SFP:1101; X-OriginatorOrg: xilinx.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 18 Aug 2021 06:59:41.4344 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 1d310938-8ae8-43dc-596a-08d96215c0a7 X-MS-Exchange-CrossTenant-Id: 657af505-d5df-48d0-8300-c31994686c5c X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=657af505-d5df-48d0-8300-c31994686c5c;Ip=[149.199.62.198];Helo=[xsj-pvapexch02.xlnx.xilinx.com] X-MS-Exchange-CrossTenant-AuthSource: BN1NAM02FT010.eop-nam02.prod.protection.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: BYAPR02MB4919 Precedence: bulk List-ID: X-Mailing-List: linux-clk@vger.kernel.org Fix a memory leak of mux. Signed-off-by: Shubhrajyoti Datta --- drivers/clk/zynqmp/clk-mux-zynqmp.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/clk/zynqmp/clk-mux-zynqmp.c b/drivers/clk/zynqmp/clk-mux-zynqmp.c index 157d4a960bf4..17afce594f28 100644 --- a/drivers/clk/zynqmp/clk-mux-zynqmp.c +++ b/drivers/clk/zynqmp/clk-mux-zynqmp.c @@ -159,7 +159,7 @@ struct clk_hw *zynqmp_clk_register_mux(const char *name, u32 clk_id, hw = &mux->hw; ret = clk_hw_register(NULL, hw); if (ret) { - kfree(hw); + kfree(mux); hw = ERR_PTR(ret); } From patchwork Wed Aug 18 06:59:29 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Shubhrajyoti Datta X-Patchwork-Id: 12443435 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-16.8 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER,INCLUDES_PATCH, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_GIT autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id EA60AC432BE for ; Wed, 18 Aug 2021 06:59:46 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id CF4F961051 for ; Wed, 18 Aug 2021 06:59:46 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S238267AbhHRHAU (ORCPT ); Wed, 18 Aug 2021 03:00:20 -0400 Received: from mail-dm6nam11on2041.outbound.protection.outlook.com ([40.107.223.41]:26721 "EHLO NAM11-DM6-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S238080AbhHRHAU (ORCPT ); Wed, 18 Aug 2021 03:00:20 -0400 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=ZfZd8kMSkQcZS8IVmdXn+mcOhGH9UR5wn7NzxRaXjfKFL5EWB4OI/gkQGMa/y32M3FAslrSTDFUnUJibUj6ZbsfNiCaQO4eFhdr+L8BaqZVJ/9F/9aFirbOMIpAEp+2dtDtL7ZAYUhEsRwvhKcuuGU5NQcZ/xQ2ve0zk1IiU12qIA0cVPyj+X3zjwv96VWqFb/sMZ0AgvRqzE0OpuASv6tuqIfC8hyM8OEBm3lsNdbfgCQa0mrL1skWdjS4xSjzWlSj3BUrrfCAt2lcT2qfOabHuu0TVVHBjTCc9yUKrJfqEU5Y0MtUMUBM1Y0niqnDRXXAqsXIOqGWILvltSG9/Zw== 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-SenderADCheck; bh=B7Ww3GellYG/6YcU3UQehzqHHZ4UStaSmUMYZ61hD1s=; b=AdBQzgm3PnuB+G6lDYCwy8zh28EfaUqcENFoWR6Gac8qoLkYIB6Iydb0WQxoSS3tdkJ34b2SN0piav8FE2gdiNo36k6uWt4PIidIv9NKq0GFlvRmixAdV3qtlLyEeZIV4RTIM/88esl99cT59dWeABMpxuxlgL3/70l18vaFQdvb3h9ZdZy6NZiRBIZRICTfgJ2QnLBgblSZUdAtD5MKZPuF5na6m0KyIHHQoH4tM4yvNM9HHEDguTeHagyCoopJ4GmIpjbxC0VPnOdN7GzaamzYqeeRUM1s60OmiTOHeL0e72daoWzRAiSOdud7pxEPW9E3/oWrvgc1ZLwApHgmxA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 149.199.62.198) smtp.rcpttodomain=vger.kernel.org smtp.mailfrom=xilinx.com; dmarc=pass (p=none sp=none pct=100) action=none header.from=xilinx.com; dkim=none (message not signed); arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=xilinx.onmicrosoft.com; s=selector2-xilinx-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=B7Ww3GellYG/6YcU3UQehzqHHZ4UStaSmUMYZ61hD1s=; b=WfwWT8uQJKUUQDqZEKnNdq/E8BQy7Uli8tjjs3T2WPVI8T/soJ2gT/oLYYGVajAtY/7K6gtN1Of4XVMeNifSLDaJmef/0aHteugzRMJrceH2ZRegxFlBuch22u8J7Fab9QQX35oeRGyiqNXi+lBpKMTLjKA6Mln4zNTXMhvqQqk= Received: from SA0PR12CA0021.namprd12.prod.outlook.com (2603:10b6:806:6f::26) by SJ0PR02MB7648.namprd02.prod.outlook.com (2603:10b6:a03:327::5) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4415.14; Wed, 18 Aug 2021 06:59:44 +0000 Received: from SN1NAM02FT0016.eop-nam02.prod.protection.outlook.com (2603:10b6:806:6f:cafe::7d) by SA0PR12CA0021.outlook.office365.com (2603:10b6:806:6f::26) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4436.19 via Frontend Transport; Wed, 18 Aug 2021 06:59:44 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 149.199.62.198) smtp.mailfrom=xilinx.com; vger.kernel.org; dkim=none (message not signed) header.d=none;vger.kernel.org; dmarc=pass action=none header.from=xilinx.com; Received-SPF: Pass (protection.outlook.com: domain of xilinx.com designates 149.199.62.198 as permitted sender) receiver=protection.outlook.com; client-ip=149.199.62.198; helo=xsj-pvapexch01.xlnx.xilinx.com; Received: from xsj-pvapexch01.xlnx.xilinx.com (149.199.62.198) by SN1NAM02FT0016.mail.protection.outlook.com (10.97.4.82) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.20.4436.19 via Frontend Transport; Wed, 18 Aug 2021 06:59:43 +0000 Received: from xsj-pvapexch02.xlnx.xilinx.com (172.19.86.41) by xsj-pvapexch01.xlnx.xilinx.com (172.19.86.40) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2176.14; Tue, 17 Aug 2021 23:59:43 -0700 Received: from smtp.xilinx.com (172.19.127.96) by xsj-pvapexch02.xlnx.xilinx.com (172.19.86.41) with Microsoft SMTP Server id 15.1.2176.14 via Frontend Transport; Tue, 17 Aug 2021 23:59:43 -0700 Envelope-to: linux-clk@vger.kernel.org, sboyd@kernel.org, mturquette@baylibre.com Received: from [10.140.6.59] (port=53032 helo=xhdshubhraj40.xilinx.com) by smtp.xilinx.com with esmtp (Exim 4.90) (envelope-from ) id 1mGFY5-000GPx-On; Tue, 17 Aug 2021 23:59:42 -0700 From: Shubhrajyoti Datta To: CC: , , , , Ian Nam , Rajan Vaja , Shubhrajyoti Datta Subject: [PATCH 3/3] clk: zynqmp: Fix stack-out-of-bounds in strncpy` Date: Wed, 18 Aug 2021 12:29:29 +0530 Message-ID: <20210818065929.12835-4-shubhrajyoti.datta@xilinx.com> X-Mailer: git-send-email 2.17.1 In-Reply-To: <20210818065929.12835-1-shubhrajyoti.datta@xilinx.com> References: <20210818065929.12835-1-shubhrajyoti.datta@xilinx.com> MIME-Version: 1.0 X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 390cea75-8fc0-4cd0-41cd-08d96215c222 X-MS-TrafficTypeDiagnostic: SJ0PR02MB7648: X-Microsoft-Antispam-PRVS: X-Auto-Response-Suppress: DR, RN, NRN, OOF, AutoReply X-MS-Oob-TLC-OOBClassifiers: OLM:4941; X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: Dl+tS2XzwMFkUt3t6mfF0GVq60+oWU1iPdc4B+ZEPM9rBHXhw6+xdboKUM2vnT3zFPl3Rp9LUEAkenJPAjA8iQscKBMpTvFkym0TAC7tbmhmnPwB2dIyFO5XxtZ/bm1VogcI76qE8H6EkMkHCfUxG+k1/piJhIoAvXqrNEdKlLC5b776qyyNzE5dhQ72tXt4uaZcbxEZFo96F3ReCHtF58rrhYor8Lmpm6FwFZd8kH1YaT7wtx4J3mHF7bIL4q/jJE80xm+aZ+3F/J1/uaxh9zSmjKi2ql27eZ4mWi8UC356hiNxOfqypuClbVQNPo7LXsGp55oP/gUPWP8iq3mlcjpR6qG9PVC8inLYRalVqDNiRkbJgtIVt82lSTWw4x+Qm21CTWklAvViQAY1WU5mEhv/6f0tCK8TtyWujWnWQnNsDn4Pdoazh8mcxXbIeREsIYBMUnlhaj+bnGhkMT5wk9wzTiQTvNFykTRCusaytRGEQaKcYaPY0vRTN80bOIIQAMeb4Es1ygVnLI54McbWCsRs3LwfAbrWh1MbA1mWk444R8ST4xVNJvBo5QAqkL1i+Fu/1vBKbVyoyQaqGLtgZEXDsx30DpBpFcBNiRalpdUN5HNAJ0Z5YXyvwXSjm9iYbfbVMOrvMoYU4wXHCBhYL9NvMZvfruRNuW1JAPfyehJQif3WQ2Bm0D9m5mrbF0DIOWwjhyz6oWyrrE2rQGio7FwxzK4aowXbEIFtv6jtyXObY61iBdmrRlLMP7GFPBkFvMhc5UQePEEkB5vPkI4lVXbgcAPc2uO8OiQIpmBOUAs= X-Forefront-Antispam-Report: CIP:149.199.62.198;CTRY:US;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:xsj-pvapexch01.xlnx.xilinx.com;PTR:unknown-62-198.xilinx.com;CAT:NONE;SFS:(4636009)(376002)(346002)(39860400002)(136003)(396003)(36840700001)(46966006)(83380400001)(36906005)(82740400003)(2906002)(478600001)(1076003)(8676002)(8936002)(7696005)(7636003)(26005)(54906003)(6666004)(426003)(47076005)(5660300002)(6916009)(316002)(9786002)(82310400003)(70206006)(70586007)(336012)(356005)(107886003)(4326008)(2616005)(44832011)(36756003)(36860700001)(186003)(102446001)(81973001)(505234006);DIR:OUT;SFP:1101; X-OriginatorOrg: xilinx.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 18 Aug 2021 06:59:43.9676 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 390cea75-8fc0-4cd0-41cd-08d96215c222 X-MS-Exchange-CrossTenant-Id: 657af505-d5df-48d0-8300-c31994686c5c X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=657af505-d5df-48d0-8300-c31994686c5c;Ip=[149.199.62.198];Helo=[xsj-pvapexch01.xlnx.xilinx.com] X-MS-Exchange-CrossTenant-AuthSource: SN1NAM02FT0016.eop-nam02.prod.protection.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: SJ0PR02MB7648 Precedence: bulk List-ID: X-Mailing-List: linux-clk@vger.kernel.org From: Ian Nam "BUG: KASAN: stack-out-of-bounds in strncpy+0x30/0x68" Linux-ATF interface is using 16 bytes of SMC payload. In case clock name is longer than 15 bytes, string terminated NULL character will not be received by Linux. Add explicit NULL character at last byte to fix issues when clock name is longer. This fixes below bug reported by KASAN: [ 7.522474] ================================================================== [ 7.529795] BUG: KASAN: stack-out-of-bounds in strncpy+0x30/0x68 [ 7.535871] Read of size 1 at addr ffff0008c89a7410 by task swapper/0/1 [ 7.542557] [ 7.544065] CPU: 1 PID: 1 Comm: swapper/0 Not tainted 5.4.0-00396-g81ef9e7-dirty #3 [ 7.551809] Hardware name: Xilinx Versal vck190 Eval board revA (QSPI) (DT) [ 7.558847] Call trace: [ 7.561321] dump_backtrace+0x0/0x1e8 [ 7.565023] show_stack+0x14/0x20 [ 7.568374] dump_stack+0xd4/0x108 [ 7.571817] print_address_description.isra.0+0xbc/0x37c [ 7.577189] __kasan_report+0x144/0x198 [ 7.581068] kasan_report+0xc/0x18 [ 7.584507] __asan_load1+0x5c/0x68 [ 7.588032] strncpy+0x30/0x68 [ 7.591120] zynqmp_clock_probe+0x238/0x7b8 [ 7.595350] platform_drv_probe+0x6c/0xc8 [ 7.599405] really_probe+0x14c/0x418 [ 7.603108] driver_probe_device+0x74/0x130 [ 7.607339] __device_attach_driver+0xc4/0xe8 [ 7.611744] bus_for_each_drv+0xec/0x150 [ 7.615711] __device_attach+0x160/0x1d8 [ 7.619678] device_initial_probe+0x10/0x18 [ 7.623907] bus_probe_device+0xe0/0xf0 [ 7.627785] device_add+0x528/0x950 [ 7.631312] of_device_add+0x5c/0x80 [ 7.634926] of_platform_device_create_pdata+0x120/0x168 [ 7.640299] of_platform_bus_create+0x244/0x4e0 [ 7.644880] of_platform_populate+0x50/0xe8 [ 7.649110] zynqmp_firmware_probe+0x370/0x3a8 [ 7.653602] platform_drv_probe+0x6c/0xc8 [ 7.657656] really_probe+0x14c/0x418 [ 7.661359] driver_probe_device+0x74/0x130 [ 7.665589] device_driver_attach+0x94/0xa0 [ 7.669820] __driver_attach+0x70/0x108 [ 7.673698] bus_for_each_dev+0xe4/0x158 [ 7.677664] driver_attach+0x30/0x40 [ 7.681278] bus_add_driver+0x21c/0x2b8 [ 7.685157] driver_register+0xbc/0x1d0 [ 7.689035] __platform_driver_register+0x7c/0x88 [ 7.693793] zynqmp_firmware_driver_init+0x1c/0x24 [ 7.698637] do_one_initcall+0xa4/0x234 [ 7.702518] kernel_init_freeable+0x1b0/0x24c [ 7.706924] kernel_init+0x10/0x110 [ 7.710450] ret_from_fork+0x10/0x18 [ 7.714058] [ 7.715559] The buggy address belongs to the page: [ 7.720405] page:ffff0008f9be1c88 refcount:0 mapcount:0 mapping:0000000000000000 index:0x0 [ 7.728772] raw: 0008d00000000000 ffff0008f9be1c90 ffff0008f9be1c90 0000000000000000 [ 7.736606] raw: 0000000000000000 0000000000000000 00000000ffffffff [ 7.742942] page dumped because: kasan: bad access detected [ 7.748572] [ 7.750076] addr ffff0008c89a7410 is located in stack of task swapper/0/1 at offset 112 in frame: [ 7.759052] zynqmp_clock_probe+0x0/0x7b8 [ 7.763103] [ 7.764604] this frame has 3 objects: [ 7.768306] [32, 44) 'response' [ 7.768312] [64, 80) 'ret_payload' [ 7.771573] [96, 112) 'name' [ 7.775095] [ 7.779585] Memory state around the buggy address: [ 7.784430] ffff0008c89a7300: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 [ 7.791735] ffff0008c89a7380: 00 00 00 00 f1 f1 f1 f1 00 04 f2 f2 00 00 f2 f2 [ 7.799040] >ffff0008c89a7400: 00 00 f3 f3 00 00 00 00 00 00 00 00 00 00 00 00 [ 7.806342] ^ [ 7.810132] ffff0008c89a7480: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 [ 7.817437] ffff0008c89a7500: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 [ 7.824738] ================================================================== Signed-off-by: Ian Nam Signed-off-by: Rajan Vaja Signed-off-by: Shubhrajyoti Datta --- drivers/clk/zynqmp/clkc.c | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/drivers/clk/zynqmp/clkc.c b/drivers/clk/zynqmp/clkc.c index eb25303eefed..2c9da6623b84 100644 --- a/drivers/clk/zynqmp/clkc.c +++ b/drivers/clk/zynqmp/clkc.c @@ -710,6 +710,13 @@ static void zynqmp_get_clock_info(void) FIELD_PREP(CLK_ATTR_NODE_INDEX, i); zynqmp_pm_clock_get_name(clock[i].clk_id, &name); + + /* + * Terminate with NULL character in case name provided by firmware + * is longer and truncated due to size limit. + */ + name.name[sizeof(name.name) - 1] = '\0'; + if (!strcmp(name.name, RESERVED_CLK_NAME)) continue; strncpy(clock[i].clk_name, name.name, MAX_NAME_LEN);