summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorStefan Weil <sw@weilnetz.de>2012-08-24 07:03:03 +0200
committerAnthony Liguori <aliguori@us.ibm.com>2012-08-29 08:25:55 -0500
commit4bdb1a3059d7d3a931de0748a2eec39a0ab41b4e (patch)
tree8fcf6f2700ba8967b652b27d3000216c449440ff
parent7e2a62d82a3f1397acd67685c3008046eba8344b (diff)
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>
-rw-r--r--qemu-ga.c4
1 files changed, 3 insertions, 1 deletions
diff --git a/qemu-ga.c b/qemu-ga.c
index 26671fee3..762307988 100644
--- a/qemu-ga.c
+++ b/qemu-ga.c
@@ -438,7 +438,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