Use waitpid() instead of wait() to avoid returning early from another child

exiting, and loop the waitpid() on EINTR

ok deraadt@ millert@

1 files changed, 3 insertions, 2 deletions

diff --git a/src/lib/libc/net/rcmdsh.c b/src/lib/libc/net/rcmdsh.c
index ad3f79ecc5..a8cd0e60df 100644
--- a/src/lib/libc/net/rcmdsh.c
+++ b/src/lib/libc/net/rcmdsh.c
@@ -1,4 +1,4 @@
-/*      $OpenBSD: rcmdsh.c,v 1.15 2015/09/12 14:56:50 guenther Exp $    */ 
+/*      $OpenBSD: rcmdsh.c,v 1.16 2015/10/23 04:45:32 guenther Exp $    */ 
 
 /*
  * Copyright (c) 2001, MagniComp
@@ -176,7 +176,8 @@ rcmdsh(char **ahost, int rport, const char *locuser, const char *remuser,
                 /* Parent. close sp[1], return sp[0]. */
                 (void) close(sp[1]);
                 /* Reap child. */
-                (void) wait(NULL);
+                while (waitpid(cpid, NULL, 0) == -1 && errno == EINTR)
+                        ;
                 return(sp[0]);
         }
         /* NOTREACHED */