From be956c4e38770dd61998815bbea4b7c2fc38a7d6 Mon Sep 17 00:00:00 2001 From: Andrea Bolognani Date: Mon, 20 Nov 2017 18:07:20 +0100 Subject: [PATCH] conf: Improve error handling in virDomainChrDefFormat() We don't need to store the return value since we never modify it. Signed-off-by: Andrea Bolognani Reviewed-by: Pavel Hrdina --- src/conf/domain_conf.c | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c index c0e239defb..cb98631fbc 100644 --- a/src/conf/domain_conf.c +++ b/src/conf/domain_conf.c @@ -24037,8 +24037,6 @@ virDomainChrDefFormat(virBufferPtr buf, const char *elementName = virDomainChrDeviceTypeToString(def->deviceType); bool tty_compat; - int ret = 0; - if (!elementName) { virReportError(VIR_ERR_INTERNAL_ERROR, _("unexpected char device type %d"), @@ -24059,14 +24057,14 @@ virDomainChrDefFormat(virBufferPtr buf, virDomainChrSourceDefFormat(buf, def->source, flags); if (virDomainChrTargetDefFormat(buf, def, flags) < 0) - return -1; + return -1; virDomainDeviceInfoFormat(buf, &def->info, flags); virBufferAdjustIndent(buf, -2); virBufferAsprintf(buf, "\n", elementName); - return ret; + return 0; } static int