[ruby-dev:3968] load (Re: interpreter reinitialization)

前田です。
# なんとか今年中に卒論が完成しそうです。

In message "[ruby-dev:3960] Re: interpreter reinitialization"
Yukihiro Matsumoto <[e-mail]> wrote:

> |# safe_levelを4にすると、mod_ruby自体rb_load()を呼べなくなって
> |# しまいますね:-(
> 
> あれ？ loadやrb_loadはsafe_levelに関わらず呼べるんでないです
> か？ 第2引数をtrueに指定すれば．

では、これはバグですね。


この変更をした後でも、

ruby -e 'FILE="test.rb"; $SAFE=3; p FILE.tainted?; load(FILE, true)'
false
/tmp/rba07840:1:in `load': Insecure operation - load (SecurityError)
        from /tmp/rba07840:1

となってしまいますが、これはなぜでしょう。
エラーメッセージから推測するとCheck_SafeStrでひっかかってるみたいなので
すが、tainted?ではfalseが返ってるみたいですし。うーむ。

あともう一つバグを見つけたのですが、それは別メールにします。

> それと，$SAFEに関しては多分
> loadの前にスレッドを起こして
> 
>   Thread.start do
>     load file, true
>   end
> 
> という組合せで使うべきものなんじゃないかと思ってます(そのつ
> もりで設計しました)．

$SAFEってスレッドローカルなんですね...。
# 知らなかったので、rubyレベルで$SAFEを変更できないようにしてました:-(
# 下手な考え休むに似たり。

> |blue[ruby]$ ruby -e 'load("foo.rb", true)'
> |/home/shugo/ruby/foo.rb:4: undefined local variable or method `foo' for #<Object:0x401c5f4c> (NameError)
> |        from /tmp/rba00835:1:in `load'
> |        from /tmp/rba00835:1
> |
> |というのはちょっとまずいと思います。
> 
> ちょっとじゃなくまずいですね．修正します．

あと、

def foo
  puts "foo"
end

class Bar
  def bar
    foo
  end
end

b = Bar.new
b.bar

というのもまずいですね。

モジュール内でクラス定義された場合、そのクラスで自動的にモジュールを
インクルードしてくれると解決すると思うのですが、いかがでしょう。


これはloadを使う時以外でも有用だと思います。

-- 
前田 修吾 (mailto:[e-mail])