# HG changeset patch # User Dan Smith <danms(a)us.ibm.com&gt; # Date 1222277765 25200 # Node ID 61a72b41f401bbba89b6c503bf4629400fcce60f # Parent 06e1fa131671f9d0ecd76a89c927864a887f0b0d (#2) Fix return value of ModifySystemSettings Changes: - Fix silly compile error Signed-off-by: Dan Smith <danms(a)us.ibm.com&gt; diff -r 06e1fa131671 -r 61a72b41f401 src/Virt_VirtualSystemManagementService.c --- a/src/Virt_VirtualSystemManagementService.c Wed Sep 24 09:30:41 2008 -0700 +++ b/src/Virt_VirtualSystemManagementService.c Wed Sep 24 10:36:05 2008 -0700 @@ -1210,17 +1210,27 @@ CMPIArgs *argsout) { CMPIInstance *inst; + CMPIStatus s; + uint32_t rc; if (cu_get_inst_arg(argsin, "SystemSettings", &inst) != CMPI_RC_OK) { - CMPIStatus s; cu_statusf(_BROKER, &s, CMPI_RC_ERR_FAILED, "Missing SystemSettings"); - return s; + goto out; } - return update_system_settings(context, reference, inst); + s = update_system_settings(context, reference, inst); + out: + if (s.rc == CMPI_RC_OK) + rc = CIM_SVPC_RETURN_COMPLETED; + else + rc = CIM_SVPC_RETURN_FAILED; + + CMReturnData(results, &rc, CMPI_uint32); + + return s; } typedef CMPIStatus (*resmod_fn)(struct domain *,