X-Git-Url: https://git.xinqibao.xyz/slstatus.git/blobdiff_plain/d47d78255ef8ede773c69a97e2cb5de78cee7da9..a18439af114b98ab026921b0cac61164f7a4eea8:/components/netspeeds.c

diff --git a/components/netspeeds.c b/components/netspeeds.c
index b4e6972..7ab1590 100644
--- a/components/netspeeds.c
+++ b/components/netspeeds.c
@@ -11,40 +11,52 @@
 	netspeed_rx(const char *interface)
 	{
 		uint64_t oldrxbytes;
-		static uint64_t rxbytes = 0;
+		static uint64_t rxbytes;
 		extern const unsigned int interval;
 		char path[PATH_MAX];
 
 		oldrxbytes = rxbytes;
 
-		snprintf(path, sizeof(path),
-		         "/sys/class/net/%s/statistics/rx_bytes", interface);
+		if (esnprintf(path, sizeof(path),
+		              "/sys/class/net/%s/statistics/rx_bytes",
+		              interface) < 0) {
+			return NULL;
+		}
 		if (pscanf(path, "%llu", &rxbytes) != 1) {
 			return NULL;
 		}
+		if (oldrxbytes == 0) {
+			return NULL;
+		}
 
-		return oldrxbytes ? fmt_scaled((rxbytes - oldrxbytes) /
-		                               interval * 1000) : NULL;
+		return fmt_human((rxbytes - oldrxbytes) * 1000 / interval,
+		                 1024);
 	}
 
 	const char *
 	netspeed_tx(const char *interface)
 	{
 		uint64_t oldtxbytes;
-		static uint64_t txbytes = 0;
+		static uint64_t txbytes;
 		extern const unsigned int interval;
 		char path[PATH_MAX];
 
 		oldtxbytes = txbytes;
 
-		snprintf(path, sizeof(path),
-		         "/sys/class/net/%s/statistics/tx_bytes", interface);
+		if (esnprintf(path, sizeof(path),
+		              "/sys/class/net/%s/statistics/tx_bytes",
+		              interface) < 0) {
+			return NULL;
+		}
 		if (pscanf(path, "%llu", &txbytes) != 1) {
 			return NULL;
 		}
+		if (oldtxbytes == 0) {
+			return NULL;
+		}
 
-		return oldtxbytes ? fmt_scaled((txbytes - oldtxbytes) /
-		                               interval * 1000) : NULL;
+		return fmt_human((txbytes - oldtxbytes) * 1000 / interval,
+		                 1024);
 	}
 #elif defined(__OpenBSD__)
 	#include <string.h>
@@ -59,7 +71,7 @@
 		struct ifaddrs *ifal, *ifa;
 		struct if_data *ifd;
 		uint64_t oldrxbytes;
-		static uint64_t rxbytes = 0;
+		static uint64_t rxbytes;
 		extern const unsigned int interval;
 		int if_ok = 0;
 
@@ -81,9 +93,12 @@
 			warn("reading 'if_data' failed");
 			return NULL;
 		}
+		if (oldrxbytes == 0) {
+			return NULL;
+		}
 
-		return oldrxbytes ? fmt_scaled((rxbytes - oldrxbytes) /
-		                               interval * 1000) : NULL;
+		return fmt_human((rxbytes - oldrxbytes) * 1000 / interval,
+		                 1024);
 	}
 
 	const char *
@@ -92,7 +107,7 @@
 		struct ifaddrs *ifal, *ifa;
 		struct if_data *ifd;
 		uint64_t oldtxbytes;
-		static uint64_t txbytes = 0;
+		static uint64_t txbytes;
 		extern const unsigned int interval;
 		int if_ok = 0;
 
@@ -114,8 +129,11 @@
 			warn("reading 'if_data' failed");
 			return NULL;
 		}
+		if (oldtxbytes == 0) {
+			return NULL;
+		}
 
-		return oldtxbytes ? fmt_scaled((txbytes - oldtxbytes) /
-		                               interval * 1000) : NULL;
+		return fmt_human((txbytes - oldtxbytes) * 1000 / interval,
+		                 1024);
 	}
 #endif