On 2022-11-29 13:01, Gaosheng Cui wrote:
The thunderbay_add_functions() will free memory of thunderbay_funcs when everything is ok, but thunderbay_funcs will not be freed when thunderbay_add_functions() fails, then there will be a memory leak, so we need to add kfree() when thunderbay_add_functions() fails to fix it. In addition, doing some cleaner works, moving kfree(funcs) from thunderbay_add_functions() to thunderbay_build_functions(). Fixes: 12422af8194d ("pinctrl: Add Intel Thunder Bay pinctrl driver") Signed-off-by: Gaosheng Cui <cuigaosheng1@xxxxxxxxxx>
FWIW: Reviewed-by: Rafał Miłecki <rafal@xxxxxxxxxx> (not sure about pinctrl-thunderbay.c removal plans)