Re: [PATCH] hw/misc/npcm7xx_clk: Don't leak string in npcm7xx_clk_sel_init()

[Hao Wu]

On Tue, Mar 8, 2022 at 10:09 AM Richard Henderson <richard.henderson@linaro.org> wrote:

On 3/8/22 07:03, Peter Maydell wrote:
> In npcm7xx_clk_sel_init() we allocate a string with g_strdup_printf().
> Use g_autofree so we free it rather than leaking it.
>
> (Detected with the clang leak sanitizer.)
>
> Signed-off-by: Peter Maydell<peter.maydell@linaro.org>
> ---
>   hw/misc/npcm7xx_clk.c | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)

Reviewed-by: Richard Henderson <richard.henderson@linaro.org>

Reviewed-by: Hao Wu <wuhaotsh@google.com> 

r~