[ruby-list:37487] Re: Tmpfile.new がデフォルトで/tmpを利用すること

▼ [ruby-list:37483] Re: Tmpfile.newがデフォルトで/tmpを利用すること

Siena. です。

》> $SAFE > 0 の時のデフォルトの動作は、最初に /tmp で試して、例外が
》> 上がったら / を、更に例外が上がったら . を使う、という提案に修正します。

》/と.の順序については違和感がないでもないですが、とりあえずこん
》なところでしょうか。

ありがとうございます。

/ を使うのは避けたいというのは同感です。
しかし、[ruby-list:37479] に書いたように、事故で削除されずに
残ってしまう一時ファイルが行方不明になるよりはましかな、と。
異論もあるでしょうから、議論があって良いと思います。

[ruby-list:37483] のパッチで、環境変数が設定されていない
時にデフォルトの tmpdir が '/tmp' になる点について。
$SAFE == 0 の時には、大元の [ruby-list:37421] と同じように
例外が上がってしまいますね。

これは、あえてこのままにして「/tmp が存在しない環境では必ず
TMPDIR or TMP or TEMP を設定しないければならない」と強制するか、
どの環境でも同じ設定で動く事を優先して [ruby-list:37480] での
提案と同じように代替のディレクトリ ( / or . ) を使うようにするか、
どちらが望ましいでしょうか。

判断つきかねますが、とりあえず後者になるようにしてみました。

---
Siena. <mailto:[e-mail]>


--- tempfile-1.17.1/tempfile.rb	Tue Apr  1 19:02:48 2003
+++ tempfile-1.17.2/tempfile.rb	Tue Apr  1 20:25:18 2003
@@ -22,11 +22,7 @@ class Tempfile < SimpleDelegator
   # tmpdir is tainted, it uses /tmp. (Note that ENV values are
   # tainted by default)
-  def initialize(basename, tmpdir=ENV['TMPDIR']||ENV['TMP']||ENV['TEMP']||'/tmp')
-    if $SAFE > 0 and tmpdir.tainted?
-      tmplist = %w'/tmp / .'
-      tmpdir = tmplist.shift
-    else
-      tmplist = []
-    end
+  def initialize(basename, tmpdir=ENV['TMPDIR']||ENV['TMP']||ENV['TEMP'])
+    tmplist = %w'/tmp / .'
+    tmpdir = tmplist.shift  if $SAFE > 0 and tmpdir.tainted? or tmpdir.nil?
 
     lock = nil