PR feedback

Check if there's running processes in the job object before promoting to a silo Signed-off-by: Daniel Canter <[email protected]>
microsoft · Mar 23, 2022 · 3cfdf8a · 3cfdf8a
1 parent 1bbaa26
commit 3cfdf8a
Showing 1 changed file with 10 additions and 1 deletion.
diff --git a/internal/jobobject/jobobject.go b/internal/jobobject/jobobject.go
@@ -522,7 +522,16 @@ func (job *JobObject) PromoteToSilo() error {
 		return nil
 	}
 
-	_, err := windows.SetInformationJobObject(
+	pids, err := job.Pids()
+	if err != nil {
+		return err
+	}
+
+	if len(pids) != 0 {
+		return fmt.Errorf("job cannot have running processes to be promoted to a silo, found %d running processes", len(pids))
+	}
+
+	_, err = windows.SetInformationJobObject(
 		job.handle,
 		winapi.JobObjectCreateSilo,
 		0,