Merge remote-tracking branch 'github/pr/387'

[collectd.git] / src / redis.c

diff --git a/src/redis.c b/src/redis.c
index 9d26828eee03d35f45d049ca96696192a7fd5dda..85a8354629240553f7e60a65ae0c5cb72446ae29 100644 (file)
--- a/src/redis.c
+++ b/src/redis.c
@@ -28,7 +28,12 @@
 #include <pthread.h>
 #include <credis.h>
 
+#ifndef HOST_NAME_MAX
+# define HOST_NAME_MAX _POSIX_HOST_NAME_MAX
+#endif
+
 #define REDIS_DEF_HOST   "localhost"
+#define REDIS_DEF_PASSWD ""
 #define REDIS_DEF_PORT    6379
 #define REDIS_DEF_TIMEOUT 2000
 #define MAX_REDIS_NODE_NAME 64
@@ -133,7 +138,7 @@ static int redis_config_node (oconfig_item_t *ci) /* {{{ */
     }
     else if (strcasecmp ("Timeout", option->key) == 0)
       status = cf_util_get_int (option, &rn.timeout);
-    else if (strcasecmp ("Passwd", option->key) == 0)
+    else if (strcasecmp ("Password", option->key) == 0)
       status = cf_util_get_string_buffer (option, rn.passwd, sizeof (rn.passwd));
     else
       WARNING ("redis plugin: Option `%s' not allowed inside a `Node' "
@@ -225,8 +230,8 @@ static void redis_submit_d (char *plugin_instance,
 
 static int redis_init (void) /* {{{ */
 {
-  redis_node_t rn = { "default", REDIS_DEF_HOST, REDIS_DEF_PORT,
-    REDIS_DEF_TIMEOUT, /* next = */ NULL };
+  redis_node_t rn = { "default", REDIS_DEF_HOST, REDIS_DEF_PASSWD,
+    REDIS_DEF_PORT, REDIS_DEF_TIMEOUT, /* next = */ NULL };
 
   if (nodes_head == NULL)
     redis_node_add (&rn);
@@ -254,9 +259,11 @@ static int redis_read (void) /* {{{ */
       continue;
     }
 
-    if ( strlen(rn->passwd) > 0 ) {
+    if (strlen (rn->passwd) > 0)
+    {
       DEBUG ("redis plugin: authenticanting node `%s' passwd(%s).", rn->name, rn->passwd);
-      if ( credis_auth(rh, rn->passwd) != 0 )
+      status = credis_auth(rh, rn->passwd);
+      if (status != 0)
       {
         WARNING ("redis plugin: unable to authenticate on node `%s'.", rn->name);
         credis_close (rh);