From 08cb30801c79de24ca7e40cf1d6a94df94ee2b4b Mon Sep 17 00:00:00 2001 From: Yang Tse Date: Mon, 26 Nov 2007 14:26:40 +0000 Subject: [PATCH] Stop ssh and socks servers when verification fails --- tests/runtests.pl | 17 ++++++++++------- 1 file changed, 10 insertions(+), 7 deletions(-) diff --git a/tests/runtests.pl b/tests/runtests.pl index 1ae0f06..b1258d0 100755 --- a/tests/runtests.pl +++ b/tests/runtests.pl @@ -1067,9 +1067,8 @@ sub runsshserver { if($sshpid <= 0 || !kill(0, $sshpid)) { # it is NOT alive logmsg "RUN: failed to start the SSH server\n"; - # failed to talk to it properly. Kill the server and return failure - logmsg "TRACESSH:runsshserver: calling stopserver with sshpid: $sshpid pid2: $pid2\n"; - stopserver("$sshpid $pid2"); + logmsg "TRACESSH:runsshserver: calling stopserver with pid2: $pid2\n"; + stopserver("$pid2"); $doesntrun{$pidfile} = 1; logmsg "TRACESSH:runsshserver: later dont try to start a server with pidfile: $pidfile\n"; return (0,0); @@ -1077,7 +1076,9 @@ sub runsshserver { if (!verifyserver('ssh',$ip,$port)) { logmsg "RUN: SSH server failed verification\n"; - logmsg "TRACESSH:runsshserver: BUT It seems that we are letting sshpid: $sshpid pid2: $pid2 alive\n"; + # failed to talk to it properly. Kill the server and return failure + logmsg "TRACESSH:runsshserver: calling stopserver with sshpid: $sshpid pid2: $pid2\n"; + stopserver("$sshpid $pid2"); $doesntrun{$pidfile} = 1; logmsg "TRACESSH:runsshserver: later dont try to start a server with pidfile: $pidfile\n"; return (0,0); @@ -1115,12 +1116,11 @@ sub runsocksserver { if($sshpid <= 0 || !kill(0, $sshpid)) { # it is NOT alive logmsg "RUN: failed to start the SOCKS server\n"; - # failed to talk to it properly. Kill the server and return failure logmsg "=== Start of file log/ssh.log\n"; displaylogcontent("log/ssh.log"); logmsg "=== End of file log/ssh.log\n"; - logmsg "TRACESSH:runsocksserver: calling stopserver with sshpid: $sshpid pid2: $pid2\n"; - stopserver("$sshpid $pid2"); + logmsg "TRACESSH:runsocksserver: calling stopserver with pid2: $pid2\n"; + stopserver("$pid2"); $doesntrun{$pidfile} = 1; logmsg "TRACESSH:runsocksserver: later dont try to start a server with pidfile: $pidfile\n"; return (0,0); @@ -1129,6 +1129,9 @@ sub runsocksserver { # Ugly hack but ssh doesn't support pid files if (!verifyserver('socks',$ip,$port)) { logmsg "RUN: SOCKS server failed verification\n"; + # failed to talk to it properly. Kill the server and return failure + logmsg "TRACESSH:runsocksserver: calling stopserver with sshpid: $sshpid pid2: $pid2\n"; + stopserver("$sshpid $pid2"); $doesntrun{$pidfile} = 1; logmsg "TRACESSH:runsocksserver: later dont try to start a server with pidfile: $pidfile\n"; return (0,0); -- 2.7.4