Cc: qemu-trivial <[email protected]> writes:
> From: Pan Nengyuan <[email protected]> > > spotted by ASAN > > Fixes: 81d8ccb1bea4fb9eaaf4c8e30bd4021180a9a39f > Reported-by: Euler Robot <[email protected]> > Signed-off-by: Pan Nengyuan <[email protected]> > --- > util/module.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/util/module.c b/util/module.c > index e9fe3e5..8c5315a 100644 > --- a/util/module.c > +++ b/util/module.c > @@ -214,6 +214,7 @@ bool module_load_one(const char *prefix, const char > *lib_name) > > if (!success) { > g_hash_table_remove(loaded_modules, module_name); > + g_free(module_name); > } > > for (i = 0; i < n_dirs; i++) { Reviewed-by: Markus Armbruster <[email protected]> Reviewing this made me wonder: @loaded_modules is global state, but there appears to be no synchronization. What's the safety argument?
