diff options
author | Alex Chen | 2020-11-25 11:24:03 +0100 |
---|---|---|
committer | Thomas Huth | 2020-12-09 08:04:34 +0100 |
commit | 43d1da7cb99ea66008583f531b584235d36fd0a3 (patch) | |
tree | 409d132a2fa875ad6fc4cdcf341349bbebbac899 /tests | |
parent | gitlab-ci: Add Xen cross-build jobs (diff) | |
download | qemu-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.c | 1 |
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, |