Added a Process.waitall thread when there's a timeout, to avoid zombies.

Without this call, every time there's a timeout, we'll get a zombie.

1 files changed, 5 insertions, 0 deletions

diff --git a/lib/facter/util/resolution.rb b/lib/facter/util/resolution.rb
index 19f2c1e..b50c9e9 100644
--- a/lib/facter/util/resolution.rb
+++ b/lib/facter/util/resolution.rb
@@ -121,6 +121,11 @@ class Facter::Util::Resolution
             end
         rescue Timeout::Error => detail
             warn "Timed out seeking value for %s" % self.name
+
+            # This call avoids zombies -- basically, create a thread that will
+            # dezombify all of the child processes that we're ignoring because
+            # of the timeout.
+            Thread.new { Process.waitall }
             return nil
         end