From c1a1c7595249c91553e4d5033f9ac14ca242c20e Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Fabiano=20Fid=C3=AAncio?= Date: Thu, 19 Dec 2019 09:39:26 +0100 Subject: [PATCH] qemu: Don't check the output of virGetUserConfigDirectory() MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit virGetUserConfigDirectory() *never* *ever* returns NULL, making the checks for it completely unnecessary. Signed-off-by: Fabiano Fidêncio Reviewed-by: Ján Tomko --- src/qemu/qemu_conf.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/qemu/qemu_conf.c b/src/qemu/qemu_conf.c index 634c65095e..aa96d50e41 100644 --- a/src/qemu/qemu_conf.c +++ b/src/qemu/qemu_conf.c @@ -177,8 +177,7 @@ virQEMUDriverConfigPtr virQEMUDriverConfigNew(bool privileged) cfg->swtpmStateDir = g_strdup_printf("%s/swtpm", cfg->stateDir); - if (!(cfg->configBaseDir = virGetUserConfigDirectory())) - return NULL; + cfg->configBaseDir = virGetUserConfigDirectory(); cfg->libDir = g_strdup_printf("%s/qemu/lib", cfg->configBaseDir); cfg->saveDir = g_strdup_printf("%s/qemu/save", cfg->configBaseDir);