aboutsummaryrefslogtreecommitdiff
path: root/sysdeps
diff options
context:
space:
mode:
authorEric Wong <normalperson@yhbt.net>2014-05-29 09:41:29 +0530
committerSiddhesh Poyarekar <siddhesh@redhat.com>2014-05-29 09:41:29 +0530
commit26b0d2e1a1606da10c4dc4e109d7b9f2d92aab6a (patch)
tree80f00460ffda84fa25dae201b40c5442646cf213 /sysdeps
parent0d3b7a190c3abbdce27d4b0b9f16517bf5664099 (diff)
downloadglibc-26b0d2e1a1606da10c4dc4e109d7b9f2d92aab6a.zip
glibc-26b0d2e1a1606da10c4dc4e109d7b9f2d92aab6a.tar.gz
glibc-26b0d2e1a1606da10c4dc4e109d7b9f2d92aab6a.tar.bz2
Avoid stat/fstat in statvfs/fstatvfs (BZ #15132)
Delay the use of stat/fstat until stat data is required. When the kernel returns ST_VALID, stat data is not used by __internal_statvfs.
Diffstat (limited to 'sysdeps')
-rw-r--r--sysdeps/unix/sysv/linux/fstatvfs.c4
-rw-r--r--sysdeps/unix/sysv/linux/fstatvfs64.c10
-rw-r--r--sysdeps/unix/sysv/linux/internal_statvfs.c15
-rw-r--r--sysdeps/unix/sysv/linux/statvfs.c6
-rw-r--r--sysdeps/unix/sysv/linux/statvfs64.c10
5 files changed, 18 insertions, 27 deletions
diff --git a/sysdeps/unix/sysv/linux/fstatvfs.c b/sysdeps/unix/sysv/linux/fstatvfs.c
index da1f27e..689ab7a 100644
--- a/sysdeps/unix/sysv/linux/fstatvfs.c
+++ b/sysdeps/unix/sysv/linux/fstatvfs.c
@@ -22,7 +22,7 @@
#include <sys/statvfs.h>
extern void __internal_statvfs (const char *name, struct statvfs *buf,
- struct statfs *fsbuf, struct stat64 *st);
+ struct statfs *fsbuf, int fd);
int
@@ -36,7 +36,7 @@ fstatvfs (int fd, struct statvfs *buf)
return -1;
/* Convert the result. */
- __internal_statvfs (NULL, buf, &fsbuf, fstat64 (fd, &st) == -1 ? NULL : &st);
+ __internal_statvfs (NULL, buf, &fsbuf, fd);
/* We signal success if the statfs call succeeded. */
return 0;
diff --git a/sysdeps/unix/sysv/linux/fstatvfs64.c b/sysdeps/unix/sysv/linux/fstatvfs64.c
index dd10c1f..006b059 100644
--- a/sysdeps/unix/sysv/linux/fstatvfs64.c
+++ b/sysdeps/unix/sysv/linux/fstatvfs64.c
@@ -25,7 +25,7 @@
extern void __internal_statvfs64 (const char *name, struct statvfs64 *buf,
- struct statfs64 *fsbuf, struct stat64 *st);
+ struct statfs64 *fsbuf, int fd);
/* Return information about the filesystem on which FD resides. */
@@ -60,12 +60,8 @@ __fstatvfs64 (int fd, struct statvfs64 *buf)
#endif
if (res == 0)
- {
- /* Convert the result. */
- struct stat64 st;
- __internal_statvfs64 (NULL, buf, &fsbuf,
- fstat64 (fd, &st) == -1 ? NULL : &st);
- }
+ /* Convert the result. */
+ __internal_statvfs64 (NULL, buf, &fsbuf, fd);
return res;
}
diff --git a/sysdeps/unix/sysv/linux/internal_statvfs.c b/sysdeps/unix/sysv/linux/internal_statvfs.c
index 2424c13..d71ed0c 100644
--- a/sysdeps/unix/sysv/linux/internal_statvfs.c
+++ b/sysdeps/unix/sysv/linux/internal_statvfs.c
@@ -43,9 +43,11 @@
# ifndef __ASSUME_STATFS_F_FLAGS
int
-__statvfs_getflags (const char *name, int fstype, struct stat64 *st)
+__statvfs_getflags (const char *name, int fstype, int fd)
{
- if (st == NULL)
+ struct stat64 st;
+
+ if ((fd < 0 ? stat64 (name, &st) : fstat64 (fd, &st)) < 0)
return 0;
const char *fsname = NULL;
@@ -159,7 +161,7 @@ __statvfs_getflags (const char *name, int fstype, struct stat64 *st)
/* Find out about the device the current entry is for. */
struct stat64 fsst;
if (stat64 (mntbuf.mnt_dir, &fsst) >= 0
- && st->st_dev == fsst.st_dev)
+ && st.st_dev == fsst.st_dev)
{
/* Bingo, we found the entry for the device FD is on.
Now interpret the option string. */
@@ -222,14 +224,13 @@ __statvfs_getflags (const char *name, int fstype, struct stat64 *st)
}
# endif
#else
-extern int __statvfs_getflags (const char *name, int fstype,
- struct stat64 *st);
+extern int __statvfs_getflags (const char *name, int fstype, int fd);
#endif
void
INTERNAL_STATVFS (const char *name, struct STATVFS *buf,
- struct STATFS *fsbuf, struct stat64 *st)
+ struct STATFS *fsbuf, int fd)
{
/* Now fill in the fields we have information for. */
buf->f_bsize = fsbuf->f_bsize;
@@ -272,7 +273,7 @@ INTERNAL_STATVFS (const char *name, struct STATVFS *buf,
the /etc/mtab file and search for the entry which matches the given
file. The way we can test for matching filesystem is using the
device number. */
- buf->f_flag = __statvfs_getflags (name, fsbuf->f_type, st);
+ buf->f_flag = __statvfs_getflags (name, fsbuf->f_type, fd);
else
#endif
buf->f_flag = fsbuf->f_flags ^ ST_VALID;
diff --git a/sysdeps/unix/sysv/linux/statvfs.c b/sysdeps/unix/sysv/linux/statvfs.c
index 8a3df97..1427961 100644
--- a/sysdeps/unix/sysv/linux/statvfs.c
+++ b/sysdeps/unix/sysv/linux/statvfs.c
@@ -22,22 +22,20 @@
#include <sys/statvfs.h>
extern void __internal_statvfs (const char *name, struct statvfs *buf,
- struct statfs *fsbuf, struct stat64 *st);
+ struct statfs *fsbuf, int fd);
int
statvfs (const char *file, struct statvfs *buf)
{
struct statfs fsbuf;
- struct stat64 st;
/* Get as much information as possible from the system. */
if (__statfs (file, &fsbuf) < 0)
return -1;
/* Convert the result. */
- __internal_statvfs (file, buf, &fsbuf,
- stat64 (file, &st) == -1 ? NULL : &st);
+ __internal_statvfs (file, buf, &fsbuf, -1);
/* We signal success if the statfs call succeeded. */
return 0;
diff --git a/sysdeps/unix/sysv/linux/statvfs64.c b/sysdeps/unix/sysv/linux/statvfs64.c
index e33e923..0ec8524 100644
--- a/sysdeps/unix/sysv/linux/statvfs64.c
+++ b/sysdeps/unix/sysv/linux/statvfs64.c
@@ -26,7 +26,7 @@
extern void __internal_statvfs64 (const char *name, struct statvfs64 *buf,
- struct statfs64 *fsbuf, struct stat64 *st);
+ struct statfs64 *fsbuf, int fd);
/* Return information about the filesystem on which FILE resides. */
@@ -61,12 +61,8 @@ __statvfs64 (const char *file, struct statvfs64 *buf)
#endif
if (res == 0)
- {
- /* Convert the result. */
- struct stat64 st;
- __internal_statvfs64 (file, buf, &fsbuf,
- stat64 (file, &st) == -1 ? NULL : &st);
- }
+ /* Convert the result. */
+ __internal_statvfs64 (file, buf, &fsbuf, -1);
return res;
}