swap_perc: check for division by zero on obsd too

[slstatus.git] / components / disk.c

diff --git a/components/disk.c b/components/disk.c
index 1251edb..998ad47 100644 (file)
--- a/components/disk.c
+++ b/components/disk.c
@@ -12,12 +12,11 @@ disk_free(const char *mnt)
        struct statvfs fs;
 
        if (statvfs(mnt, &fs) < 0) {
-               fprintf(stderr, "statvfs '%s': %s\n", mnt, strerror(errno));
+               warn("statvfs '%s':", mnt);
                return NULL;
        }
 
-       return bprintf("%f",
-                      (float)fs.f_bsize * (float)fs.f_bfree / 1024 / 1024 / 1024);
+       return fmt_human_2(fs.f_frsize * fs.f_bavail, "B");
 }
 
 const char *
@@ -26,12 +25,12 @@ disk_perc(const char *mnt)
        struct statvfs fs;
 
        if (statvfs(mnt, &fs) < 0) {
-               fprintf(stderr, "statvfs '%s': %s\n", mnt, strerror(errno));
+               warn("statvfs '%s':", mnt);
                return NULL;
        }
 
-       return bprintf("%d", (int)(100 *
-                                  (1.0f - ((float)fs.f_bfree / (float)fs.f_blocks))));
+       return bprintf("%d%%", (int)(100 *
+                      (1.0f - ((float)fs.f_bavail / (float)fs.f_blocks))));
 }
 
 const char *
@@ -40,12 +39,11 @@ disk_total(const char *mnt)
        struct statvfs fs;
 
        if (statvfs(mnt, &fs) < 0) {
-               fprintf(stderr, "statvfs '%s': %s\n", mnt, strerror(errno));
+               warn("statvfs '%s':", mnt);
                return NULL;
        }
 
-       return bprintf("%f",
-                      (float)fs.f_bsize * (float)fs.f_blocks / 1024 / 1024 / 1024);
+       return fmt_human_2(fs.f_frsize * fs.f_blocks, "B");
 }
 
 const char *
@@ -54,11 +52,9 @@ disk_used(const char *mnt)
        struct statvfs fs;
 
        if (statvfs(mnt, &fs) < 0) {
-               fprintf(stderr, "statvfs '%s': %s\n", mnt, strerror(errno));
+               warn("statvfs '%s':", mnt);
                return NULL;
        }
 
-       return bprintf("%f",
-                      (float)fs.f_bsize * ((float)fs.f_blocks -
-                      (float)fs.f_bfree) / 1024 / 1024 / 1024);
+       return fmt_human_2(fs.f_frsize * (fs.f_blocks - fs.f_bfree), "B");
 }