# HG changeset patch # User Richard M. Stallman # Date 745388885 0 # Node ID 004c38daf0ae7f431816358d60f2177209541771 # Parent aceaf48ee60dcf96987a282e77f3af15543ea618 (mkdir, rmdir): Use wait_for_termination to wait. Redirect descriptors 0...2 to /dev/null. diff -r aceaf48ee60d -r 004c38daf0ae src/sysdep.c --- a/src/sysdep.c Sun Aug 15 03:55:29 1993 +0000 +++ b/src/sysdep.c Sun Aug 15 04:28:05 1993 +0000 @@ -3112,7 +3112,7 @@ char *dpath; int dmode; { - int cpid, status; + int cpid, status, fd; struct stat statbuf; if (stat (dpath, &statbuf) == 0) @@ -3125,10 +3125,11 @@ if (errno != ENOENT) return -1; + synch_process_alive = 1; switch (cpid = fork ()) { - case -1: /* Error in fork() */ + case -1: /* Error in fork */ return (-1); /* Errno is set already */ case 0: /* Child process */ @@ -3140,14 +3141,21 @@ */ status = umask (0); /* Get current umask */ status = umask (status | (0777 & ~dmode)); /* Set for mkdir */ + fd = sys_open("/dev/null", 2); + if (fd >= 0) + { + dup2 (fd, 0); + dup2 (fd, 1); + dup2 (fd, 2); + } execl ("/bin/mkdir", "mkdir", dpath, (char *) 0); _exit (-1); /* Can't exec /bin/mkdir */ default: /* Parent process */ - while (cpid != wait (&status)); /* Wait for kid to finish */ + wait_for_termination (cpid); } - if (WIFSIGNALED (status) || WEXITSTATUS (status) != 0) + if (synch_process_death != 0 || synch_process_retcode != 0) { errno = EIO; /* We don't know why, but */ return -1; /* /bin/mkdir failed */ @@ -3162,7 +3170,7 @@ rmdir (dpath) char *dpath; { - int cpid, status; + int cpid, status, fd; struct stat statbuf; if (stat (dpath, &statbuf) != 0) @@ -3171,24 +3179,32 @@ return -1; } + synch_process_alive = 1; switch (cpid = fork ()) { - case -1: /* Error in fork() */ + case -1: /* Error in fork */ return (-1); /* Errno is set already */ case 0: /* Child process */ + fd = sys_open("/dev/null", 2); + if (fd >= 0) + { + dup2 (fd, 0); + dup2 (fd, 1); + dup2 (fd, 2); + } execl ("/bin/rmdir", "rmdir", dpath, (char *) 0); _exit (-1); /* Can't exec /bin/mkdir */ default: /* Parent process */ - while (cpid != wait (&status)); /* Wait for kid to finish */ + wait_for_termination (cpid); } - if (WIFSIGNALED (status) || WEXITSTATUS (status) != 0) + if (synch_process_death != 0 || synch_process_retcode != 0) { errno = EIO; /* We don't know why, but */ - return -1; /* /bin/mkdir failed */ + return -1; /* /bin/rmdir failed */ } return 0;