tests/qtest/migration-test: Use g_autofree to avoid leaks on error paths
commitff7b9b56cd7053eef0aaf40c32c2be475fd37df8
authorPeter Maydell <peter.maydell@linaro.org>
Thu, 6 May 2021 18:58:19 +0000 (6 19:58 +0100)
committerDr. David Alan Gilbert <dgilbert@redhat.com>
Thu, 13 May 2021 17:21:14 +0000 (13 18:21 +0100)
tree9329ae38b040186f64f0874673d9659d1c756cc5
parenta1209bb7107c850be2909393a8304ea7ae747137
tests/qtest/migration-test: Use g_autofree to avoid leaks on error paths

Coverity notices that several places in the migration-test code fail
to free memory in error-exit paths.  This is pretty unimportant in
test case code, but we can avoid having to manually free the memory
entirely by using g_autofree.

The places where Coverity spotted a leak were relating to early exits
not freeing 'uri' in test_precopy_unix(), do_test_validate_uuid(),
migrate_postcopy_prepare() and test_migrate_auto_converge().  This
patch converts all the string-allocation in the test code to
g_autofree for consistency.

Fixes: Coverity CID 1432313143231514323521432364
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Message-Id: <20210506185819.9010-1-peter.maydell@linaro.org>
Reviewed-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
Signed-off-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
tests/qtest/migration-test.c