# HG changeset patch # User Dan Smith <danms(a)us.ibm.com&gt; # Date 1206553643 25200 # Node ID 055180f1b8538a497dac8272a02ff63c626f3190 # Parent c2b027700cb7a3075a9b9f56acd6d52cbd1f6fdf (#2) Make isMigratable return true if it makes it all the way through the checks, false otherwise. These should be independent of the return code and the status, which the patch fixes. Changes: - Removed stupidity Signed-off-by: Dan Smith <danms(a)us.ibm.com&gt; diff -r c2b027700cb7 -r 055180f1b853 src/Virt_VSMigrationService.c --- a/src/Virt_VSMigrationService.c Tue Mar 25 13:45:01 2008 -0700 +++ b/src/Virt_VSMigrationService.c Wed Mar 26 10:47:23 2008 -0700 @@ -543,7 +543,7 @@ static CMPIStatus vs_migratable(const CM CMPIStatus s; virConnectPtr conn = NULL; virConnectPtr dconn = NULL; - uint32_t retcode = 1; + uint32_t retcode = CIM_SVPC_RETURN_COMPLETED; CMPIBoolean isMigratable = 0; uint16_t type; virDomainPtr dom = NULL; @@ -598,14 +598,13 @@ static CMPIStatus vs_migratable(const CM goto out; } - retcode = CIM_SVPC_RETURN_COMPLETED; + isMigratable = 1; cu_statusf(_BROKER, &s, CMPI_RC_OK, ""); out: CMReturnData(results, (CMPIValue *)&retcode, CMPI_uint32); - isMigratable = (retcode == 0); CMAddArg(argsout, "IsMigratable", (CMPIValue *)&isMigratable, CMPI_boolean);