X-Git-Url: https://git.octo.it/?a=blobdiff_plain;f=src%2Fcontextswitch.c;h=06055ca59372299616a6f8b1bf105d30e34760f4;hb=a72332c0ac9a2d806ac2a7bd5f8750d5b41f0bab;hp=0db727c78373a524ee15b3c5856f72e2c3f8f53a;hpb=11c3445e8931bbcefcb5b665a28fdae7691ebc6a;p=collectd.git

diff --git a/src/contextswitch.c b/src/contextswitch.c
index 0db727c7..06055ca5 100644
--- a/src/contextswitch.c
+++ b/src/contextswitch.c
@@ -27,12 +27,12 @@
 # error "No applicable input method."
 #endif
 
-static void cs_submit (unsigned long context_switches)
+static void cs_submit (derive_t context_switches)
 {
 	value_t values[1];
 	value_list_t vl = VALUE_LIST_INIT;
 
-	values[0].derive = context_switches;
+	values[0].derive = (derive_t) context_switches;
 
 	vl.values = values;
 	vl.values_len = 1;
@@ -48,8 +48,9 @@ static int cs_read (void)
 	FILE *fh;
 	char buffer[64];
 	int numfields;
-	char *fields[2];
-	unsigned long result = 0;
+	char *fields[3];
+	derive_t result = 0;
+	int status = -2;
 
 	fh = fopen ("/proc/stat", "r");
 	if (fh == NULL) {
@@ -58,35 +59,37 @@ static int cs_read (void)
 		return (-1);
 	}
 
-	while (fgets(buffer, sizeof(buffer), fh))
+	while (fgets(buffer, sizeof(buffer), fh) != NULL)
 	{
-		if (strncmp(buffer, "ctxt", 4))
+		char *endptr;
+
+		numfields = strsplit(buffer, fields, STATIC_ARRAY_SIZE (fields));
+		if (numfields != 2)
 			continue;
 
-		numfields = strsplit(buffer, fields, 2);
-		if (numfields != 2) {
-			ERROR ("contextswitch plugin: ctxt in /proc/stat contains more than 2 fields.");
-			break;
-		}
+		if (strcmp("ctxt", fields[0]) != 0)
+			continue;
 
-		result = strtoul(fields[1], NULL, 10);
-		if (errno == ERANGE && result == ULONG_MAX) {
-			ERROR ("contextswitch plugin: ctxt value in /proc/stat overflows.");
+		errno = 0;
+		endptr = NULL;
+		result = (derive_t) strtoll (fields[1], &endptr, /* base = */ 10);
+		if ((endptr == fields[1]) || (errno != 0)) {
+			ERROR ("contextswitch plugin: Cannot parse ctxt value: %s",
+					fields[1]);
+			status = -1;
 			break;
 		}
 
+		cs_submit(result);
+		status = 0;
 		break;
 	}
 	fclose(fh);
 
-	if (result == 0) {
-		ERROR ("contextswitch plugin: unable to find context switch value.");
-		return -1;
-	}
-
-	cs_submit(result);
+	if (status == -2)
+		ERROR ("contextswitch plugin: Unable to find context switch value.");
 
-	return 0;
+	return status;
 }
 
 void module_register (void)