]> xenbits.xensource.com Git - libvirt.git/commitdiff
virdbus: Show method name in error message
authorCole Robinson <crobinso@redhat.com>
Sat, 3 May 2014 19:53:03 +0000 (15:53 -0400)
committerCole Robinson <crobinso@redhat.com>
Wed, 21 May 2014 13:33:25 +0000 (09:33 -0400)
If you trigger bug 1033369, we get the error message:

  error from service: Invalid argument

Which is a bit too generic to pinpoint what is actually failing. This
changes it to:

  error from service: CreateMachine: Invalid argument

Acked-by: Eric Blake <eblake@redhat.com>
src/util/virdbus.c

index 03ec028dc98f93c29e500b12c96f71eda55d1b3e..31251fe106b0a6aed8b138e558c0a5663fbd1283 100644 (file)
@@ -1408,7 +1408,8 @@ static int
 virDBusCall(DBusConnection *conn,
             DBusMessage *call,
             DBusMessage **replyout,
-            DBusError *error)
+            DBusError *error,
+            const char *member)
 {
     DBusMessage *reply = NULL;
     DBusError localerror;
@@ -1424,7 +1425,7 @@ virDBusCall(DBusConnection *conn,
         if (error)
             ret = 0;
         else {
-            virReportError(VIR_ERR_DBUS_SERVICE, "%s",
+            virReportError(VIR_ERR_DBUS_SERVICE, _("%s: %s"), member,
                 localerror.message ? localerror.message : _("unknown error"));
         }
         goto cleanup;
@@ -1502,7 +1503,7 @@ int virDBusCallMethod(DBusConnection *conn,
 
     ret = -1;
 
-    ret = virDBusCall(conn, call, replyout, error);
+    ret = virDBusCall(conn, call, replyout, error, member);
 
  cleanup:
     if (call)