diff options
author | Stefan Weil <sw@weilnetz.de> | 2012-08-24 07:03:03 +0200 |
---|---|---|
committer | Michael Roth <mdroth@linux.vnet.ibm.com> | 2012-08-30 14:54:16 -0500 |
commit | cccb5446a6d1083b51d851ef5a4f1acff9173127 (patch) | |
tree | e51006be2e515f1456e44a9626f7c91ce8c334e0 | |
parent | memory: Fix copy&paste mistake in memory_region_iorange_write (diff) | |
download | qemu-kvm-cccb5446a6d1083b51d851ef5a4f1acff9173127.tar.gz qemu-kvm-cccb5446a6d1083b51d851ef5a4f1acff9173127.tar.bz2 qemu-kvm-cccb5446a6d1083b51d851ef5a4f1acff9173127.zip |
qemu-ga: Fix null pointer passed to unlink in failure branch
Clang reports this warning:
Null pointer passed as an argument to a 'nonnull' parameter
Reviewed-by: Luiz Capitulino <lcapitulino@redhat.com>
Signed-off-by: Stefan Weil <sw@weilnetz.de>
Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
(cherry picked from commit 4bdb1a3059d7d3a931de0748a2eec39a0ab41b4e)
Signed-off-by: Michael Roth <mdroth@linux.vnet.ibm.com>
-rw-r--r-- | qemu-ga.c | 4 |
1 files changed, 3 insertions, 1 deletions
@@ -439,7 +439,9 @@ static void become_daemon(const char *pidfile) return; fail: - unlink(pidfile); + if (pidfile) { + unlink(pidfile); + } g_critical("failed to daemonize"); exit(EXIT_FAILURE); #endif |