[ruby-dev:14547] setuid/setgid

なかだです。

File::Stat.*_real?が実効グループで許可されているときにも真を返
しています。

  # cat test 
  p open($0.untaint).stat.readable_real?

  $ ls -l ./ruby test
  -rwxr-sr-x    1 root     guest        3300  8月 21 22:39 ./ruby*
  -rw-r-----    1 root     guest          39  8月 21 22:49 test
  $ cat test
  cat: test: 許可がありません
  $ ./ruby test
  true

それとeaccess()について気づいた点。

  * euidをキャッシュしているため、Process.euid=しても反映されな
    い。

  * 複数のmodeを指定された場合、どれかが許可されていれば0を返し
    ている。access(2)はすべてが許可されていなければ-1を返す。

二つ目はそういう使い方はしていないので実際には問題になりません
が、合わせておいたほうがいいような気がします。


Index: file.c
===================================================================
RCS file: /cvs/ruby/src/ruby/file.c,v
retrieving revision 1.68
diff -u -2 -p -r1.68 file.c
--- file.c	2001/08/01 16:49:11	1.68
+++ file.c	2001/08/21 13:28:09
@@ -391,5 +391,5 @@ group_member(gid)
 {
 #if !defined(NT)
-    if (getgid() ==  gid || getegid() == gid)
+    if (getgid() ==  gid)
 	return Qtrue;
 
@@ -423,15 +423,14 @@ eaccess(path, mode)
 #ifdef S_IXGRP
   struct stat st;
-  static int euid = -1;
+  int euid;
 
   if (stat(path, &st) < 0) return (-1);
 
-  if (euid == -1)
-    euid = geteuid ();
+  euid = geteuid();
 
   if (euid == 0)
     {
       /* Root can read or write any file. */
-      if (mode != X_OK)
+      if (!(mode & X_OK))
 	return 0;
 
@@ -440,12 +439,14 @@ eaccess(path, mode)
       if (st.st_mode & S_IXUGO)
 	return 0;
+
+      return -1;
     }
 
   if (st.st_uid == euid)        /* owner */
     mode <<= 6;
-  else if (group_member (st.st_gid))
+  else if (getegid() == st.st_gid || group_member(st.st_gid))
     mode <<= 3;
 
-  if (st.st_mode & mode) return 0;
+  if ((st.st_mode & mode) != mode) return 0;
 
   return -1;


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