On Sun, 6 Sept 2026 at 09:54, Marc-André Lureau <[email protected]> wrote: > > g_free() isn't the appropriate way to release those types. > > Fixes: 0b70c6ce7b31 ("Add functional and unit tests for the vm-launch-update > device") > Signed-off-by: Marc-André Lureau <[email protected]> > --- > tests/qtest/launchupdate-test.c | 5 ++--- > 1 file changed, 2 insertions(+), 3 deletions(-) > > diff --git a/tests/qtest/launchupdate-test.c b/tests/qtest/launchupdate-test.c > index 225c843df5b9..ed007cdce29f 100644 > --- a/tests/qtest/launchupdate-test.c > +++ b/tests/qtest/launchupdate-test.c > @@ -192,8 +192,8 @@ static guint32 match_string(char *serial_f, const char > *exp_out) > { > GError *error = NULL; > g_autofree gchar *f_contents = NULL; > - g_autofree GRegex *regex = NULL; > - g_autofree GMatchInfo *match_info = NULL; > + g_autoptr(GRegex) regex = NULL; > + g_autoptr(GMatchInfo) match_info = NULL; > gsize len; > guint32 count = 0; > gboolean ret; > @@ -214,7 +214,6 @@ static guint32 match_string(char *serial_f, const char > *exp_out) > g_match_info_next(match_info, &error); > count++; > } > - g_regex_unref(regex); > return count; > }
Reviewed-by: Peter Maydell <[email protected]> thanks -- PMM
