[PATCH] pinctrl: thunderbay: fix possible memory leak in thunderbay_add_functions()

[Gaosheng Cui <cuigaosheng1@xxxxxxxxxx>]

The memory of funcs is allocated by thunderbay_build_functions(),
thunderbay_add_functions() will free funcs when everything is ok,
but it will not be freed when thunderbay_add_functions() fails,
then there will be a memory leak, so add kfree() when
thunderbay_add_functions() fails to fix it.

Fixes: 25d2e41cf59b ("pinctrl: thunderbay: rework loops looking for groups names")
Signed-off-by: Gaosheng Cui <cuigaosheng1@xxxxxxxxxx>
---
 drivers/pinctrl/pinctrl-thunderbay.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/drivers/pinctrl/pinctrl-thunderbay.c b/drivers/pinctrl/pinctrl-thunderbay.c
index 9328b17485cf..10a7380af8e6 100644
--- a/drivers/pinctrl/pinctrl-thunderbay.c
+++ b/drivers/pinctrl/pinctrl-thunderbay.c
@@ -784,8 +784,10 @@ static int thunderbay_add_functions(struct thunderbay_pinctrl *tpc, struct funct
 
 		group_names = devm_kcalloc(tpc->dev, func->num_group_names,
 					   sizeof(*group_names), GFP_KERNEL);
-		if (!group_names)
+		if (!group_names) {
+			kfree(funcs);
 			return -ENOMEM;
+		}
 
 		for (j = 0; j < tpc->soc->npins; j++) {
 			const struct pinctrl_pin_desc *pin_info = &thunderbay_pins[j];
-- 
2.25.1