ohai 6.18.0.rc.0 → 6.18.0.rc.1

data/lib/ohai/mixin/command.rb

@@ -224,6 +224,10 @@ module Ohai
 
         begin
           e = Marshal.load ps.first
+          pw.last.close
+          pr.first.close
+          pe.first.close
+          Process.wait(cid)
           raise(Exception === e ? e : "unknown failure!")
         rescue EOFError # If we get an EOF error, then the exec was successful
           42

data/lib/ohai/version.rb

@@ -18,6 +18,6 @@
 
 module Ohai
   OHAI_ROOT = File.expand_path(File.dirname(__FILE__))
-  VERSION = '6.18.0.rc.0'
+  VERSION = '6.18.0.rc.1'
 end
 

data/spec/unit/mixin/command_spec.rb

@@ -44,4 +44,26 @@ describe Ohai::Mixin::Command, "popen4" do
     end
   end
 
+  it "reaps zombie processes after exec fails [OHAI-455]" do
+    # NOTE: depending on ulimit settings, GC, etc., before the OHAI-455 patch,
+    # ohai could also exhaust the available file descriptors when creating this
+    # many zombie processes. A regression _could_ cause Errno::EMFILE but this
+    # probably won't be consistent on different environments.
+    created_procs = 0
+    100.times do
+      begin
+        Ohai::Mixin::Command.popen4("/bin/this-is-not-a-real-command") {|p,i,o,e| nil }
+      rescue Ohai::Exceptions::Exec
+        created_procs += 1
+      end
+    end
+    created_procs.should == 100
+    reaped_procs = 0
+    begin
+      loop { Process.wait(-1); reaped_procs += 1 }
+    rescue Errno::ECHILD
+    end
+    reaped_procs.should == 0
+  end
+
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: ohai
 version: !ruby/object:Gem::Version
-  version: 6.18.0.rc.0
+  version: 6.18.0.rc.1
   prerelease: 7
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-06-25 00:00:00.000000000 Z
+date: 2013-06-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: systemu