From: Erik Skultety <eskultet@redhat.com>
Date: Wed, 21 Jan 2015 16:49:51 +0000 (+0100)
Subject: conf: virDomainDefMaybeAddController tweak return code
X-Git-Url: http://xenbits.xensource.com/gitweb?a=commitdiff_plain;h=852cea52ec2e0db63b561ca1fdffb5250eaf737e;p=libvirt.git

conf: virDomainDefMaybeAddController tweak return code

Previously the function returned either -1 in case of an error or 0 on
success. However, we should also distinguish between a case we
successfully added a controller and a case there wasn't a need to add any
controller
---

diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c
index 8792f5ee70..9ff38195cb 100644
--- a/src/conf/domain_conf.c
+++ b/src/conf/domain_conf.c
@@ -12416,7 +12416,7 @@ virDomainDefMaybeAddController(virDomainDefPtr def,
         return -1;
     }
 
-    return 0;
+    return 1;
 }