From: Florian Forster <octo@leeloo.lan.home.verplant.org>
Date: Wed, 4 Aug 2010 12:43:38 +0000 (+0200)
Subject: redis plugin: Use "sizeof" rather than repeating the size.
X-Git-Tag: collectd-5.0.0-beta0~50^2~8
X-Git-Url: https://git.verplant.org/?a=commitdiff_plain;h=47d5efe6eae80527119114892aa539d5bfd925df;p=collectd.git

redis plugin: Use "sizeof" rather than repeating the size.
---

diff --git a/src/redis.c b/src/redis.c
index dfac6d63..ec69631d 100644
--- a/src/redis.c
+++ b/src/redis.c
@@ -81,7 +81,7 @@ static int redis_config_node (redis_node_t *rn, oconfig_item_t *ci) /* {{{ */
     status = 0;
 
     if (strcasecmp ("Host", option->key) == 0)
-      status = cf_util_get_string_buffer (option, rn->host, HOST_NAME_MAX);
+      status = cf_util_get_string_buffer (option, rn->host, sizeof (rn->host));
     else if (strcasecmp ("Port", option->key) == 0)
       status = rn->port = cf_util_get_port_number (option);
     else if (strcasecmp ("Timeout", option->key) == 0)
@@ -125,7 +125,7 @@ static int redis_node_add (const redis_node_t *rn) /* {{{ */
   memcpy (rn_copy, rn, sizeof (redis_node_t));
   if (*rn_copy->name == '\0')
   {
-    (void) strncpy(rn_copy->name, "default", MAX_REDIS_NODE_NAME); /* in theory never fails */
+    (void) strncpy(rn_copy->name, "default", sizeof (rn_copy->name)); /* in theory never fails */
   }
 
   DEBUG ("redis plugin: adding entry `%s' to the tree.", rn_copy->name);
@@ -183,7 +183,7 @@ static int redis_config (oconfig_item_t *ci) /* {{{ */
         status = redis_node_add (&rn);
     }
     else if (strcasecmp ("Host", option->key) == 0)
-      status = cf_util_get_string_buffer (option, rn.host, HOST_NAME_MAX);
+      status = cf_util_get_string_buffer (option, rn.host, sizeof (rn.host));
     else if (strcasecmp ("Port", option->key) == 0)
       status = rn.port = cf_util_get_port_number (option);
     else if (strcasecmp ("Timeout", option->key) == 0)