diff options
| author | Rich Felker <dalias@aerifal.cx> | 2013-11-08 21:33:42 -0500 |
|---|---|---|
| committer | Rich Felker <dalias@aerifal.cx> | 2013-11-08 21:33:42 -0500 |
| commit | 41a9ba25cd810fb86ad0d6943bf2d5556d91b99e (patch) | |
| tree | bc8160ad56689739c15d53db146c5e1b5ea58463 /src | |
| parent | 1d0d2df609b2e47615c7798d1a09e69a2caff24b (diff) | |
| download | musl-41a9ba25cd810fb86ad0d6943bf2d5556d91b99e.tar.gz musl-41a9ba25cd810fb86ad0d6943bf2d5556d91b99e.tar.bz2 musl-41a9ba25cd810fb86ad0d6943bf2d5556d91b99e.tar.xz musl-41a9ba25cd810fb86ad0d6943bf2d5556d91b99e.zip | |
remove O_NOFOLLOW from __map_file used for time zone file loading
it's not clear why I originally wrote O_NOFOLLOW into this; I suspect
the reason was with an aim of making the function more general for
mapping partially or fully untrusted files provided by the user.
however, the timezone code already precludes use of absolute or
relative pathnames in suid/sgid programs, and disallows .. in
pathnames which are relative to one of the system timezone locations,
so there is no threat of opening a symlink which is not trusted by
appropriate user. since some users may wish to put symbolic links in
the zoneinfo directories to alias timezones, it seems preferable to
allow this.
Diffstat (limited to 'src')
| -rw-r--r-- | src/time/__map_file.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/time/__map_file.c b/src/time/__map_file.c index b322f095..84ae839f 100644 --- a/src/time/__map_file.c +++ b/src/time/__map_file.c @@ -9,7 +9,7 @@ const char unsigned *__map_file(const char *pathname, size_t *size) { struct stat st; const unsigned char *map = MAP_FAILED; - int flags = O_RDONLY|O_LARGEFILE|O_CLOEXEC|O_NOFOLLOW|O_NONBLOCK; + int flags = O_RDONLY|O_LARGEFILE|O_CLOEXEC|O_NONBLOCK; int fd = __syscall(SYS_open, pathname, flags); if (fd < 0) return 0; if (!__syscall(SYS_fstat, fd, &st)) |