Merge pull request #62 from Martchus/process-group-handling-2

Martchus · web-flow · commit a0468f55e6ed · 2025-01-15T13:04:05.000+01:00
Fix race condition in `is_running` when `kill_whole_group` is set
diff --git a/lib/Mojo/IOLoop/ReadWriteProcess.pm b/lib/Mojo/IOLoop/ReadWriteProcess.pm
@@ -404,7 +404,7 @@ sub is_running {
   my ($self) = shift;
   $self->session->consume_collected_info;
   return 0 unless my $pid = $self->process_id;
-  kill(0, ($self->kill_whole_group ? -$pid : $pid));
+  kill(0, ($self->kill_whole_group ? (-$pid, $pid) : ($pid)));
 }
 
 sub write_pidfile {

Original file line number	Diff line number	Diff line change
`@@ -404,7 +404,7 @@ sub is_running {`
`404`	`404`	`my ($self) = shift;`
`405`	`405`	`$self->session->consume_collected_info;`
`406`	`406`	`return 0 unless my $pid = $self->process_id;`
`407`		`- kill(0, ($self->kill_whole_group ? -$pid : $pid));`
	`407`	`+ kill(0, ($self->kill_whole_group ? (-$pid, $pid) : ($pid)));`
`408`	`408`	`}`
`409`	`409`
`410`	`410`	`sub write_pidfile {`