commit 24ac07dec7f23c58dc48aa7754f872781b386d46
author Juan Quintela <quintela@redhat.com> Thu Mar 04 10:00:31 2010 +0100
committer Anthony Liguori <aliguori@us.ibm.com> Tue Mar 09 11:23:00 2010 -0600
tree d3fd89fe264d656e59517292b481a2b183e70fb6
parent 48b66db9ea19e30ac360872dae590e79cd009d1a

slirp: check system() success

we shouldn't call W*() macros until we check that fork worked.

Signed-off-by: Juan Quintela <quintela@redhat.com>
Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>

net/slirp.c

diff --git a/net/slirp.c b/net/slirp.c
index 317cca7..7f846ec 100644
--- a/net/slirp.c
+++ b/net/slirp.c
@@ -472,7 +472,7 @@
     if (s->smb_dir[0] != '\0') {
         snprintf(cmd, sizeof(cmd), "rm -rf %s", s->smb_dir);
         ret = system(cmd);
-        if (!WIFEXITED(ret)) {
+        if (ret == -1 || !WIFEXITED(ret)) {
             qemu_error("'%s' failed.\n", cmd);
         } else if (WEXITSTATUS(ret)) {
             qemu_error("'%s' failed. Error code: %d\n",