On 12/4/18 7:29 AM, Peter Maydell wrote: > When we add a new entry to the ARMCPRegInfo hash table in > add_cpreg_to_hashtable(), we allocate memory for tehe > ARMCPRegInfo struct itself, and we also g_strdup() the > name string. So the hashtable's value destructor function > must free the name string as well as the struct. > > Spotted by clang's leak sanitizer. The leak here is a > small one-off leak at startup, because we don't support > CPU hotplug, and so the only time when we destroy > hash table entries is for the case where ARM_CP_OVERRIDE > means we register a wildcard entry and then override it later. > > Signed-off-by: Peter Maydell <peter.mayd...@linaro.org> > --- > target/arm/cpu.c | 16 +++++++++++++++- > 1 file changed, 15 insertions(+), 1 deletion(-)
Reviewed-by: Richard Henderson <richard.hender...@linaro.org> r~
