[Libvirt-cim] [PATCH 2 of 4] Removed error messages that cause wrong error returns

Heidi Eckhart heidieck at linux.vnet.ibm.com
Fri Nov 30 10:03:59 UTC 2007


# HG changeset patch
# User Heidi Eckhart <heidieck at 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 at 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;
         }
 




More information about the Libvirt-cim mailing list