[ruby-dev:26226] Re: FileUtils.rm_rf security problem

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

> とりあえず、報告した例は直っていません。

報告した例は直ったようです。

ただ、fu_fix_permission の定義において、fu_find 内の lstat と
File.chown の間に path が指すディレクトリが他のファイルにすり替えられ
た場合、意図しないファイルを chown してしまうことになると思うのですが、
どうでしょう?

  def fu_fix_permission(prefix)   #:nodoc:
    fu_find([prefix]) do |path, lstat|
      if lstat.directory?
        begin
          File.chown Process.euid, nil, path
        rescue Errno::EPERM
        end
        begin
          File.chmod 0700, path
        rescue Errno::EPERM
        end
      end
    end
  rescue
  end

たとえば、root が FileUtils.rm_rf("/tmp/a") としたときに、/tmp/a を
lstat した後、chown する前に /tmp/a が sudo コマンド (や他の setuid
command) への link にされてしまうと、setuid bit が落ちて痛いことになる
気がするのですが。
-- 
[田中 哲][たなか あきら][Tanaka Akira]