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].
This commit is contained in:
Jim Meyering 2008-03-03 18:11:16 +00:00
parent f9e94c2e13
commit 4090a0870c
2 changed files with 5 additions and 1 deletions

View File

@ -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.

View File

@ -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);
}