X-Git-Url: https://git.verplant.org/?a=blobdiff_plain;f=src%2Fconntrack.c;h=26f62786b242872dd89c4a943358f3d0164b2d45;hb=b42b7eaaaa3d2b375dc17b52f8858c0ebe2f5f40;hp=e7bccad353064f6e8d16a6b6420a303b51a017a0;hpb=575e601fb46732bc0538ef92ea6ac3b00ef3aafd;p=collectd.git

diff --git a/src/conntrack.c b/src/conntrack.c
index e7bccad3..26f62786 100644
--- a/src/conntrack.c
+++ b/src/conntrack.c
@@ -18,10 +18,11 @@
  * Authors:
  *   Tomasz Pala <gotar at pld-linux.org>
  * based on entropy.c by:
- *   Florian octo Forster <octo at verplant.org>
+ *   Florian octo Forster <octo at collectd.org>
  **/
 
 #include "collectd.h"
+
 #include "common.h"
 #include "plugin.h"
 
@@ -31,6 +32,27 @@
 
 #define CONNTRACK_FILE "/proc/sys/net/netfilter/nf_conntrack_count"
 #define CONNTRACK_MAX_FILE "/proc/sys/net/netfilter/nf_conntrack_max"
+#define CONNTRACK_FILE_OLD "/proc/sys/net/ipv4/netfilter/ip_conntrack_count"
+#define CONNTRACK_MAX_FILE_OLD "/proc/sys/net/ipv4/netfilter/ip_conntrack_max"
+
+static const char *config_keys[] =
+{
+	"OldFiles"
+};
+static int config_keys_num = STATIC_ARRAY_SIZE (config_keys);
+/*
+    Each table/chain combo that will be queried goes into this list
+*/
+
+static int old_files = 0;
+
+static int conntrack_config(const char *key, const char *value)
+{
+    if (strcmp(key, "OldFiles") == 0)
+        old_files = 1;
+
+    return 0;
+}
 
 static void conntrack_submit (const char *type, const char *type_instance,
 			      value_t conntrack)
@@ -39,7 +61,6 @@ static void conntrack_submit (const char *type, const char *type_instance,
 
 	vl.values = &conntrack;
 	vl.values_len = 1;
-	sstrncpy (vl.host, hostname_g, sizeof (vl.host));
 	sstrncpy (vl.plugin, "conntrack", sizeof (vl.plugin));
 	sstrncpy (vl.type, type, sizeof (vl.type));
 	if (type_instance != NULL)
@@ -52,67 +73,33 @@ static void conntrack_submit (const char *type, const char *type_instance,
 static int conntrack_read (void)
 {
 	value_t conntrack, conntrack_max, conntrack_pct;
-	FILE *fh;
-	char buffer[64];
-	size_t buffer_len;
-
-	fh = fopen (CONNTRACK_FILE, "r");
-	if (fh == NULL)
-		return (-1);
 
-	memset (buffer, 0, sizeof (buffer));
-	if (fgets (buffer, sizeof (buffer), fh) == NULL)
+	char const *path = old_files ? CONNTRACK_FILE_OLD : CONNTRACK_FILE;
+	if (parse_value_file (path, &conntrack, DS_TYPE_GAUGE) != 0)
 	{
-		fclose (fh);
+		ERROR ("conntrack plugin: Reading \"%s\" failed.", path);
 		return (-1);
 	}
-	fclose (fh);
-
-	/* strip trailing newline. */
-	buffer_len = strlen (buffer);
-	while ((buffer_len > 0) && isspace ((int) buffer[buffer_len - 1]))
-	{
-		buffer[buffer_len - 1] = 0;
-		buffer_len--;
-	}
-
-	if (parse_value (buffer, &conntrack, DS_TYPE_GAUGE) != 0)
-		return (-1);
-
-	conntrack_submit ("conntrack", NULL, conntrack);
-
-	fh = fopen (CONNTRACK_MAX_FILE, "r");
-	if (fh == NULL)
-		return (-1);
 
-	memset (buffer, 0, sizeof (buffer));
-	if (fgets (buffer, sizeof (buffer), fh) == NULL)
+	path = old_files ? CONNTRACK_MAX_FILE_OLD : CONNTRACK_MAX_FILE;
+	if (parse_value_file (path, &conntrack_max, DS_TYPE_GAUGE) != 0)
 	{
-		fclose (fh);
+		ERROR ("conntrack plugin: Reading \"%s\" failed.", path);
 		return (-1);
 	}
-	fclose (fh);
-
-	/* strip trailing newline. */
-	buffer_len = strlen (buffer);
-	while ((buffer_len > 0) && isspace ((int) buffer[buffer_len - 1]))
-	{
-		buffer[buffer_len - 1] = 0;
-		buffer_len--;
-	}
-
-	if (parse_value (buffer, &conntrack_max, DS_TYPE_GAUGE) != 0)
-		return (-1);
 
-	conntrack_submit ("conntrack", "max", conntrack_max);
 	conntrack_pct.gauge = (conntrack.gauge / conntrack_max.gauge) * 100;
-	conntrack_submit ("percent", "used", conntrack_pct);
 
+	conntrack_submit ("conntrack", NULL,   conntrack);
+	conntrack_submit ("conntrack", "max",  conntrack_max);
+	conntrack_submit ("percent",   "used", conntrack_pct);
 
 	return (0);
 } /* static int conntrack_read */
 
 void module_register (void)
 {
+	plugin_register_config ("conntrack", conntrack_config,
+                            config_keys, config_keys_num);
 	plugin_register_read ("conntrack", conntrack_read);
 } /* void module_register */