From: Juan Quintela <quintela@redhat.com>
Date: Thu, 4 Mar 2010 09:00:31 +0000 (+0100)
Subject: slirp: check system() success
X-Git-Tag: TizenStudio_2.0_p2.3~5442
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=e57f7724e67410133ee6c7baf00d64d0ed524366;p=sdk%2Femulator%2Fqemu.git

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

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 @@ static void slirp_smb_cleanup(SlirpState *s)
     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",