diff options
author | Andreas Schneider <asn@cryptomilk.org> | 2015-10-01 13:55:13 +0200 |
---|---|---|
committer | Andreas Schneider <asn@cryptomilk.org> | 2016-01-14 08:04:49 +0100 |
commit | 0e98f1214dd6d80d15aee7f74df40bf7ddd6f81e (patch) | |
tree | 86d102fc73af2be21aee8f9d226ff1f3bdd5d5f8 | |
parent | 6e7eae968a3e20050a3b1d9ebab68c96660babe4 (diff) | |
download | libssh-0e98f1214dd6d80d15aee7f74df40bf7ddd6f81e.tar.gz libssh-0e98f1214dd6d80d15aee7f74df40bf7ddd6f81e.tar.xz libssh-0e98f1214dd6d80d15aee7f74df40bf7ddd6f81e.zip |
torture: Improve process termination function
Signed-off-by: Andreas Schneider <asn@cryptomilk.org>
-rw-r--r-- | tests/torture.c | 21 |
1 files changed, 16 insertions, 5 deletions
diff --git a/tests/torture.c b/tests/torture.c index 9cd311d1..719f8f16 100644 --- a/tests/torture.c +++ b/tests/torture.c @@ -411,6 +411,8 @@ int torture_terminate_process(const char *pidfile) ssize_t rc; pid_t pid; int fd; + int is_running = 1; + int count; /* read the pidfile */ fd = open(pidfile, O_RDONLY); @@ -433,11 +435,20 @@ int torture_terminate_process(const char *pidfile) pid = (pid_t)(tmp & 0xFFFF); - /* Make sure the process goes away! */ - kill(pid, SIGTERM); + for (count = 0; count < 10; count++) { + /* Make sure the daemon goes away! */ + kill(pid, SIGTERM); - kill(pid, 0); - if (rc == 0) { + usleep(200); + + rc = kill(pid, 0); + if (rc != 0) { + is_running = 0; + break; + } + } + + if (is_running) { fprintf(stderr, "WARNING: The process server is still running!\n"); } @@ -992,7 +1003,7 @@ void torture_teardown_sshd_server(void **state) rc = torture_terminate_process(s->srv_pidfile); if (rc != 0) { - fprintf(stderr, "Failed to terminate sshd"); + fprintf(stderr, "XXXXXX Failed to terminate sshd\n"); } torture_teardown_socket_dir(state); |