From a43eeaef7529f979eb17276cd04bd87fabb25968 Mon Sep 17 00:00:00 2001
From: matz <matz@b2dd03c8-39d4-4d8f-98ff-823fe69b080e>
Date: Wed, 24 Mar 2004 01:08:20 +0000
Subject: * lib/tempfile.rb (Tempfile::_close): should not clear @tmpname  
 until the file is really removed. [ruby-core:02684]

git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@6007 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
---
 lib/tempfile.rb | 11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)

(limited to 'lib/tempfile.rb')

diff --git a/lib/tempfile.rb b/lib/tempfile.rb
index 2169a20ec5..8ebabc9ea9 100644
--- a/lib/tempfile.rb
+++ b/lib/tempfile.rb
@@ -76,12 +76,13 @@ class Tempfile < SimpleDelegator
 
   def _close	# :nodoc:
     @tmpfile.close if @tmpfile
-    @data[1] = @tmpfile = nil
-    @data = @tmpname = nil
+    @tmpfile = nil
+    @data[1] = nil if @data
+    @data = nil
   end
   protected :_close
 
-  # Closes the file.  If the optional flag is true, unlinks the file
+  #Closes the file.  If the optional flag is true, unlinks the file
   # after closing.
   #
   # If you don't explicitly unlink the temporary file, the removal
@@ -99,6 +100,7 @@ class Tempfile < SimpleDelegator
     _close
     @clean_proc.call
     ObjectSpace.undefine_finalizer(self)
+    @tmpname = nil
   end
 
   # Unlinks the file.  On UNIX-like systems, it is often a good idea
@@ -108,7 +110,8 @@ class Tempfile < SimpleDelegator
   def unlink
     # keep this order for thread safeness
     File.unlink(@tmpname) if File.exist?(@tmpname)
-    @@cleanlist.delete(@tmpname) if @@cleanlist
+    @@cleanlist.delete(@tmpname)
+    @tmpname = nil
   end
   alias delete unlink
 
-- 
cgit v1.2.3