moparisthebest/curl
1
0
Fork 0
mirror of https://github.com/moparisthebest/curl synced 2025-01-09 04:58:00 -05:00
Code Issues Releases Wiki Activity

runtests.pl/stopserver: space separate pids

Browse Source 
The stopserver function would append pids to kill and could append them
without separating them with space properly. The result would be a very
large number that by (some implementations of) kill would be interpreted
as a negative number and that process group would be wiped...

Bug: http://curl.haxx.se/bug/view.cgi?id=3188836
Reported by: Greg Pratt
This commit is contained in:
Daniel Stenberg 2011-02-22 13:28:27 +01:00
parent 7aa2d10e0d
commit 2345c1dd66
1 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
tests/runtests.pl
View File

@ -651,7 +651,9 @@ sub stopserver {
# #
foreach my $server (@killservers) { foreach my $server (@killservers) {
if($run{$server}) { if($run{$server}) {
$pidlist .= "$run{$server} "; # we must prepend a space since $pidlist may already contain
# a pid
$pidlist .= " $run{$server}";
$run{$server} = 0; $run{$server} = 0;
} }
$runcert{$server} = 0 if($runcert{$server}); $runcert{$server} = 0 if($runcert{$server});