summaryrefslogtreecommitdiffstats
path: root/tests
diff options
context:
space:
mode:
authorAlex Chen2020-11-25 11:24:03 +0100
committerThomas Huth2020-12-09 08:04:34 +0100
commit43d1da7cb99ea66008583f531b584235d36fd0a3 (patch)
tree409d132a2fa875ad6fc4cdcf341349bbebbac899 /tests
parentgitlab-ci: Add Xen cross-build jobs (diff)
downloadqemu-43d1da7cb99ea66008583f531b584235d36fd0a3.tar.gz
qemu-43d1da7cb99ea66008583f531b584235d36fd0a3.tar.xz
qemu-43d1da7cb99ea66008583f531b584235d36fd0a3.zip
test-qga: fix a resource leak in test_qga_guest_get_osinfo()
The fixture->fd is created in fixture_setup() and, likewise, needs to be closed in fixture_tear_down(). Reported-by: Euler Robot <euler.robot@huawei.com> Signed-off-by: Alex Chen <alex.chen@huawei.com> Reviewed-by: Li Qiang <liq3ea@gmail.com> Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com> Message-Id: <20201125102403.57709-1-alex.chen@huawei.com> Signed-off-by: Thomas Huth <thuth@redhat.com>
Diffstat (limited to 'tests')
-rw-r--r--tests/test-qga.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/tests/test-qga.c b/tests/test-qga.c
index c1b173b3cb..eb33264e8e 100644
--- a/tests/test-qga.c
+++ b/tests/test-qga.c
@@ -111,6 +111,7 @@ fixture_tear_down(TestFixture *fixture, gconstpointer data)
g_rmdir(fixture->test_dir);
g_free(fixture->test_dir);
+ close(fixture->fd);
}
static void qmp_assertion_message_error(const char *domain,