Patchwork [02/10] slirp: check system() success

login
register
mail settings
Submitter Juan Quintela
Date March 3, 2010, 12:06 p.m.
Message ID <93c1dcb48b0cfa33775fc45a1ffb9c0edb44d833.1267617582.git.quintela@redhat.com>
Download mbox | patch
Permalink /patch/46791/
State New
Headers show

Comments

Juan Quintela - March 3, 2010, 12:06 p.m.
we shouldn't call W*() macros until we check that fork worked.

Signed-off-by: Juan Quintela <quintela@redhat.com>
---
 net/slirp.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

Patch

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