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 3736EC001B0 for ; Sun, 16 Jul 2023 20:14:11 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231567AbjGPUOK (ORCPT ); Sun, 16 Jul 2023 16:14:10 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:40894 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231562AbjGPUOJ (ORCPT ); Sun, 16 Jul 2023 16:14:09 -0400 Received: from dfw.source.kernel.org (dfw.source.kernel.org [IPv6:2604:1380:4641:c500::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 42C8890 for ; Sun, 16 Jul 2023 13:14:09 -0700 (PDT) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id CD47B60E88 for ; Sun, 16 Jul 2023 20:14:08 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id E075FC433C8; Sun, 16 Jul 2023 20:14:07 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1689538448; bh=PnPAnvrqw/dVRWaLGv37cyp+zCHXUr/NVoBySgyapW4=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=KK5BMlVJ8Flgi2JX6+11VpGRrG4hreOvbFXW2O0XeVXrenFerIL090xuohpVPOn5K zXBa0QVRbKDxsN8PPrL6NbVrruZ7orpCmkISZjI2jaKlnBXGnnvzpZOL8ONUJ9yHgz LBANnvsJ2dbk/lTg98goABlvcN8yLm028gyGjuKg= From: Greg Kroah-Hartman To: stable@vger.kernel.org Cc: Greg Kroah-Hartman , patches@lists.linux.dev, Fei Shao , Dan Carpenter , Stephen Boyd , Sasha Levin Subject: [PATCH 6.4 427/800] clk: Fix memory leak in devm_clk_notifier_register() Date: Sun, 16 Jul 2023 21:44:40 +0200 Message-ID: <20230716194959.005277851@linuxfoundation.org> X-Mailer: git-send-email 2.41.0 In-Reply-To: <20230716194949.099592437@linuxfoundation.org> References: <20230716194949.099592437@linuxfoundation.org> User-Agent: quilt/0.67 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: stable@vger.kernel.org From: Fei Shao [ Upstream commit 7fb933e56f77a57ef7cfc59fc34cbbf1b1fa31ff ] devm_clk_notifier_register() allocates a devres resource for clk notifier but didn't register that to the device, so the notifier didn't get unregistered on device detach and the allocated resource was leaked. Fix the issue by registering the resource through devres_add(). This issue was found with kmemleak on a Chromebook. Fixes: 6d30d50d037d ("clk: add devm variant of clk_notifier_register") Signed-off-by: Fei Shao Link: https://lore.kernel.org/r/20230619112253.v2.1.I13f060c10549ef181603e921291bdea95f83033c@changeid Reviewed-by: Dan Carpenter Signed-off-by: Stephen Boyd Signed-off-by: Sasha Levin --- drivers/clk/clk.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/clk/clk.c b/drivers/clk/clk.c index e495dd7a1eae1..8c13bcf57f1ae 100644 --- a/drivers/clk/clk.c +++ b/drivers/clk/clk.c @@ -4696,6 +4696,7 @@ int devm_clk_notifier_register(struct device *dev, struct clk *clk, if (!ret) { devres->clk = clk; devres->nb = nb; + devres_add(dev, devres); } else { devres_free(devres); } -- 2.39.2