[ruby-dev:29478] Re: /tmp

なかだです。

At Tue, 5 Sep 2006 01:01:38 +0900,
Yukihiro Matsumoto wrote in [ruby-dev:29477]:
> |私が尋ねたいのは、/tmp/foo/bin などの world writable でない
> |サブディレクトリについてです。
> |
> |そのようなサブディレクトリについても警告が出る理由はなんでしょ
> |う?
> 
> えーと、親ディレクトリがworld writableでもsticky bitがついて
> いれば、サブディレクトリについて警告を出す必要はない、という
> ことですかね。

それは大分前に無効になっているというか、それ以前の「stickyなサ
ブディレクトリもOK」という状態に戻されてますね。

> 中田さんのパッチ([ruby-dev:29433]とか)はこれに沿ったものなん
> でしょうか。

なってないです。


Index: file.c
===================================================================
RCS file: /cvs/ruby/src/ruby/file.c,v
retrieving revision 1.247
diff -p -U 2 -r1.247 file.c
--- file.c	2 Sep 2006 14:42:05 -0000	1.247
+++ file.c	4 Sep 2006 19:42:32 -0000
@@ -4047,5 +4047,5 @@ is_absolute_path(const char *path)
 #ifndef DOSISH
 static int
-path_check_0(VALUE path, int loadpath)
+path_check_0(VALUE path, int stickyok)
 {
     struct stat st;
@@ -4062,5 +4062,5 @@ path_check_0(VALUE path, int loadpath)
 	rb_str_cat2(newpath, "/");
 	rb_str_cat2(newpath, p0);
-	return path_check_0(newpath, loadpath);
+	p0 = RSTRING_PTR(path = newpath);
     }
     for (;;) {
@@ -4070,5 +4070,5 @@ path_check_0(VALUE path, int loadpath)
 	if (stat(p0, &st) == 0 && S_ISDIR(st.st_mode) && (st.st_mode & S_IWOTH)
 #ifdef S_ISVTX
-	    && (loadpath || !(st.st_mode & S_ISVTX))
+	    && !(p && stickyok && (st.st_mode & S_ISVTX))
 #endif
 	    && !access(p0, W_OK)) {
@@ -4248,7 +4248,4 @@ rb_find_file(VALUE path)
 	else {
 	    lpath = RSTRING_PTR(tmp);
-	    if (rb_safe_level() >= 1 && !rb_path_check(lpath)) {
-		rb_raise(rb_eSecurityError, "loading from unsafe path %s", lpath);
-	    }
 	}
     }


-- 
--- 僕の前にBugはない。
--- 僕の後ろにBugはできる。
    中田 伸悦