[Libvirt-cim] [PATCH 2 of 3] Removed error messages that cause wrong error returns
# HG changeset patch
# User Heidi Eckhart <heidieck(a)linux.vnet.ibm.com>
# Date 1196333496 -3600
# Node ID a68a67f532bca09201092ba1e8e6c9c688c0a242
# Parent 393cc9ffc3e218d537618d314ca4eb6120fc6043
Removed error messages that cause wrong error returns
The case that no handler was found for a request caused an
error as return. This is wrong as the not-found handler
tells only, that the provider is not responsible for the
given request. In scenarios with a general request causing
the call of several association providers that return
valid instances, the former behavior causes a wrong
result, as the CIMOM would return the error and revoke all
valid results.
Signed-off-by: Heidi Eckhart <heidieck(a)linux.vnet.ibm.com>
diff -r 393cc9ffc3e2 -r a68a67f532bc std_association.c
--- a/std_association.c Thu Nov 29 11:51:28 2007 +0100
+++ b/std_association.c Thu Nov 29 11:51:36 2007 +0100
@@ -170,9 +170,6 @@ static CMPIStatus do_assoc(struct std_as
handler = std_assoc_get_handler(ctx, ref);
if (handler == NULL) {
CU_DEBUG("No handler found.");
- cu_statusf(ctx->brkr, &s,
- CMPI_RC_ERR_FAILED,
- "Unable to handle this association");
goto out;
}
@@ -269,9 +266,7 @@ static CMPIStatus do_ref(struct std_asso
handler = std_assoc_get_handler(ctx, ref);
if (handler == NULL) {
- cu_statusf(ctx->brkr, &s,
- CMPI_RC_ERR_FAILED,
- "Unable to handle this association");
+ CU_DEBUG("No handler found.");
goto out;
}