Don't use first byte of string as a pointer.

* src/qemu_conf.c (qemudReportError): Use the pointer, errorMessage,
not its first byte, errorMessage[0].

diff --git a/ChangeLog b/ChangeLog
index d331a4cb82d4eaaedb65f0c02fd7047d84e2adf4..50e48f6aff395685dc8d746d2c25aabf50e4dc96 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,9 @@
 Mon Mar  3 19:09:51 CET 2008 Jim Meyering <meyering@redhat.com>
 
+       Don't use first byte of string as a pointer.
+       * src/qemu_conf.c (qemudReportError): Use the pointer, errorMessage,
+       not its first byte, errorMessage[0].
+
        Avoid segfault upon early libvirtd failure.
        * qemud/qemud.c (main): Don't call qemudCleanup on an
        uninitialized pointer.

diff --git a/src/qemu_conf.c b/src/qemu_conf.c
index eead0bca29220cc8da41a0abf84b3f876079a275..e54da5b23009b926f6913c37eef77deab9292905 100644 (file)
--- a/src/qemu_conf.c
+++ b/src/qemu_conf.c
@@ -68,7 +68,7 @@ void qemudReportError(virConnectPtr conn,
         errorMessage[0] = '\0';
     }
 
-    virerr = __virErrorMsg(code, (errorMessage[0] ? errorMessage[0] : NULL));
+    virerr = __virErrorMsg(code, (errorMessage[0] ? errorMessage : NULL));
     __virRaiseError(conn, dom, net, VIR_FROM_QEMU, code, VIR_ERR_ERROR,
                     virerr, errorMessage, NULL, -1, -1, virerr, errorMessage);
 }