Commit ea1a9e41 authored by unknown's avatar unknown
Browse files

Don't bother to remove the pid_file in mtr_kill_leftovers as it could... 

Don't bother to remove the pid_file in mtr_kill_leftovers as it could potentially remove a the pidfile before the process is killed.
The pid file will be removed later when var/ directory is recreated.
parent 0dc08b90

mysql-test/lib/mtr_process.pl

+0 −8
Original line number Diff line number Diff line
@@ -470,14 +470,6 @@ sub mtr_kill_leftovers () { 


        mtr_debug("Got pid: $pid from file '$pidfile'");



        # Race, could have been removed between I tested with -f

        # and the unlink() below, so I better check again with -f



        if ( ! unlink($pidfile) and -f $pidfile )

        {

          mtr_error("can't remove $pidfile");

        }



        if ( $::glob_cygwin_perl or kill(0, $pid) )

        {

          mtr_debug("There is process with pid $pid -- scheduling for kill.");