[ruby-dev:31436] Re: File exists - /tmp/bootstraptest.tmpwd

In article <[e-mail]>,
  Tanaka Akira <[e-mail]> writes:

>> 普通のテンポラリファイルの慣習どおりに名前を毎回変えて重なら
>> ないようにしたらいいと思うんですが。
>
> というわけで、こういうように Dir.mktmpdir を使うのはどうでしょ
> うか。
>
> というか、FileUtils.rm_rf で /tmp/bootstraptest.tmpwd を消す
> のって、セキュリティ的に問題がありますよね。
>
> (FileUtils.rm_rf の問題に関しては
> http://www.ruby-lang.org/ja/man/?cmd=view;name=fileutils に
> いくらか記載があります。)

うぅむ。bootstraptest は違う ruby で実行されることがあるから、
Dir.mktmpdir があるとは限らないですね。

とすると、ないときは自前で定義することにして、こうでしょうか。

Index: bootstraptest/runner.rb
===================================================================
--- bootstraptest/runner.rb	(リビジョン 12943)
+++ bootstraptest/runner.rb	(作業コピー)
@@ -13,10 +13,38 @@
   retry
 end
 
+if !Dir.respond_to?(:mktmpdir)
+  # copied from lib/tmpdir.rb
+  def Dir.mktmpdir(prefix="d", tmpdir=nil)
+    tmpdir ||= Dir.tmpdir
+    t = Time.now.strftime("%Y%m%d")
+    n = nil
+    begin
+      path = "#{tmpdir}/#{prefix}#{t}-#{$$}-#{rand(0x100000000).to_s(36)}"
+      path << "-#{n}" if n
+      Dir.mkdir(path, 0700)
+    rescue Errno::EEXIST
+      n ||= 0
+      n += 1
+      retry
+    end
+
+    if block_given?
+      begin
+        yield path
+      ensure
+        FileUtils.remove_entry_secure path
+      end
+    else
+      path
+    end
+  end
+end
+
 def main
   @ruby = File.expand_path('miniruby')
   @verbose = false
-  dir = File.join(Dir.tmpdir, 'bootstraptest.tmpwd')
+  dir = nil
   quiet = false
   tests = nil
   ARGV.delete_if {|arg|
@@ -176,11 +204,19 @@
 end
 
 def in_temporary_working_directory(dir)
-  FileUtils.rm_rf dir
-  Dir.mkdir dir
-  Dir.chdir(dir) {
-    yield
-  }
+  if dir
+    FileUtils.rm_rf dir
+    Dir.mkdir dir
+    Dir.chdir(dir) {
+      yield
+    }
+  else
+    Dir.mktmpdir("bootstraptest.tmpwd") {|d|
+      Dir.chdir(d) {
+        yield
+      }
+    }
+  end
 end
 
 def cleanup_coredump
-- 
[田中 哲][たなか あきら][Tanaka Akira]