Code

nginx plugin: Use strcmp rather than strcpy to compare strings.
authorSaulius Grigaliunas <saulius.grigaliunas@gmail.com>
Thu, 13 Dec 2007 06:58:51 +0000 (07:58 +0100)
committerFlorian Forster <octo@huhu.verplant.org>
Thu, 13 Dec 2007 06:58:51 +0000 (07:58 +0100)
I believe I found a typo in nginx.c which prevents collectd with nginx plugin
from working. Collectd segfaults at startup. Here is a tiny patch to correct
the issue. When applied, everything works as expected.

Signed-off-by: Florian Forster <octo@huhu.verplant.org>
src/nginx.c

index 69dc49e227b161d92461a7960531ee810cc1cbec..a44e8a5778bb9de85802c599a91574745048458e 100644 (file)
@@ -141,9 +141,9 @@ static void submit (char *type, char *inst, long long value)
   value_t values[1];
   value_list_t vl = VALUE_LIST_INIT;
 
-  if (strcpy (type, "nginx_connections") == 0)
+  if (strcmp (type, "nginx_connections") == 0)
     values[0].gauge = value;
-  else if (strcpy (type, "nginx_requests") == 0)
+  else if (strcmp (type, "nginx_requests") == 0)
     values[0].counter = value;
   else
     return;