From: Luke Heberling <collectd@c-ware.com>
Date: Thu, 20 Aug 2009 12:10:37 +0000 (+0200)
Subject: powerdns plugin: Fix handling of the “LocalSocket” config option.
X-Git-Tag: collectd-4.6.5~19
X-Git-Url: https://git.octo.it/?a=commitdiff_plain;h=23cbc33ec79a98119b284c2db6f5a228a97cac3b;p=collectd.git

powerdns plugin: Fix handling of the “LocalSocket” config option.

 → See also: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=535787
---

diff --git a/src/powerdns.c b/src/powerdns.c
index 09ad8353..7288c372 100644
--- a/src/powerdns.c
+++ b/src/powerdns.c
@@ -923,11 +923,18 @@ static int powerdns_config (oconfig_item_t *ci) /* {{{ */
       powerdns_config_add_server (option);
     else if (strcasecmp ("LocalSocket", option->key) == 0)
     {
-      char *temp = strdup (option->key);
-      if (temp == NULL)
-        return (1);
-      sfree (local_sockpath);
-      local_sockpath = temp;
+      if ((option->values_num != 1) || (option->values[0].type != OCONFIG_TYPE_STRING))
+      {
+        WARNING ("powerdns plugin: `%s' needs exactly one string argument.", option->key);
+      }
+      else
+      {
+        char *temp = strdup (option->values[0].value.string);
+        if (temp == NULL)
+          return (1);
+        sfree (local_sockpath);
+        local_sockpath = temp;
+      }
     }
     else
     {