]> git.proxmox.com Git - mirror_ubuntu-hirsute-kernel.git/commit
mfd: syscon: Free the allocated name field of struct regmap_config
authorLimeng <Meng.Li@windriver.com>
Wed, 7 Apr 2021 05:25:25 +0000 (13:25 +0800)
committerStefan Bader <stefan.bader@canonical.com>
Mon, 20 Sep 2021 13:39:22 +0000 (15:39 +0200)
commitd3c900e86814c81fb6a97be028d6131231f0cdfd
treeb978b919eecff36a3e34717b4e44fd7acd781e75
parent8ee3a94bc6ce6a86eb41e1a827ea11ae4ac672e8
mfd: syscon: Free the allocated name field of struct regmap_config

BugLink: https://bugs.launchpad.net/bugs/1939450
commit 56a1188159cb2b87fbcb5a7a7afb38a4dd9db0c1 upstream.

The commit 529a1101212a("mfd: syscon: Don't free allocated name
for regmap_config") doesn't free the allocated name field of struct
regmap_config, but introduce a memory leak. There is another
commit 94cc89eb8fa5("regmap: debugfs: Fix handling of name string
for debugfs init delays") fixing this debugfs init issue from root
cause. With this fixing, the name field in struct regmap_debugfs_node
is removed. When initialize debugfs for syscon driver, the name
field of struct regmap_config is not used anymore. So, the allocated
name field of struct regmap_config is need to be freed directly after
regmap initialization to avoid memory leak.

Cc: stable@vger.kernel.org
Fixes: 529a1101212a("mfd: syscon: Don't free allocated name for regmap_config")
Signed-off-by: Meng Li <Meng.Li@windriver.com>
Signed-off-by: Lee Jones <lee.jones@linaro.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Kamal Mostafa <kamal@canonical.com>
Signed-off-by: Stefan Bader <stefan.bader@canonical.com>
drivers/mfd/syscon.c