tests: Prevent child process falling to defunct state

Parent process does not wait for child process(which gets forked after
accepting a client connection) to exit inside echo server which in turn
puts child into defunct state. Even though those defunct childs are
cleaned up during parent exit it is better to wait from parent process
itself.

Signed-off-by: Anoop C S <anoopcs@redhat.com>
Reviewed-by: Andreas Schneider <asn@samba.org>
Reviewed-by: Stefan Metzmacher <metze@samba.org>

1 files changed, 2 insertions, 0 deletions

diff --git a/tests/echo_srv.c b/tests/echo_srv.c
index 771904a..3f8b2a9 100644
--- a/tests/echo_srv.c
+++ b/tests/echo_srv.c
@@ -5,6 +5,7 @@
 #include <sys/types.h>
 #include <sys/socket.h>
 #include <sys/stat.h>
+#include <sys/wait.h>
 
 #include <arpa/inet.h>
 #include <netinet/in.h>
@@ -535,6 +536,7 @@ static void echo_tcp(int sock)
             close(s);
             exit(0);
         }
+        waitpid(-1, NULL, 0);
         close(s);
     }