diff options
author | kosaki <kosaki@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2013-02-21 04:41:11 +0000 |
---|---|---|
committer | kosaki <kosaki@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2013-02-21 04:41:11 +0000 |
commit | ba37db04533cec102212605488989c5d02bea67f (patch) | |
tree | bfb7f2503116497af27bbe5f1c5ce64ff0ccd414 | |
parent | 7c465090fd976fe6892a8ec64b036fa0696e3723 (diff) | |
download | ruby-ba37db04533cec102212605488989c5d02bea67f.tar.gz |
* file.c (eaccess): use access() when not using setuid nor setgid.
This is minor optimization.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@39350 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r-- | ChangeLog | 5 | ||||
-rw-r--r-- | file.c | 8 |
2 files changed, 11 insertions, 2 deletions
@@ -1,3 +1,8 @@ +Thu Feb 21 13:04:59 2013 KOSAKI Motohiro <kosaki.motohiro@gmail.com> + + * file.c (eaccess): use access() when not using setuid nor setgid. + This is minor optimization. + Thu Feb 21 12:56:19 2013 KOSAKI Motohiro <kosaki.motohiro@gmail.com> * file.c (rb_group_member): get rid of NGROUPS dependency. @@ -1077,11 +1077,15 @@ eaccess(const char *path, int mode) struct stat st; rb_uid_t euid; + euid = geteuid(); + + /* no setuid nor setgid. run shortcut. */ + if (getuid() == euid && getgid() == getegid()) + return access(path, mode); + if (STAT(path, &st) < 0) return -1; - euid = geteuid(); - if (euid == 0) { /* Root can read or write any file. */ if (!(mode & X_OK)) |