From a139c4c8e3555bc30b72fe05e6009da5cc37c280 Mon Sep 17 00:00:00 2001
From: Eric Wong <normalperson@yhbt.net>
Date: Wed, 1 Apr 2009 21:34:32 -0700
Subject: Close std{err,out} redirection targets

The newly open file descriptors live on as fd=1
and fd=2 anyways, so there's no reason to keep
duplicates around.
---
 lib/unicorn.rb | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/lib/unicorn.rb b/lib/unicorn.rb
index c1d68c1..c6a4e36 100644
--- a/lib/unicorn.rb
+++ b/lib/unicorn.rb
@@ -94,8 +94,8 @@ module Unicorn
       raise ArgumentError, "no listeners" if @listeners.empty?
       self.pid = @config[:pid]
       build_app! if @preload_app
-      $stderr.reopen(File.open(@stderr_path, "a")) if @stderr_path
-      $stdout.reopen(File.open(@stdout_path, "a")) if @stdout_path
+      File.open(@stderr_path, "a") { |fp| $stderr.reopen(fp) } if @stderr_path
+      File.open(@stdout_path, "a") { |fp| $stdout.reopen(fp) } if @stdout_path
       $stderr.sync = $stdout.sync = true
       spawn_missing_workers
       self
-- 
cgit v1.2.3-24-ge0c7