Merge pull request #27 from pyromaniac/master

Fork bomb fix
DAddYE · Apr 24, 2013 · 0fcd088 · 0fcd088
2 parents f265b65 + a0f6eec
commit 0fcd088
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/lib/forever/base.rb b/lib/forever/base.rb
@@ -109,7 +109,7 @@ def initialize(options={}, &block)
             if forking
               begin
                 GC.start
-                pids << fork { job_call(job) }
+                pids << Process.detach(fork { job_call(job) })
               rescue Errno::EAGAIN
                 puts "\n\nWait all processes since os cannot create a new one\n\n"
                 Process.waitall
@@ -121,7 +121,7 @@ def initialize(options={}, &block)
           end
 
           # Detach zombies, our ps will be happier
-          pids.delete_if { |p| Process.detach(p).stop? }
+          pids.delete_if { |p| p.stop? }
 
           sleep 0.5
         end