summaryrefslogtreecommitdiff
path: root/lib/chef/monkey_patches/tempfile.rb
blob: b9179f182baca98a81bd969e6601854242395e88 (plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
#
# Author:: Adam Jacob (<adam@opscode.com>)
# Copyright:: Copyright (c) 2008 Opscode, Inc.
# License:: Apache License, Version 2.0
#
# Licensed under the Apache License, Version 2.0 (the "License");
# you may not use this file except in compliance with the License.
# You may obtain a copy of the License at
#
#     http://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing, software
# distributed under the License is distributed on an "AS IS" BASIS,
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
# See the License for the specific language governing permissions and
# limitations under the License.
#

# == Tempfile (Patch)
# Tempfile has a horrible bug where it causes an IOError: closed stream in its
# finalizer, leading to intermittent application crashes with confusing stack
# traces. Here we monkey patch the fix into place. You can track the bug on
# ruby's redmine: http://redmine.ruby-lang.org/issues/show/3119
#
# The patch is slightly different for Ruby 1.8 and Ruby 1.9, both patches are
# included here.
class Tempfile # :nodoc:
  # Tempfile has changes between 1.8.x and 1.9.x
  # so we monkey patch separately
  if RUBY_VERSION =~ /^1\.8/
    def unlink
      # keep this order for thread safeness
      begin
        File.unlink(@tmpname) if File.exist?(@tmpname)
        @@cleanlist.delete(@tmpname)
        @tmpname = nil
        ObjectSpace.undefine_finalizer(self)
      rescue Errno::EACCES
        # may not be able to unlink on Windows; just ignore
      end
    end
    alias delete unlink


  # There is a patch for this, to be merged into 1.9 at some point.
  # When that happens, we'll want to also check the RUBY_PATCHLEVEL
  elsif RUBY_VERSION =~ /^1\.9/
    def unlink
      # keep this order for thread safeness
      return unless @tmpname
      begin
        if File.exist?(@tmpname)
          File.unlink(@tmpname)
        end
        # remove tmpname from remover
        @data[0] = @data[2] = nil
        @tmpname = nil
      rescue Errno::EACCES
        # may not be able to unlink on Windows; just ignore
      end
    end
    alias delete unlink
  end
end