Xinqi Bao's Git

swap_perc: check for division by zero
[slstatus.git] / components / swap.c
index 73d24df..465ffd4 100644 (file)
@@ -48,7 +48,7 @@
                }
                sscanf(match, "SwapFree: %ld kB\n", &free);
 
                }
                sscanf(match, "SwapFree: %ld kB\n", &free);
 
-               return fmt_scaled(free * 1024);
+               return fmt_human_2(free * 1024, "B");
        }
 
        const char *
        }
 
        const char *
                }
                sscanf(match, "SwapFree: %ld kB\n", &free);
 
                }
                sscanf(match, "SwapFree: %ld kB\n", &free);
 
-               return bprintf("%d", 100 * (total - free - cached) / total);
+               if (total == 0) {
+                       return NULL;
+               }
+
+               return bprintf("%d%%", 100 * (total - free - cached) / total);
        }
 
        const char *
        }
 
        const char *
@@ -94,7 +98,7 @@
                }
                sscanf(match, "SwapTotal: %ld kB\n", &total);
 
                }
                sscanf(match, "SwapTotal: %ld kB\n", &total);
 
-               return fmt_scaled(total * 1024);
+               return fmt_human_2(total * 1024, "B");
        }
 
        const char *
        }
 
        const char *
                }
                sscanf(match, "SwapFree: %ld kB\n", &free);
 
                }
                sscanf(match, "SwapFree: %ld kB\n", &free);
 
-               return fmt_scaled((total - free - cached) * 1024);
+               return fmt_human_2((total - free - cached) * 1024, "B");
        }
 #elif defined(__OpenBSD__)
        #include <stdlib.h>
        }
 #elif defined(__OpenBSD__)
        #include <stdlib.h>
 
                getstats(&total, &used);
 
 
                getstats(&total, &used);
 
-               return fmt_scaled((total - used) * 1024);
+               return fmt_human_2((total - used) * 1024, "B");
        }
 
        const char *
        }
 
        const char *
 
                getstats(&total, &used);
 
 
                getstats(&total, &used);
 
-               return bprintf("%d", 100 * used / total);
+               return bprintf("%d%%", 100 * used / total);
        }
 
        const char *
        }
 
        const char *
 
                getstats(&total, &used);
 
 
                getstats(&total, &used);
 
-               return fmt_scaled(total * 1024);
+               return fmt_human_2(total * 1024, "B");
        }
 
        const char *
        }
 
        const char *
 
                getstats(&total, &used);
 
 
                getstats(&total, &used);
 
-               return fmt_scaled(used * 1024);
+               return fmt_human_2(used * 1024, "B");
        }
 #endif
        }
 #endif