Beautifying many debug messages..
[collectd.git] / src / unixsock.c
index 84d49fd..ddf4b67 100644 (file)
 #include "common.h"
 #include "plugin.h"
 #include "configfile.h"
-#include "utils_debug.h"
 
 /* Folks without pthread will need to disable this plugin. */
 #include <pthread.h>
 
 #include <sys/socket.h>
 #include <sys/un.h>
-#include <sys/poll.h>
 
 #include <grp.h>
 
@@ -50,6 +48,7 @@ typedef struct value_cache_s
        int        values_num;
        gauge_t   *gauge;
        counter_t *counter;
+       const data_set_t *ds;
        time_t     time;
        struct value_cache_s *next;
 } value_cache_t;
@@ -67,13 +66,15 @@ static const char *config_keys[] =
 };
 static int config_keys_num = 3;
 
+static int loop = 0;
+
 /* socket configuration */
 static int   sock_fd    = -1;
 static char *sock_file  = NULL;
 static char *sock_group = NULL;
 static int   sock_perms = S_IRWXU | S_IRWXG;
 
-static pthread_t listen_thread = -1;
+static pthread_t listen_thread = (pthread_t) 0;
 
 /* Linked list and auxilliary variables for saving values */
 static value_cache_t   *cache_head = NULL;
@@ -96,65 +97,44 @@ static value_cache_t *cache_search (const char *name)
        return (vc);
 } /* value_cache_t *cache_search */
 
-static int cache_alloc_name (char *ret, int ret_len,
-               const char *plugin, const char *plugin_instance,
-               const char *type, const char *type_instance)
-{
-       int  status;
-
-       assert (plugin != NULL);
-       assert (type != NULL);
-
-       if ((plugin_instance == NULL) || (strlen (plugin_instance) == 0))
-       {
-               if ((type_instance == NULL) || (strlen (type_instance) == 0))
-                       status = snprintf (ret, ret_len, "%s/%s",
-                                       plugin, type);
-               else
-                       status = snprintf (ret, ret_len, "%s/%s-%s",
-                                       plugin, type, type_instance);
-       }
-       else
-       {
-               if ((type_instance == NULL) || (strlen (type_instance) == 0))
-                       status = snprintf (ret, ret_len, "%s-%s/%s",
-                                       plugin, plugin_instance, type);
-               else
-                       status = snprintf (ret, ret_len, "%s-%s/%s-%s",
-                                       plugin, plugin_instance,
-                                       type, type_instance);
-       }
-
-       if ((status < 1) || (status >= ret_len))
-               return (-1);
-       return (0);
-} /* int cache_alloc_name */
-
 static int cache_insert (const data_set_t *ds, const value_list_t *vl)
 {
        /* We're called from `cache_update' so we don't need to lock the mutex */
        value_cache_t *vc;
        int i;
 
-       DBG ("ds->ds_num = %i; vl->values_len = %i;",
-                       ds->ds_num, vl->values_len);
+       DEBUG ("unixsock plugin: cache_insert: ds->type = %s; ds->ds_num = %i;"
+                       " vl->values_len = %i;",
+                       ds->type, ds->ds_num, vl->values_len);
+#if COLLECT_DEBUG
        assert (ds->ds_num == vl->values_len);
+#else
+       if (ds->ds_num != vl->values_len)
+       {
+               ERROR ("unixsock plugin: ds->type = %s: (ds->ds_num = %i) != "
+                               "(vl->values_len = %i)",
+                               ds->type, ds->ds_num, vl->values_len);
+               return (-1);
+       }
+#endif
 
        vc = (value_cache_t *) malloc (sizeof (value_cache_t));
        if (vc == NULL)
        {
+               char errbuf[1024];
                pthread_mutex_unlock (&cache_lock);
-               syslog (LOG_ERR, "unixsock plugin: malloc failed: %s",
-                               strerror (errno));
+               ERROR ("unixsock plugin: malloc failed: %s",
+                               sstrerror (errno, errbuf, sizeof (errbuf)));
                return (-1);
        }
 
        vc->gauge = (gauge_t *) malloc (sizeof (gauge_t) * vl->values_len);
        if (vc->gauge == NULL)
        {
+               char errbuf[1024];
                pthread_mutex_unlock (&cache_lock);
-               syslog (LOG_ERR, "unixsock plugin: malloc failed: %s",
-                               strerror (errno));
+               ERROR ("unixsock plugin: malloc failed: %s",
+                               sstrerror (errno, errbuf, sizeof (errbuf)));
                free (vc);
                return (-1);
        }
@@ -162,20 +142,19 @@ static int cache_insert (const data_set_t *ds, const value_list_t *vl)
        vc->counter = (counter_t *) malloc (sizeof (counter_t) * vl->values_len);
        if (vc->counter == NULL)
        {
+               char errbuf[1024];
                pthread_mutex_unlock (&cache_lock);
-               syslog (LOG_ERR, "unixsock plugin: malloc failed: %s",
-                               strerror (errno));
+               ERROR ("unixsock plugin: malloc failed: %s",
+                               sstrerror (errno, errbuf, sizeof (errbuf)));
                free (vc->gauge);
                free (vc);
                return (-1);
        }
 
-       if (cache_alloc_name (vc->name, sizeof (vc->name),
-                               vl->plugin, vl->plugin_instance,
-                               ds->type, vl->type_instance) != 0)
+       if (FORMAT_VL (vc->name, sizeof (vc->name), vl, ds))
        {
                pthread_mutex_unlock (&cache_lock);
-               syslog (LOG_ERR, "unixsock plugin: cache_alloc_name failed.");
+               ERROR ("unixsock plugin: FORMAT_VL failed.");
                free (vc->counter);
                free (vc->gauge);
                free (vc);
@@ -201,6 +180,7 @@ static int cache_insert (const data_set_t *ds, const value_list_t *vl)
                }
        }
        vc->values_num = ds->ds_num;
+       vc->ds = ds;
 
        vc->next = cache_head;
        cache_head = vc;
@@ -219,21 +199,31 @@ static int cache_update (const data_set_t *ds, const value_list_t *vl)
        value_cache_t *vc;
        int i;
 
-       if (cache_alloc_name (name, sizeof (name),
-                               vl->plugin, vl->plugin_instance,
-                               ds->type, vl->type_instance) != 0)
+       if (FORMAT_VL (name, sizeof (name), vl, ds) != 0)
                return (-1);
 
        pthread_mutex_lock (&cache_lock);
 
        vc = cache_search (name);
 
+       /* pthread_mutex_lock is called by cache_insert. */
        if (vc == NULL)
                return (cache_insert (ds, vl));
 
        assert (vc->values_num == ds->ds_num);
        assert (vc->values_num == vl->values_len);
 
+       /* Avoid floating-point exceptions due to division by zero. */
+       if (vc->time >= vl->time)
+       {
+               pthread_mutex_unlock (&cache_lock);
+               ERROR ("unixsock plugin: vc->time >= vl->time. vc->time = %u; "
+                               "vl->time = %u; vl = %s;",
+                               (unsigned int) vc->time, (unsigned int) vl->time,
+                               name);
+               return (-1);
+       } /* if (vc->time >= vl->time) */
+
        /*
         * Update the values. This is possibly a lot more that you'd expect
         * because we honor min and max values and handle counter overflows here.
@@ -261,20 +251,12 @@ static int cache_update (const data_set_t *ds, const value_list_t *vl)
                                        / (vl->time - vc->time);
                        }
 
-                       DBG ("name = %s; old counter: %llu; new counter: %llu; rate: %lf;",
-                                       name,
-                                       vc->counter[i], vl->values[i].counter,
-                                       vc->gauge[i]);
-
                        vc->counter[i] = vl->values[i].counter;
                }
                else if (ds->ds[i].type == DS_TYPE_GAUGE)
                {
                        vc->gauge[i] = vl->values[i].gauge;
                        vc->counter[i] = 0;
-
-                       DBG ("name, %s; gauge: %lf;",
-                                       name, vc->gauge[i]);
                }
                else
                {
@@ -282,12 +264,13 @@ static int cache_update (const data_set_t *ds, const value_list_t *vl)
                        vc->counter[i] = 0;
                }
 
-               if ((vc->gauge[i] == NAN)
-                               || ((ds->ds[i].min != NAN) && (vc->gauge[i] < ds->ds[i].min))
-                               || ((ds->ds[i].max != NAN) && (vc->gauge[i] > ds->ds[i].max)))
+               if (isnan (vc->gauge[i])
+                               || (!isnan (ds->ds[i].min) && (vc->gauge[i] < ds->ds[i].min))
+                               || (!isnan (ds->ds[i].max) && (vc->gauge[i] > ds->ds[i].max)))
                        vc->gauge[i] = NAN;
        } /* for i = 0 .. ds->ds_num */
 
+       vc->ds = ds;
        vc->time = vl->time;
 
        if (vc->time < cache_oldest)
@@ -356,8 +339,9 @@ static int us_open_socket (void)
        sock_fd = socket (PF_UNIX, SOCK_STREAM, 0);
        if (sock_fd < 0)
        {
-               syslog (LOG_ERR, "unixsock plugin: socket failed: %s",
-                               strerror (errno));
+               char errbuf[1024];
+               ERROR ("unixsock plugin: socket failed: %s",
+                               sstrerror (errno, errbuf, sizeof (errbuf)));
                return (-1);
        }
 
@@ -367,13 +351,14 @@ static int us_open_socket (void)
                        sizeof (sa.sun_path) - 1);
        /* unlink (sa.sun_path); */
 
+       DEBUG ("unixsock plugin: socket path = %s", sa.sun_path);
+
        status = bind (sock_fd, (struct sockaddr *) &sa, sizeof (sa));
        if (status != 0)
        {
-               DBG ("bind failed: %s; sa.sun_path = %s",
-                               strerror (errno), sa.sun_path);
-               syslog (LOG_ERR, "unixsock plugin: bind failed: %s",
-                               strerror (errno));
+               char errbuf[1024];
+               sstrerror (errno, errbuf, sizeof (errbuf));
+               ERROR ("unixsock plugin: bind failed: %s", errbuf);
                close (sock_fd);
                sock_fd = -1;
                return (-1);
@@ -382,8 +367,9 @@ static int us_open_socket (void)
        status = listen (sock_fd, 8);
        if (status != 0)
        {
-               syslog (LOG_ERR, "unixsock plugin: listen failed: %s",
-                               strerror (errno));
+               char errbuf[1024];
+               ERROR ("unixsock plugin: listen failed: %s",
+                               sstrerror (errno, errbuf, sizeof (errbuf)));
                close (sock_fd);
                sock_fd = -1;
                return (-1);
@@ -391,29 +377,37 @@ static int us_open_socket (void)
 
        do
        {
+               char *grpname;
                struct group *g;
+               struct group sg;
+               char grbuf[2048];
 
-               errno = 0;
-               g = getgrnam ((sock_group != NULL) ? sock_group : COLLECTD_GRP_NAME);
+               grpname = (sock_group != NULL) ? sock_group : COLLECTD_GRP_NAME;
+               g = NULL;
 
-               if (errno != 0)
+               status = getgrnam_r (grpname, &sg, grbuf, sizeof (grbuf), &g);
+               if (status != 0)
                {
-                       syslog (LOG_WARNING, "unixsock plugin: getgrnam (%s) failed: %s",
-                                       (sock_group != NULL) ? sock_group : COLLECTD_GRP_NAME,
-                                       strerror (errno));
+                       char errbuf[1024];
+                       WARNING ("unixsock plugin: getgrnam_r (%s) failed: %s", grpname,
+                                       sstrerror (errno, errbuf, sizeof (errbuf)));
                        break;
                }
-
                if (g == NULL)
+               {
+                       WARNING ("unixsock plugin: No such group: `%s'",
+                                       grpname);
                        break;
+               }
 
                if (chown ((sock_file != NULL) ? sock_file : US_DEFAULT_PATH,
                                        (uid_t) -1, g->gr_gid) != 0)
                {
-                       syslog (LOG_WARNING, "unixsock plugin: chown (%s, -1, %i) failed: %s",
+                       char errbuf[1024];
+                       WARNING ("unixsock plugin: chown (%s, -1, %i) failed: %s",
                                        (sock_file != NULL) ? sock_file : US_DEFAULT_PATH,
                                        (int) g->gr_gid,
-                                       strerror (errno));
+                                       sstrerror (errno, errbuf, sizeof (errbuf)));
                }
        } while (0);
 
@@ -422,7 +416,8 @@ static int us_open_socket (void)
 
 static int us_handle_getval (FILE *fh, char **fields, int fields_num)
 {
-       char *plugin = fields[1];
+       char *hostname;
+       char *plugin;
        char *plugin_instance;
        char *type;
        char *type_instance;
@@ -432,43 +427,56 @@ static int us_handle_getval (FILE *fh, char **fields, int fields_num)
        int   i;
 
        if (fields_num != 2)
-               return (-1);
-
-       type = strchr (plugin, '/');
-       if (type == NULL)
-               return (-1);
-       *type = '\0'; type++;
-
-       plugin_instance = strchr (plugin, '-');
-       if (plugin_instance != NULL)
        {
-               *plugin_instance = '\0';
-               plugin_instance++;
+               DEBUG ("unixsock plugin: Wrong number of fields: %i", fields_num);
+               fprintf (fh, "-1 Wrong number of fields: Got %i, expected 2.\n",
+                               fields_num);
+               fflush (fh);
+               return (-1);
        }
+       DEBUG ("unixsock plugin: Got query for `%s'", fields[1]);
 
-       type_instance = strchr (type, '-');
-       if (type_instance != NULL)
+       status = parse_identifier (fields[1], &hostname,
+                       &plugin, &plugin_instance,
+                       &type, &type_instance);
+       if (status != 0)
        {
-               *type_instance = '\0';
-               type_instance++;
+               DEBUG ("unixsock plugin: Cannot parse `%s'", fields[1]);
+               fprintf (fh, "-1 Cannot parse identifier.\n");
+               fflush (fh);
+               return (-1);
        }
 
-       status = cache_alloc_name (name, sizeof (name),
-                       plugin, plugin_instance, type, type_instance);
+       status = format_name (name, sizeof (name),
+                       hostname, plugin, plugin_instance, type, type_instance);
        if (status != 0)
+       {
+               fprintf (fh, "-1 format_name failed.\n");
                return (-1);
+       }
 
        pthread_mutex_lock (&cache_lock);
 
+       DEBUG ("vc = cache_search (%s)", name);
        vc = cache_search (name);
 
-       fprintf (fh, "%i", vc->values_num);
-       for (i = 0; i < vc->values_num; i++)
+       if (vc == NULL)
        {
-               if (vc->gauge[i] == NAN)
-                       fprintf (fh, " NaN");
-               else
-                       fprintf (fh, " %12e", vc->gauge[i]);
+               DEBUG ("Did not find cache entry.");
+               fprintf (fh, "-1 No such value");
+       }
+       else
+       {
+               DEBUG ("Found cache entry.");
+               fprintf (fh, "%i", vc->values_num);
+               for (i = 0; i < vc->values_num; i++)
+               {
+                       fprintf (fh, " %s=", vc->ds->ds[i].name);
+                       if (isnan (vc->gauge[i]))
+                               fprintf (fh, "NaN");
+                       else
+                               fprintf (fh, "%12e", vc->gauge[i]);
+               }
        }
 
        /* Free the mutex as soon as possible and definitely before flushing */
@@ -480,6 +488,149 @@ static int us_handle_getval (FILE *fh, char **fields, int fields_num)
        return (0);
 } /* int us_handle_getval */
 
+static int us_handle_putval (FILE *fh, char **fields, int fields_num)
+{
+       char *hostname;
+       char *plugin;
+       char *plugin_instance;
+       char *type;
+       char *type_instance;
+       int   status;
+       int   i;
+
+       const data_set_t *ds;
+       value_list_t vl = VALUE_LIST_INIT;
+
+       char **value_ptr;
+
+       if (fields_num != 3)
+       {
+               DEBUG ("unixsock plugin: Wrong number of fields: %i", fields_num);
+               fprintf (fh, "-1 Wrong number of fields: Got %i, expected 3.\n",
+                               fields_num);
+               fflush (fh);
+               return (-1);
+       }
+
+       status = parse_identifier (fields[1], &hostname,
+                       &plugin, &plugin_instance,
+                       &type, &type_instance);
+       if (status != 0)
+       {
+               DEBUG ("unixsock plugin: Cannot parse `%s'", fields[1]);
+               fprintf (fh, "-1 Cannot parse identifier.\n");
+               fflush (fh);
+               return (-1);
+       }
+
+       if ((strlen (hostname) >= sizeof (vl.host))
+                       || (strlen (plugin) >= sizeof (vl.plugin))
+                       || ((plugin_instance != NULL)
+                               && (strlen (plugin_instance) >= sizeof (vl.plugin_instance)))
+                       || ((type_instance != NULL)
+                               && (strlen (type_instance) >= sizeof (vl.type_instance))))
+       {
+               fprintf (fh, "-1 Identifier too long.");
+               return (-1);
+       }
+
+       strcpy (vl.host, hostname);
+       strcpy (vl.plugin, plugin);
+       if (plugin_instance != NULL)
+               strcpy (vl.plugin_instance, plugin_instance);
+       if (type_instance != NULL)
+               strcpy (vl.type_instance, type_instance);
+
+       { /* parse the time */
+               char *t = fields[2];
+               char *v = strchr (t, ':');
+               if (v == NULL)
+               {
+                       fprintf (fh, "-1 No time found.");
+                       return (-1);
+               }
+               *v = '\0'; v++;
+
+               vl.time = (time_t) atoi (t);
+               if (vl.time == 0)
+                       vl.time = time (NULL);
+
+               fields[2] = v;
+       }
+
+       ds = plugin_get_ds (type);
+       if (ds == NULL)
+               return (-1);
+
+       value_ptr = (char **) calloc (ds->ds_num, sizeof (char *));
+       if (value_ptr == NULL)
+       {
+               fprintf (fh, "-1 calloc failed.");
+               return (-1);
+       }
+
+       { /* parse the value-list. It's colon-separated. */
+               char *dummy;
+               char *ptr;
+               char *saveptr;
+
+               i = 0;
+               dummy = fields[2];
+               saveptr = NULL;
+               while ((ptr = strtok_r (dummy, ":", &saveptr)) != NULL)
+               {
+                       dummy = NULL;
+                       if (i >= ds->ds_num)
+                       {
+                               i = ds->ds_num + 1;
+                               break;
+                       }
+                       value_ptr[i] = ptr;
+                       i++;
+               }
+
+               if (i != ds->ds_num)
+               {
+                       sfree (value_ptr);
+                       fprintf (fh, "-1 Number of values incorrect: Got %i, "
+                                       "expected %i.", i, ds->ds_num);
+                       return (-1);
+               }
+       } /* done parsing the value-list */
+
+       vl.values_len = ds->ds_num;
+       vl.values = (value_t *) malloc (vl.values_len * sizeof (value_t));
+       if (vl.values == NULL)
+       {
+               sfree (value_ptr);
+               fprintf (fh, "-1 malloc failed.");
+               return (-1);
+       }
+       DEBUG ("value_ptr = 0x%p; vl.values = 0x%p;", (void *) value_ptr, (void *) vl.values);
+
+       for (i = 0; i < ds->ds_num; i++)
+       {
+               if (strcmp (value_ptr[i], "U") == 0)
+                       vl.values[i].gauge = NAN;
+               else if (ds->ds[i].type == DS_TYPE_COUNTER)
+                       vl.values[i].counter = atoll (value_ptr[i]);
+               else if (ds->ds[i].type == DS_TYPE_GAUGE)
+                       vl.values[i].gauge = atof (value_ptr[i]);
+       } /* for (i = 2 .. fields_num) */
+
+       plugin_dispatch_values (type, &vl);
+
+       DEBUG ("value_ptr = 0x%p; vl.values = 0x%p;", (void *) value_ptr, (void *) vl.values);
+
+       sfree (value_ptr);
+       sfree (vl.values); 
+
+       fprintf (fh, "0 Success\n");
+       fflush (fh);
+
+       return (0);
+} /* int us_handle_putval */
+
 static void *us_handle_client (void *arg)
 {
        int fd;
@@ -492,20 +643,31 @@ static void *us_handle_client (void *arg)
        free (arg);
        arg = NULL;
 
-       DBG ("Reading from fd #%i", fd);
+       DEBUG ("Reading from fd #%i", fd);
 
        fh = fdopen (fd, "r+");
        if (fh == NULL)
        {
-               syslog (LOG_ERR, "unixsock plugin: fdopen failed: %s",
-                               strerror (errno));
+               char errbuf[1024];
+               ERROR ("unixsock plugin: fdopen failed: %s",
+                               sstrerror (errno, errbuf, sizeof (errbuf)));
                close (fd);
                pthread_exit ((void *) 1);
        }
 
        while (fgets (buffer, sizeof (buffer), fh) != NULL)
        {
-               DBG ("fgets -> buffer = %s", buffer);
+               int len;
+
+               len = strlen (buffer);
+               while ((len > 0)
+                               && ((buffer[len - 1] == '\n') || (buffer[len - 1] == '\r')))
+                       buffer[--len] = '\0';
+
+               if (len == 0)
+                       continue;
+
+               DEBUG ("fgets -> buffer = %s; len = %i;", buffer, len);
 
                fields_num = strsplit (buffer, fields,
                                sizeof (fields) / sizeof (fields[0]));
@@ -520,14 +682,18 @@ static void *us_handle_client (void *arg)
                {
                        us_handle_getval (fh, fields, fields_num);
                }
+               else if (strcasecmp (fields[0], "putval") == 0)
+               {
+                       us_handle_putval (fh, fields, fields_num);
+               }
                else
                {
-                       fprintf (fh, "Unknown command: %s\n", fields[0]);
+                       fprintf (fh, "-1 Unknown command: %s\n", fields[0]);
                        fflush (fh);
                }
        } /* while (fgets) */
 
-       DBG ("Exiting..");
+       DEBUG ("Exiting..");
        close (fd);
 
        pthread_exit ((void *) 0);
@@ -538,21 +704,24 @@ static void *us_server_thread (void *arg)
        int  status;
        int *remote_fd;
        pthread_t th;
+       pthread_attr_t th_attr;
 
        if (us_open_socket () != 0)
                pthread_exit ((void *) 1);
 
-       while (42)
+       while (loop != 0)
        {
-               DBG ("Calling accept..");
+               DEBUG ("unixsock plugin: Calling accept..");
                status = accept (sock_fd, NULL, NULL);
                if (status < 0)
                {
+                       char errbuf[1024];
+
                        if (errno == EINTR)
                                continue;
 
-                       syslog (LOG_ERR, "unixsock plugin: accept failed: %s",
-                                       strerror (errno));
+                       ERROR ("unixsock plugin: accept failed: %s",
+                                       sstrerror (errno, errbuf, sizeof (errbuf)));
                        close (sock_fd);
                        sock_fd = -1;
                        pthread_exit ((void *) 1);
@@ -561,25 +730,42 @@ static void *us_server_thread (void *arg)
                remote_fd = (int *) malloc (sizeof (int));
                if (remote_fd == NULL)
                {
-                       syslog (LOG_WARNING, "unixsock plugin: malloc failed: %s",
-                                       strerror (errno));
+                       char errbuf[1024];
+                       WARNING ("unixsock plugin: malloc failed: %s",
+                                       sstrerror (errno, errbuf, sizeof (errbuf)));
                        close (status);
                        continue;
                }
                *remote_fd = status;
 
-               DBG ("Spawning child to handle connection on fd #%i", *remote_fd);
+               DEBUG ("Spawning child to handle connection on fd #%i", *remote_fd);
 
-               status = pthread_create (&th, NULL, us_handle_client, (void *) remote_fd);
+               pthread_attr_init (&th_attr);
+               pthread_attr_setdetachstate (&th_attr, PTHREAD_CREATE_DETACHED);
+
+               status = pthread_create (&th, &th_attr, us_handle_client, (void *) remote_fd);
                if (status != 0)
                {
-                       syslog (LOG_WARNING, "unixsock plugin: pthread_create failed: %s",
-                                       strerror (status));
+                       char errbuf[1024];
+                       WARNING ("unixsock plugin: pthread_create failed: %s",
+                                       sstrerror (errno, errbuf, sizeof (errbuf)));
                        close (*remote_fd);
                        free (remote_fd);
                        continue;
                }
-       } /* while (42) */
+       } /* while (loop) */
+
+       close (sock_fd);
+       sock_fd = -1;
+
+       status = unlink ((sock_file != NULL) ? sock_file : US_DEFAULT_PATH);
+       if (status != 0)
+       {
+               char errbuf[1024];
+               NOTICE ("unixsock plugin: unlink (%s) failed: %s",
+                               (sock_file != NULL) ? sock_file : US_DEFAULT_PATH,
+                               sstrerror (errno, errbuf, sizeof (errbuf)));
+       }
 
        return ((void *) 0);
 } /* void *us_server_thread */
@@ -612,11 +798,14 @@ static int us_init (void)
 {
        int status;
 
+       loop = 1;
+
        status = pthread_create (&listen_thread, NULL, us_server_thread, NULL);
        if (status != 0)
        {
-               syslog (LOG_ERR, "unixsock plugin: pthread_create failed: %s",
-                               strerror (status));
+               char errbuf[1024];
+               ERROR ("unixsock plugin: pthread_create failed: %s",
+                               sstrerror (errno, errbuf, sizeof (errbuf)));
                return (-1);
        }
 
@@ -627,19 +816,26 @@ static int us_shutdown (void)
 {
        void *ret;
 
-       if (listen_thread >= 0)
+       loop = 0;
+
+       if (listen_thread != (pthread_t) 0)
        {
                pthread_kill (listen_thread, SIGTERM);
                pthread_join (listen_thread, &ret);
+               listen_thread = (pthread_t) 0;
        }
 
+       plugin_unregister_init ("unixsock");
+       plugin_unregister_write ("unixsock");
+       plugin_unregister_shutdown ("unixsock");
+
        return (0);
 } /* int us_shutdown */
 
 static int us_write (const data_set_t *ds, const value_list_t *vl)
 {
        cache_update (ds, vl);
-       cache_flush (2 * atoi (COLLECTD_STEP));
+       cache_flush (2 * interval_g);
 
        return (0);
 }