From: Florian Forster <octo@leeloo.lan.home.verplant.org>
Date: Fri, 2 Oct 2009 09:37:05 +0000 (+0200)
Subject: contextswitches, processes plugins: Don't print an error if number of fields don... 
X-Git-Tag: collectd-4.9.0~56
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=53f791cb6b55512101a0a0f42f9d91c356fa9591;p=thirdparty%2Fcollectd.git

contextswitches, processes plugins: Don't print an error if number of fields don't match.

Because we pick the required line further down, having lines with more
fields is not illegal anymore.

Thanks to Patrik Weiskircher for noticing :)
---

diff --git a/src/contextswitch.c b/src/contextswitch.c
index b53b5740b..7787203dd 100644
--- a/src/contextswitch.c
+++ b/src/contextswitch.c
@@ -64,11 +64,8 @@ static int cs_read (void)
 		char *endptr;
 
 		numfields = strsplit(buffer, fields, STATIC_ARRAY_SIZE (fields));
-		if (numfields != 2) {
-			ERROR ("contextswitch plugin: ctxt in /proc/stat "
-					"contains more than 2 fields.");
+		if (numfields != 2)
 			continue;
-		}
 
 		if (strcmp("ctxt", fields[0]) != 0)
 			continue;
diff --git a/src/processes.c b/src/processes.c
index 8b937968e..5f67abaa1 100644
--- a/src/processes.c
+++ b/src/processes.c
@@ -861,11 +861,9 @@ static unsigned long read_fork_rate ()
 		char *endptr;
 
 		numfields = strsplit(buf, fields, STATIC_ARRAY_SIZE (fields));
-		if (numfields != 2) {
-			ERROR ("processes plugin: processes in /proc/stat "
-					"contains more than 2 fields.");
+		if (numfields != 2)
 			continue;
-		}
+
 		if (strcmp ("processes", fields[0]) != 0)
 			continue;