From 073b439b2a19c58729e81b47def81f0ba5a23f08 Mon Sep 17 00:00:00 2001 From: Doug MacEachern Date: Thu, 31 Jan 2008 20:43:01 +0000 Subject: [PATCH] [SIGAR-85] Fix possible integer overflow in sigar_file_system_usage --- src/os/solaris/solaris_sigar.c | 16 ++++++++++------ 1 file changed, 10 insertions(+), 6 deletions(-) diff --git a/src/os/solaris/solaris_sigar.c b/src/os/solaris/solaris_sigar.c index 52c4c701..305aeefd 100644 --- a/src/os/solaris/solaris_sigar.c +++ b/src/os/solaris/solaris_sigar.c @@ -1358,9 +1358,6 @@ int sigar_file_system_list_get(sigar_t *sigar, #include -#define SIGAR_FS_BLOCKS_TO_BYTES(buf, f) \ - (((sigar_uint64_t)buf.f * (buf.f_frsize / 512)) >> 1) - typedef struct { char device[PATH_MAX]; char name[8]; @@ -1722,19 +1719,26 @@ int sigar_disk_usage_get(sigar_t *sigar, const char *name, return status; } +#define SIGAR_FS_BLOCKS_TO_BYTES(val, bsize) ((val * bsize) >> 1) + int sigar_file_system_usage_get(sigar_t *sigar, const char *dirname, sigar_file_system_usage_t *fsusage) { struct statvfs buf; + sigar_uint64_t val, bsize; if (statvfs(dirname, &buf) != 0) { return errno; } - fsusage->total = SIGAR_FS_BLOCKS_TO_BYTES(buf, f_blocks); - fsusage->free = SIGAR_FS_BLOCKS_TO_BYTES(buf, f_bfree); - fsusage->avail = SIGAR_FS_BLOCKS_TO_BYTES(buf, f_bavail); + bsize = buf.f_frsize / 512; + val = buf.f_blocks; + fsusage->total = SIGAR_FS_BLOCKS_TO_BYTES(val, bsize); + val = buf.f_bfree; + fsusage->free = SIGAR_FS_BLOCKS_TO_BYTES(val, bsize); + val = buf.f_bavail; + fsusage->avail = SIGAR_FS_BLOCKS_TO_BYTES(val, bsize); fsusage->used = fsusage->total - fsusage->free; fsusage->files = buf.f_files; fsusage->free_files = buf.f_ffree;