Merge branch 'collectd-4.3' into collectd-4.4
authorFlorian Forster <sifnfors@informatik.stud.uni-erlangen.de>
Wed, 11 Jun 2008 12:30:56 +0000 (14:30 +0200)
committerFlorian Forster <sifnfors@informatik.stud.uni-erlangen.de>
Wed, 11 Jun 2008 12:30:56 +0000 (14:30 +0200)
12 files changed:
1  2 
src/apache.c
src/cpu.c
src/disk.c
src/email.c
src/iptables.c
src/network.c
src/nginx.c
src/ntpd.c
src/perl.c
src/rrdtool.c
src/users.c
src/utils_cmd_putval.c

diff --combined src/apache.c
  
  #include <curl/curl.h>
  
 -static char *url    = NULL;
 -static char *user   = NULL;
 -static char *pass   = NULL;
 -static char *cacert = NULL;
 +static char *url         = NULL;
 +static char *user        = NULL;
 +static char *pass        = NULL;
 +static char *verify_peer = NULL;
 +static char *verify_host = NULL;
 +static char *cacert      = NULL;
  
  static CURL *curl = NULL;
  
@@@ -48,8 -46,6 +48,8 @@@ static const char *config_keys[] 
        "URL",
        "User",
        "Password",
 +      "VerifyPeer",
 +      "VerifyHost",
        "CACert"
  };
  static int config_keys_num = STATIC_ARRAY_SIZE (config_keys);
@@@ -106,10 -102,6 +106,10 @@@ static int config (const char *key, con
                return (config_set (&user, value));
        else if (strcasecmp (key, "password") == 0)
                return (config_set (&pass, value));
 +      else if (strcasecmp (key, "verifypeer") == 0)
 +              return (config_set (&verify_peer, value));
 +      else if (strcasecmp (key, "verifyhost") == 0)
 +              return (config_set (&verify_host, value));
        else if (strcasecmp (key, "cacert") == 0)
                return (config_set (&cacert, value));
        else
@@@ -162,24 -154,6 +162,24 @@@ static int init (void
  
        curl_easy_setopt (curl, CURLOPT_URL, url);
  
 +      if ((verify_peer == NULL) || (strcmp (verify_peer, "true") == 0))
 +      {
 +              curl_easy_setopt (curl, CURLOPT_SSL_VERIFYPEER, 1);
 +      }
 +      else
 +      {
 +              curl_easy_setopt (curl, CURLOPT_SSL_VERIFYPEER, 0);
 +      }
 +
 +      if ((verify_host == NULL) || (strcmp (verify_host, "true") == 0))
 +      {
 +              curl_easy_setopt (curl, CURLOPT_SSL_VERIFYHOST, 2);
 +      }
 +      else
 +      {
 +              curl_easy_setopt (curl, CURLOPT_SSL_VERIFYHOST, 0);
 +      }
 +
        if (cacert != NULL)
        {
                curl_easy_setopt (curl, CURLOPT_CAINFO, cacert);
@@@ -199,9 -173,9 +199,9 @@@ static void submit_counter (const char 
        vl.values = values;
        vl.values_len = 1;
        vl.time = time (NULL);
-       strcpy (vl.host, hostname_g);
-       strcpy (vl.plugin, "apache");
-       strcpy (vl.plugin_instance, "");
+       sstrncpy (vl.host, hostname_g, sizeof (vl.host));
+       sstrncpy (vl.plugin, "apache", sizeof (vl.plugin));
+       sstrncpy (vl.plugin_instance, "", sizeof (vl.plugin_instance));
  
        if (type_instance != NULL)
        {
@@@ -224,9 -198,9 +224,9 @@@ static void submit_gauge (const char *t
        vl.values = values;
        vl.values_len = 1;
        vl.time = time (NULL);
-       strcpy (vl.host, hostname_g);
-       strcpy (vl.plugin, "apache");
-       strcpy (vl.plugin_instance, "");
+       sstrncpy (vl.host, hostname_g, sizeof (vl.host));
+       sstrncpy (vl.plugin, "apache", sizeof (vl.plugin));
+       sstrncpy (vl.plugin_instance, "", sizeof (vl.plugin_instance));
  
        if (type_instance != NULL)
        {
diff --combined src/cpu.c
+++ b/src/cpu.c
  # endif
  #endif /* HAVE_SYSCTLBYNAME */
  
 -#if !PROCESSOR_CPU_LOAD_INFO && !KERNEL_LINUX && !HAVE_LIBKSTAT && !HAVE_SYSCTLBYNAME
 +#if HAVE_STATGRAB_H
 +# include <statgrab.h>
 +#endif
 +
 +#if !PROCESSOR_CPU_LOAD_INFO && !KERNEL_LINUX && !HAVE_LIBKSTAT \
 +      && !HAVE_SYSCTLBYNAME && !HAVE_LIBSTATGRAB
  # error "No applicable input method."
  #endif
  
@@@ -103,11 -98,7 +103,11 @@@ static int numcpu
  
  #elif defined(HAVE_SYSCTLBYNAME)
  static int numcpu;
 -#endif /* HAVE_SYSCTLBYNAME */
 +/* #endif HAVE_SYSCTLBYNAME */
 +
 +#elif defined(HAVE_LIBSTATGRAB)
 +/* no variables needed */
 +#endif /* HAVE_LIBSTATGRAB */
  
  static int init (void)
  {
  
        if (numcpu != 1)
                NOTICE ("cpu: Only one processor supported when using `sysctlbyname' (found %i)", numcpu);
 -#endif
 +/* #endif HAVE_SYSCTLBYNAME */
 +
 +#elif defined(HAVE_LIBSTATGRAB)
 +      /* nothing to initialize */
 +#endif /* HAVE_LIBSTATGRAB */
  
        return (0);
  } /* int init */
@@@ -180,12 -167,12 +180,12 @@@ static void submit (int cpu_num, const 
        vl.values = values;
        vl.values_len = 1;
        vl.time = time (NULL);
-       strcpy (vl.host, hostname_g);
-       strcpy (vl.plugin, "cpu");
+       sstrncpy (vl.host, hostname_g, sizeof (vl.host));
+       sstrncpy (vl.plugin, "cpu", sizeof (vl.plugin));
        snprintf (vl.plugin_instance, sizeof (vl.type_instance),
                        "%i", cpu_num);
        vl.plugin_instance[DATA_MAX_NAME_LEN - 1] = '\0';
-       strcpy (vl.type_instance, type_instance);
+       sstrncpy (vl.type_instance, type_instance, sizeof (vl.type_instance));
  
        plugin_dispatch_values ("cpu", &vl);
  }
@@@ -383,25 -370,7 +383,25 @@@ static int cpu_read (void
        submit (0, "nice", cpuinfo[CP_NICE]);
        submit (0, "system", cpuinfo[CP_SYS]);
        submit (0, "idle", cpuinfo[CP_IDLE]);
 -#endif
 +/* #endif HAVE_SYSCTLBYNAME */
 +
 +#elif defined(HAVE_LIBSTATGRAB)
 +       sg_cpu_stats *cs;
 +       cs = sg_get_cpu_stats ();
 +
 +       if (cs == NULL)
 +       {
 +             ERROR ("cpu plugin: sg_get_cpu_stats failed.");
 +               return (-1);
 +       }
 +
 +       submit (0, "idle",   (counter_t) cs->idle);
 +       submit (0, "nice",   (counter_t) cs->nice);
 +       submit (0, "swap",   (counter_t) cs->swap);
 +       submit (0, "system", (counter_t) cs->kernel);
 +       submit (0, "user",   (counter_t) cs->user);
 +       submit (0, "wait",   (counter_t) cs->iowait);
 +#endif /* HAVE_LIBSTATGRAB */
  
        return (0);
  }
diff --combined src/disk.c
@@@ -1,6 -1,6 +1,6 @@@
  /**
   * collectd - src/disk.c
 - * Copyright (C) 2005-2007  Florian octo Forster
 + * Copyright (C) 2005-2008  Florian octo Forster
   *
   * This program is free software; you can redistribute it and/or modify it
   * under the terms of the GNU General Public License as published by the
@@@ -22,7 -22,6 +22,7 @@@
  #include "collectd.h"
  #include "common.h"
  #include "plugin.h"
 +#include "utils_ignorelist.h"
  
  #if HAVE_MACH_MACH_TYPES_H
  #  include <mach/mach_types.h>
  #  define UINT_MAX 4294967295U
  #endif
  
 +#if HAVE_STATGRAB_H
 +# include <statgrab.h>
 +#endif
 +
  #if HAVE_IOKIT_IOKITLIB_H
  static mach_port_t io_master_port = MACH_PORT_NULL;
  /* #endif HAVE_IOKIT_IOKITLIB_H */
@@@ -102,50 -97,10 +102,50 @@@ static kstat_t *ksp[MAX_NUMDISK]
  static int numdisk = 0;
  /* #endif HAVE_LIBKSTAT */
  
 +#elif defined(HAVE_LIBSTATGRAB)
 +/* #endif HAVE_LIBKSTATGRAB */
 +
  #else
  # error "No applicable input method."
  #endif
  
 +static const char *config_keys[] =
 +{
 +      "Disk",
 +      "IgnoreSelected"
 +};
 +static int config_keys_num = STATIC_ARRAY_SIZE (config_keys);
 +
 +static ignorelist_t *ignorelist = NULL;
 +
 +static int disk_config (const char *key, const char *value)
 +{
 +  if (ignorelist == NULL)
 +    ignorelist = ignorelist_create (/* invert = */ 1);
 +  if (ignorelist == NULL)
 +    return (1);
 +
 +  if (strcasecmp ("Disk", key) == 0)
 +  {
 +    ignorelist_add (ignorelist, value);
 +  }
 +  else if (strcasecmp ("IgnoreSelected", key) == 0)
 +  {
 +    int invert = 1;
 +    if ((strcasecmp ("True", value) == 0)
 +      || (strcasecmp ("Yes", value) == 0)
 +      || (strcasecmp ("On", value) == 0))
 +      invert = 0;
 +    ignorelist_set_invert (ignorelist, invert);
 +  }
 +  else
 +  {
 +    return (-1);
 +  }
 +
 +  return (0);
 +} /* int disk_config */
 +
  static int disk_init (void)
  {
  #if HAVE_IOKIT_IOKITLIB_H
@@@ -203,18 -158,14 +203,18 @@@ static void disk_submit (const char *pl
        value_t values[2];
        value_list_t vl = VALUE_LIST_INIT;
  
 +      /* Both `ignorelist' and `plugin_instance' may be NULL. */
 +      if (ignorelist_match (ignorelist, plugin_instance) != 0)
 +        return;
 +
        values[0].counter = read;
        values[1].counter = write;
  
        vl.values = values;
        vl.values_len = 2;
        vl.time = time (NULL);
-       strcpy (vl.host, hostname_g);
-       strcpy (vl.plugin, "disk");
+       sstrncpy (vl.host, hostname_g, sizeof (vl.host));
+       sstrncpy (vl.plugin, "disk", sizeof (vl.plugin));
        strncpy (vl.plugin_instance, plugin_instance,
                        sizeof (vl.plugin_instance));
  
@@@ -666,31 -617,13 +666,31 @@@ static int disk_read (void
                                        kio.KIO_ROPS, kio.KIO_WOPS);
                }
        }
 -#endif /* defined(HAVE_LIBKSTAT) */
 +/* #endif defined(HAVE_LIBKSTAT) */
 +
 +#elif defined(HAVE_LIBSTATGRAB)
 +      sg_disk_io_stats *ds;
 +      int disks, counter;
 +      char name[DATA_MAX_NAME_LEN];
 +      
 +      if ((ds = sg_get_disk_io_stats(&disks)) == NULL)
 +              return (0);
 +              
 +      for (counter=0; counter < disks; counter++) {
 +              strncpy(name, ds->disk_name, sizeof(name));
 +              name[sizeof(name)-1] = '\0'; /* strncpy doesn't terminate longer strings */
 +              disk_submit (name, "disk_octets", ds->read_bytes, ds->write_bytes);
 +              ds++;
 +      }
 +#endif /* defined(HAVE_LIBSTATGRAB) */
  
        return (0);
  } /* int disk_read */
  
  void module_register (void)
  {
 -      plugin_register_init ("disk", disk_init);
 -      plugin_register_read ("disk", disk_read);
 +  plugin_register_config ("disk", disk_config,
 +      config_keys, config_keys_num);
 +  plugin_register_init ("disk", disk_init);
 +  plugin_register_read ("disk", disk_read);
  } /* void module_register */
diff --combined src/email.c
@@@ -1,6 -1,6 +1,6 @@@
  /**
   * collectd - src/email.c
 - * Copyright (C) 2006,2007  Sebastian Harl
 + * Copyright (C) 2006-2008  Sebastian Harl
   *
   * This program is free software; you can redistribute it and/or modify it
   * under the terms of the GNU General Public License as published by the
  #     include <grp.h>
  #endif /* HAVE_GRP_H */
  
 -#define MODULE_NAME "email"
 -
 -/* 256 bytes ought to be enough for anybody ;-) */
 -#define BUFSIZE 256
 -
  #define SOCK_PATH LOCALSTATEDIR"/run/"PACKAGE_NAME"-email"
  #define MAX_CONNS 5
  #define MAX_CONNS_LIMIT 16384
  
 -#define log_err(...) ERROR (MODULE_NAME": "__VA_ARGS__)
 -#define log_warn(...) WARNING (MODULE_NAME": "__VA_ARGS__)
 +#define log_debug(...) DEBUG ("email: "__VA_ARGS__)
 +#define log_err(...) ERROR ("email: "__VA_ARGS__)
 +#define log_warn(...) WARNING ("email: "__VA_ARGS__)
  
  /*
   * Private data structures
@@@ -86,15 -90,19 +86,15 @@@ typedef struct collector 
        pthread_t thread;
  
        /* socket descriptor of the current/last connection */
 -      int socket;
 +      FILE *socket;
  } collector_t;
  
  /* linked list of pending connections */
  typedef struct conn {
        /* socket to read data from */
 -      int socket;
 -
 -      /* buffer to read data to */
 -      char *buffer;
 -      int  idx; /* current write position in buffer */
 -      int  length; /* length of the current line, i.e. index of '\0' */
 +      FILE *socket;
  
 +      /* linked list of connections */
        struct conn *next;
  } conn_t;
  
@@@ -117,8 -125,8 +117,8 @@@ static const char *config_keys[] 
  static int config_keys_num = STATIC_ARRAY_SIZE (config_keys);
  
  /* socket configuration */
 -static char *sock_file  = SOCK_PATH;
 -static char *sock_group = COLLECTD_GRP_NAME;
 +static char *sock_file  = NULL;
 +static char *sock_group = NULL;
  static int  sock_perms  = S_IRWXU | S_IRWXG;
  static int  max_conns   = MAX_CONNS;
  
@@@ -146,20 -154,17 +146,20 @@@ static pthread_mutex_t available_mutex 
  static int available_collectors;
  
  static pthread_mutex_t count_mutex = PTHREAD_MUTEX_INITIALIZER;
 -static type_list_t count;
 +static type_list_t list_count;
 +static type_list_t list_count_copy;
  
  static pthread_mutex_t size_mutex = PTHREAD_MUTEX_INITIALIZER;
 -static type_list_t size;
 +static type_list_t list_size;
 +static type_list_t list_size_copy;
  
  static pthread_mutex_t score_mutex = PTHREAD_MUTEX_INITIALIZER;
  static double score;
  static int score_count;
  
  static pthread_mutex_t check_mutex = PTHREAD_MUTEX_INITIALIZER;
 -static type_list_t check;
 +static type_list_t list_check;
 +static type_list_t list_check_copy;
  
  /*
   * Private functions
  static int email_config (const char *key, const char *value)
  {
        if (0 == strcasecmp (key, "SocketFile")) {
 +              if (NULL != sock_file)
 +                      free (sock_file);
                sock_file = sstrdup (value);
        }
        else if (0 == strcasecmp (key, "SocketGroup")) {
 +              if (NULL != sock_group)
 +                      free (sock_group);
                sock_group = sstrdup (value);
        }
        else if (0 == strcasecmp (key, "SocketPerms")) {
@@@ -240,10 -241,139 +240,10 @@@ static void type_list_incr (type_list_
        return;
  } /* static void type_list_incr (type_list_t *, char *) */
  
 -/* Read a single character from the socket. If an error occurs or end-of-file
 - * is reached return '\0'. */
 -static char read_char (conn_t *src)
 -{
 -      char ret = '\0';
 -
 -      fd_set fdset;
 -
 -      FD_ZERO (&fdset);
 -      FD_SET (src->socket, &fdset);
 -
 -      if (-1 == select (src->socket + 1, &fdset, NULL, NULL, NULL)) {
 -              char errbuf[1024];
 -              log_err ("select() failed: %s",
 -                              sstrerror (errno, errbuf, sizeof (errbuf)));
 -              return '\0';
 -      }
 -
 -      assert (FD_ISSET (src->socket, &fdset));
 -
 -      do {
 -              ssize_t len = 0;
 -
 -              errno = 0;
 -              if (0 > (len = read (src->socket, (void *)&ret, 1))) {
 -                      if (EINTR != errno) {
 -                              char errbuf[1024];
 -                              log_err ("read() failed: %s",
 -                                              sstrerror (errno, errbuf, sizeof (errbuf)));
 -                              return '\0';
 -                      }
 -              }
 -
 -              if (0 == len)
 -                      return '\0';
 -      } while (EINTR == errno);
 -      return ret;
 -} /* static char read_char (conn_t *) */
 -
 -/* Read a single line (terminated by '\n') from the the socket.
 - *
 - * The return value is zero terminated and does not contain any newline
 - * characters.
 - *
 - * If an error occurs or end-of-file is reached return NULL.
 - *
 - * IMPORTANT NOTE: If there is no newline character found in BUFSIZE
 - * characters of the input stream, the line will will be ignored! By
 - * definition we should not get any longer input lines, thus this is
 - * acceptable in this case ;-) */
 -static char *read_line (conn_t *src)
 -{
 -      int i = 0;
 -
 -      assert ((BUFSIZE >= src->idx) && (src->idx >= 0));
 -      assert ((src->idx > src->length) || (src->length == 0));
 -
 -      if (src->length > 0) { /* remove old line */
 -              src->idx -= (src->length + 1);
 -              memmove (src->buffer, src->buffer + src->length + 1, src->idx);
 -              src->length = 0;
 -      }
 -
 -      for (i = 0; i < src->idx; ++i) {
 -              if ('\n' == src->buffer[i])
 -                      break;
 -      }
 -
 -      if (i == src->idx) {
 -              fd_set fdset;
 -
 -              ssize_t len = 0;
 -
 -              FD_ZERO (&fdset);
 -              FD_SET (src->socket, &fdset);
 -
 -              if (-1 == select (src->socket + 1, &fdset, NULL, NULL, NULL)) {
 -                      char errbuf[1024];
 -                      log_err ("select() failed: %s",
 -                                      sstrerror (errno, errbuf, sizeof (errbuf)));
 -                      return NULL;
 -              }
 -
 -              assert (FD_ISSET (src->socket, &fdset));
 -
 -              do {
 -                      errno = 0;
 -                      if (0 > (len = read (src->socket,
 -                                                      (void *)(src->buffer + src->idx),
 -                                                      BUFSIZE - src->idx))) {
 -                              if (EINTR != errno) {
 -                                      char errbuf[1024];
 -                                      log_err ("read() failed: %s",
 -                                                      sstrerror (errno, errbuf, sizeof (errbuf)));
 -                                      return NULL;
 -                              }
 -                      }
 -
 -                      if (0 == len)
 -                              return NULL;
 -              } while (EINTR == errno);
 -
 -              src->idx += len;
 -
 -              for (i = src->idx - len; i < src->idx; ++i) {
 -                      if ('\n' == src->buffer[i])
 -                              break;
 -              }
 -
 -              if (i == src->idx) {
 -                      src->length = 0;
 -
 -                      if (BUFSIZE == src->idx) { /* no space left in buffer */
 -                              while ('\n' != read_char (src))
 -                                      /* ignore complete line */;
 -
 -                              src->idx = 0;
 -                      }
 -                      return read_line (src);
 -              }
 -      }
 -
 -      src->buffer[i] = '\0';
 -      src->length    = i;
 -
 -      return src->buffer;
 -} /* static char *read_line (conn_t *) */
 -
  static void *collect (void *arg)
  {
        collector_t *this = (collector_t *)arg;
 -
 -      char *buffer = (char *)smalloc (BUFSIZE);
 +      pthread_t    self = pthread_self ();
  
        while (1) {
                int loop = 1;
                        conns.tail = NULL;
                }
  
 -              this->socket = connection->socket;
 -
                pthread_mutex_unlock (&conns_mutex);
  
 -              connection->buffer = buffer;
 -              connection->idx    = 0;
 -              connection->length = 0;
 +              /* make the socket available to the global
 +               * thread and connection management */
 +              this->socket = connection->socket;
  
 -              { /* put the socket in non-blocking mode */
 -                      int flags = 0;
 +              log_debug ("[thread #%5lu] handling connection on fd #%i",
 +                              self, fileno (this->socket));
  
 -                      errno = 0;
 -                      if (-1 == fcntl (connection->socket, F_GETFL, &flags)) {
 -                              char errbuf[1024];
 -                              log_err ("fcntl() failed: %s",
 -                                              sstrerror (errno, errbuf, sizeof (errbuf)));
 -                              loop = 0;
 -                      }
 +              while (loop) {
 +                      /* 256 bytes ought to be enough for anybody ;-) */
 +                      char line[256 + 1]; /* line + '\0' */
 +                      int  len = 0;
  
                        errno = 0;
 -                      if (-1 == fcntl (connection->socket, F_SETFL, flags | O_NONBLOCK)) {
 -                              char errbuf[1024];
 -                              log_err ("fcntl() failed: %s",
 -                                              sstrerror (errno, errbuf, sizeof (errbuf)));
 +                      if (NULL == fgets (line, sizeof (line), this->socket)) {
                                loop = 0;
 +
 +                              if (0 != errno) {
 +                                      char errbuf[1024];
 +                                      log_err ("[thread #%5lu] reading from socket (fd #%i) "
 +                                                      "failed: %s", self, fileno (this->socket),
 +                                                      sstrerror (errno, errbuf, sizeof (errbuf)));
 +                              }
 +                              break;
                        }
 -              }
  
 -              while (loop) {
 -                      char *line = read_line (connection);
 +                      len = strlen (line);
 +                      if (('\n' != line[len - 1]) && ('\r' != line[len - 1])) {
 +                              log_warn ("[thread #%5lu] line too long (> %lu characters): "
 +                                              "'%s' (truncated)", self, sizeof (line) - 1, line);
  
 -                      if (NULL == line) {
 -                              loop = 0;
 -                              break;
 +                              while (NULL != fgets (line, sizeof (line), this->socket))
 +                                      if (('\n' == line[len - 1]) || ('\r' == line[len - 1]))
 +                                              break;
 +                              continue;
                        }
  
 +                      line[len - 1] = '\0';
 +
 +                      log_debug ("[thread #%5lu] line = '%s'", self, line);
 +
                        if (':' != line[1]) {
 -                              log_err ("syntax error in line '%s'", line);
 +                              log_err ("[thread #%5lu] syntax error in line '%s'",
 +                                              self, line);
                                continue;
                        }
  
                                int  bytes = 0;
  
                                if (NULL == tmp) {
 -                                      log_err ("syntax error in line '%s'", line);
 +                                      log_err ("[thread #%5lu] syntax error in line '%s'",
 +                                                      self, line);
                                        continue;
                                }
  
                                bytes = atoi (tmp);
  
                                pthread_mutex_lock (&count_mutex);
 -                              type_list_incr (&count, type, 1);
 +                              type_list_incr (&list_count, type, 1);
                                pthread_mutex_unlock (&count_mutex);
  
                                if (bytes > 0) {
                                        pthread_mutex_lock (&size_mutex);
 -                                      type_list_incr (&size, type, bytes);
 +                                      type_list_incr (&list_size, type, bytes);
                                        pthread_mutex_unlock (&size_mutex);
                                }
                        }
  
                                do {
                                        pthread_mutex_lock (&check_mutex);
 -                                      type_list_incr (&check, type, 1);
 +                                      type_list_incr (&list_check, type, 1);
                                        pthread_mutex_unlock (&check_mutex);
                                } while (NULL != (type = strtok_r (NULL, ",", &ptr)));
                        }
                        else {
 -                              log_err ("unknown type '%c'", line[0]);
 +                              log_err ("[thread #%5lu] unknown type '%c'", self, line[0]);
                        }
                } /* while (loop) */
  
 -              close (connection->socket);
 +              log_debug ("[thread #%5lu] shutting down connection on fd #%i",
 +                              pthread_self (), fileno (this->socket));
 +
 +              fclose (connection->socket);
                free (connection);
  
 -              this->socket = -1;
 +              this->socket = NULL;
  
                pthread_mutex_lock (&available_mutex);
                ++available_collectors;
                pthread_cond_signal (&collector_available);
        } /* while (1) */
  
 -      free (buffer);
        pthread_exit ((void *)0);
  } /* static void *collect (void *) */
  
@@@ -379,9 -499,6 +379,9 @@@ static void *open_connection (void *arg
  {
        struct sockaddr_un addr;
  
 +      char *path  = (NULL == sock_file) ? SOCK_PATH : sock_file;
 +      char *group = (NULL == sock_group) ? COLLECTD_GRP_NAME : sock_group;
 +
        /* create UNIX socket */
        errno = 0;
        if (-1 == (connector_socket = socket (PF_UNIX, SOCK_STREAM, 0))) {
  
        addr.sun_family = AF_UNIX;
  
 -      strncpy (addr.sun_path, sock_file, (size_t)(UNIX_PATH_MAX - 1));
 +      strncpy (addr.sun_path, path, (size_t)(UNIX_PATH_MAX - 1));
        addr.sun_path[UNIX_PATH_MAX - 1] = '\0';
        unlink (addr.sun_path);
  
                                        + strlen(addr.sun_path))) {
                char errbuf[1024];
                disabled = 1;
 -              connector_socket = -1; /* TODO: close? */
 +              close (connector_socket);
 +              connector_socket = -1;
                log_err ("bind() failed: %s",
                                sstrerror (errno, errbuf, sizeof (errbuf)));
                pthread_exit ((void *)1);
        if (-1 == listen (connector_socket, 5)) {
                char errbuf[1024];
                disabled = 1;
 -              connector_socket = -1; /* TODO: close? */
 +              close (connector_socket);
 +              connector_socket = -1;
                log_err ("listen() failed: %s",
                                sstrerror (errno, errbuf, sizeof (errbuf)));
                pthread_exit ((void *)1);
        }
  
 -      if ((uid_t) 0 == geteuid ())
        {
                struct group sg;
                struct group *grp;
                int status;
  
                grp = NULL;
 -              status = getgrnam_r (sock_group, &sg, grbuf, sizeof (grbuf), &grp);
 +              status = getgrnam_r (group, &sg, grbuf, sizeof (grbuf), &grp);
                if (status != 0)
                {
                        char errbuf[1024];
 -                      log_warn ("getgrnam_r (%s) failed: %s", sock_group,
 +                      log_warn ("getgrnam_r (%s) failed: %s", group,
                                        sstrerror (errno, errbuf, sizeof (errbuf)));
                }
                else if (grp == NULL)
                {
 -                      log_warn ("No such group: `%s'", sock_group);
 +                      log_warn ("No such group: `%s'", group);
                }
                else
                {
 -                      status = chown (sock_file, (uid_t) -1, grp->gr_gid);
 +                      status = chown (path, (uid_t) -1, grp->gr_gid);
                        if (status != 0)
                        {
                                char errbuf[1024];
                                log_warn ("chown (%s, -1, %i) failed: %s",
 -                                              sock_file, (int) grp->gr_gid,
 +                                              path, (int) grp->gr_gid,
                                                sstrerror (errno, errbuf, sizeof (errbuf)));
                        }
                }
        }
 -      else /* geteuid != 0 */
 -      {
 -              log_warn ("not running as root");
 -      }
  
        errno = 0;
 -      if (0 != chmod (sock_file, sock_perms)) {
 +      if (0 != chmod (path, sock_perms)) {
                char errbuf[1024];
                log_warn ("chmod() failed: %s",
                                sstrerror (errno, errbuf, sizeof (errbuf)));
  
                for (i = 0; i < max_conns; ++i) {
                        collectors[i] = (collector_t *)smalloc (sizeof (collector_t));
 -                      collectors[i]->socket = -1;
 +                      collectors[i]->socket = NULL;
  
                        if (0 != (err = pthread_create (&collectors[i]->thread, &ptattr,
                                                        collect, collectors[i]))) {
                                if (EINTR != errno) {
                                        char errbuf[1024];
                                        disabled = 1;
 -                                      connector_socket = -1; /* TODO: close? */
 +                                      close (connector_socket);
 +                                      connector_socket = -1;
                                        log_err ("accept() failed: %s",
                                                        sstrerror (errno, errbuf, sizeof (errbuf)));
                                        pthread_exit ((void *)1);
  
                connection = (conn_t *)smalloc (sizeof (conn_t));
  
 -              connection->socket = remote;
 +              connection->socket = fdopen (remote, "r");
                connection->next   = NULL;
  
 +              if (NULL == connection->socket) {
 +                      close (remote);
 +                      continue;
 +              }
 +
                pthread_mutex_lock (&conns_mutex);
  
                if (NULL == conns.head) {
@@@ -569,8 -683,6 +569,8 @@@ static int email_init (void
  
  static int email_shutdown (void)
  {
 +      type_t *ptr = NULL;
 +
        int i = 0;
  
        if (connector != ((pthread_t) 0)) {
        /* don't allow any more connections to be processed */
        pthread_mutex_lock (&conns_mutex);
  
 +      available_collectors = 0;
 +
        if (collectors != NULL) {
                for (i = 0; i < max_conns; ++i) {
                        if (collectors[i] == NULL)
                                collectors[i]->thread = (pthread_t) 0;
                        }
  
 -                      if (collectors[i]->socket >= 0) {
 -                              close (collectors[i]->socket);
 -                              collectors[i]->socket = -1;
 +                      if (collectors[i]->socket != NULL) {
 +                              fclose (collectors[i]->socket);
 +                              collectors[i]->socket = NULL;
                        }
 +
 +                      sfree (collectors[i]);
                }
 +              sfree (collectors);
        } /* if (collectors != NULL) */
  
        pthread_mutex_unlock (&conns_mutex);
  
 -      unlink (sock_file);
 -      errno = 0;
 +      for (ptr = list_count.head; NULL != ptr; ptr = ptr->next) {
 +              free (ptr->name);
 +              free (ptr);
 +      }
 +
 +      for (ptr = list_count_copy.head; NULL != ptr; ptr = ptr->next) {
 +              free (ptr->name);
 +              free (ptr);
 +      }
 +
 +      for (ptr = list_size.head; NULL != ptr; ptr = ptr->next) {
 +              free (ptr->name);
 +              free (ptr);
 +      }
  
 +      for (ptr = list_size_copy.head; NULL != ptr; ptr = ptr->next) {
 +              free (ptr->name);
 +              free (ptr);
 +      }
 +
 +      for (ptr = list_check.head; NULL != ptr; ptr = ptr->next) {
 +              free (ptr->name);
 +              free (ptr);
 +      }
 +
 +      for (ptr = list_check_copy.head; NULL != ptr; ptr = ptr->next) {
 +              free (ptr->name);
 +              free (ptr);
 +      }
 +
 +      unlink ((NULL == sock_file) ? SOCK_PATH : sock_file);
 +
 +      sfree (sock_file);
 +      sfree (sock_group);
        return (0);
  } /* static void email_shutdown (void) */
  
@@@ -657,8 -733,8 +657,8 @@@ static void email_submit (const char *t
        vl.values = values;
        vl.values_len = 1;
        vl.time = time (NULL);
-       strcpy (vl.host, hostname_g);
-       strcpy (vl.plugin, "email");
+       sstrncpy (vl.host, hostname_g, sizeof (vl.host));
+       sstrncpy (vl.plugin, "email", sizeof (vl.plugin));
        strncpy (vl.type_instance, type_instance, sizeof (vl.type_instance));
  
        plugin_dispatch_values (type, &vl);
@@@ -709,28 -785,47 +709,28 @@@ static int email_read (void
        double score_old;
        int score_count_old;
  
 -      static type_list_t *cnt;
 -      static type_list_t *sz;
 -      static type_list_t *chk;
 -
        if (disabled)
                return (-1);
  
 -      if (NULL == cnt) {
 -              cnt = (type_list_t *)smalloc (sizeof (type_list_t));
 -              cnt->head = NULL;
 -      }
 -
 -      if (NULL == sz) {
 -              sz = (type_list_t *)smalloc (sizeof (type_list_t));
 -              sz->head = NULL;
 -      }
 -
 -      if (NULL == chk) {
 -              chk = (type_list_t *)smalloc (sizeof (type_list_t));
 -              chk->head = NULL;
 -      }
 -
        /* email count */
        pthread_mutex_lock (&count_mutex);
  
 -      copy_type_list (&count, cnt);
 +      copy_type_list (&list_count, &list_count_copy);
  
        pthread_mutex_unlock (&count_mutex);
  
 -      for (ptr = cnt->head; NULL != ptr; ptr = ptr->next) {
 +      for (ptr = list_count_copy.head; NULL != ptr; ptr = ptr->next) {
                email_submit ("email_count", ptr->name, ptr->value);
        }
  
        /* email size */
        pthread_mutex_lock (&size_mutex);
  
 -      copy_type_list (&size, sz);
 +      copy_type_list (&list_size, &list_size_copy);
  
        pthread_mutex_unlock (&size_mutex);
  
 -      for (ptr = sz->head; NULL != ptr; ptr = ptr->next) {
 +      for (ptr = list_size_copy.head; NULL != ptr; ptr = ptr->next) {
                email_submit ("email_size", ptr->name, ptr->value);
        }
  
        /* spam checks */
        pthread_mutex_lock (&check_mutex);
  
 -      copy_type_list (&check, chk);
 +      copy_type_list (&list_check, &list_check_copy);
  
        pthread_mutex_unlock (&check_mutex);
  
 -      for (ptr = chk->head; NULL != ptr; ptr = ptr->next)
 +      for (ptr = list_check_copy.head; NULL != ptr; ptr = ptr->next)
                email_submit ("spam_check", ptr->name, ptr->value);
  
        return (0);
diff --combined src/iptables.c
@@@ -25,9 -25,7 +25,9 @@@
  #include "plugin.h"
  #include "configfile.h"
  
 -#if HAVE_LIBIPTC_LIBIPTC_H
 +#if OWN_LIBIPTC
 +# include "libiptc/libiptc.h"
 +#else
  # include <libiptc/libiptc.h>
  #endif
  
@@@ -221,8 -219,8 +221,8 @@@ static int submit_match (const struct i
      vl.values = values;
      vl.values_len = 1;
      vl.time = time (NULL);
-     strcpy (vl.host, hostname_g);
-     strcpy (vl.plugin, "iptables");
+     sstrncpy (vl.host, hostname_g, sizeof (vl.host));
+     sstrncpy (vl.plugin, "iptables", sizeof (vl.plugin));
  
      status = snprintf (vl.plugin_instance, sizeof (vl.plugin_instance),
            "%s-%s", chain->table, chain->chain);
diff --combined src/network.c
@@@ -1405,7 -1405,7 +1405,7 @@@ static int add_to_buffer (char *buffer
                if (write_part_string (&buffer, &buffer_size, TYPE_HOST,
                                        vl->host, strlen (vl->host)) != 0)
                        return (-1);
-               strcpy (vl_def->host, vl->host);
+               sstrncpy (vl_def->host, vl->host, sizeof (vl_def->host));
        }
  
        if (vl_def->time != vl->time)
                if (write_part_string (&buffer, &buffer_size, TYPE_PLUGIN,
                                        vl->plugin, strlen (vl->plugin)) != 0)
                        return (-1);
-               strcpy (vl_def->plugin, vl->plugin);
+               sstrncpy (vl_def->plugin, vl->plugin, sizeof (vl_def->plugin));
        }
  
        if (strcmp (vl_def->plugin_instance, vl->plugin_instance) != 0)
                                        vl->plugin_instance,
                                        strlen (vl->plugin_instance)) != 0)
                        return (-1);
-               strcpy (vl_def->plugin_instance, vl->plugin_instance);
+               sstrncpy (vl_def->plugin_instance, vl->plugin_instance, sizeof (vl_def->plugin_instance));
        }
  
        if (strcmp (type_def, ds->type) != 0)
                if (write_part_string (&buffer, &buffer_size, TYPE_TYPE,
                                        ds->type, strlen (ds->type)) != 0)
                        return (-1);
-               strcpy (type_def, ds->type);
+               sstrncpy (type_def, ds->type, sizeof (type_def));
        }
  
        if (strcmp (vl_def->type_instance, vl->type_instance) != 0)
                                        vl->type_instance,
                                        strlen (vl->type_instance)) != 0)
                        return (-1);
-               strcpy (vl_def->type_instance, vl->type_instance);
+               sstrncpy (vl_def->type_instance, vl->type_instance, sizeof (vl_def->type_instance));
        }
        
        if (write_part_values (&buffer, &buffer_size, ds, vl) != 0)
@@@ -1767,25 -1767,9 +1767,25 @@@ static int network_init (void
        return (0);
  } /* int network_init */
  
 +static int network_flush (int timeout)
 +{
 +      pthread_mutex_lock (&send_buffer_lock);
 +
 +      if (((time (NULL) - cache_flush_last) >= timeout)
 +                      && (send_buffer_fill > 0))
 +      {
 +              flush_buffer ();
 +      }
 +
 +      pthread_mutex_unlock (&send_buffer_lock);
 +
 +      return (0);
 +} /* int network_flush */
 +
  void module_register (void)
  {
        plugin_register_config ("network", network_config,
                        config_keys, config_keys_num);
        plugin_register_init   ("network", network_init);
 +      plugin_register_flush   ("network", network_flush);
  } /* void module_register */
diff --combined src/nginx.c
  
  #include <curl/curl.h>
  
 -static char *url    = NULL;
 -static char *user   = NULL;
 -static char *pass   = NULL;
 -static char *cacert = NULL;
 +static char *url         = NULL;
 +static char *user        = NULL;
 +static char *pass        = NULL;
 +static char *verify_peer = NULL;
 +static char *verify_host = NULL;
 +static char *cacert      = NULL;
  
  static CURL *curl = NULL;
  
@@@ -46,8 -44,6 +46,8 @@@ static const char *config_keys[] 
    "URL",
    "User",
    "Password",
 +  "VerifyPeer",
 +  "VerifyHost",
    "CACert"
  };
  static int config_keys_num = STATIC_ARRAY_SIZE (config_keys);
@@@ -93,10 -89,6 +93,10 @@@ static int config (const char *key, con
      return (config_set (&user, value));
    else if (strcasecmp (key, "password") == 0)
      return (config_set (&pass, value));
 +  else if (strcasecmp (key, "verifypeer") == 0)
 +    return (config_set (&verify_peer, value));
 +  else if (strcasecmp (key, "verifyhost") == 0)
 +    return (config_set (&verify_host, value));
    else if (strcasecmp (key, "cacert") == 0)
      return (config_set (&cacert, value));
    else
@@@ -136,24 -128,6 +136,24 @@@ static int init (void
      curl_easy_setopt (curl, CURLOPT_URL, url);
    }
  
 +  if ((verify_peer == NULL) || (strcmp (verify_peer, "true") == 0))
 +  {
 +    curl_easy_setopt (curl, CURLOPT_SSL_VERIFYPEER, 1);
 +  }
 +  else
 +  {
 +    curl_easy_setopt (curl, CURLOPT_SSL_VERIFYPEER, 0);
 +  }
 +
 +  if ((verify_host == NULL) || (strcmp (verify_host, "true") == 0))
 +  {
 +    curl_easy_setopt (curl, CURLOPT_SSL_VERIFYHOST, 2);
 +  }
 +  else
 +  {
 +    curl_easy_setopt (curl, CURLOPT_SSL_VERIFYHOST, 0);
 +  }
 +
    if (cacert != NULL)
    {
      curl_easy_setopt (curl, CURLOPT_CAINFO, cacert);
@@@ -177,9 -151,9 +177,9 @@@ static void submit (char *type, char *i
    vl.values = values;
    vl.values_len = 1;
    vl.time = time (NULL);
-   strcpy (vl.host, hostname_g);
-   strcpy (vl.plugin, "nginx");
-   strcpy (vl.plugin_instance, "");
+   sstrncpy (vl.host, hostname_g, sizeof (vl.host));
+   sstrncpy (vl.plugin, "nginx", sizeof (vl.plugin));
+   sstrncpy (vl.plugin_instance, "", sizeof (vl.plugin_instance));
  
    if (inst != NULL)
    {
@@@ -197,6 -171,7 +197,7 @@@ static int nginx_read (void
    char *ptr;
    char *lines[16];
    int   lines_num = 0;
+   char *saveptr;
  
    char *fields[16];
    int   fields_num;
    }
  
    ptr = nginx_buffer;
-   while ((lines[lines_num] = strtok (ptr, "\n\r")) != NULL)
+   saveptr = NULL;
+   while ((lines[lines_num] = strtok_r (ptr, "\n\r", &saveptr)) != NULL)
    {
      ptr = NULL;
      lines_num++;
diff --combined src/ntpd.c
@@@ -302,9 -302,9 +302,9 @@@ static void ntpd_submit (char *type, ch
        vl.values = values;
        vl.values_len = 1;
        vl.time = time (NULL);
-       strcpy (vl.host, hostname_g);
-       strcpy (vl.plugin, "ntpd");
-       strcpy (vl.plugin_instance, "");
+       sstrncpy (vl.host, hostname_g, sizeof (vl.host));
+       sstrncpy (vl.plugin, "ntpd", sizeof (vl.plugin));
+       sstrncpy (vl.plugin_instance, "", sizeof (vl.plugin_instance));
        strncpy (vl.type_instance, type_inst, sizeof (vl.type_instance));
  
        plugin_dispatch_values (type, &vl);
@@@ -613,7 -613,7 +613,7 @@@ static int ntpd_receive_response (int *
                if (pkt_item_len < res_item_size)
                        pkt_padding = res_item_size - pkt_item_len;
                DEBUG ("res_item_size = %i; pkt_padding = %i;",
 -                              res_item_size, pkt_padding);
 +                              res_item_size, (int) pkt_padding);
  
                /* Extract the sequence number */
                pkt_sequence = INFO_SEQ (res.auth_seq);
                 * We start by allocating some more memory.
                 */
                DEBUG ("realloc (%p, %i)", (void *) *res_data,
 -                              (items_num + pkt_item_num) * res_item_size);
 +                              (int) (items_num + pkt_item_num) * res_item_size);
                items = realloc ((void *) *res_data,
                                (items_num + pkt_item_num) * res_item_size);
                if (items == NULL)
diff --combined src/perl.c
@@@ -61,9 -61,8 +61,9 @@@
  #define PLUGIN_SHUTDOWN 3
  #define PLUGIN_LOG      4
  #define PLUGIN_NOTIF    5
 +#define PLUGIN_FLUSH    6
  
 -#define PLUGIN_TYPES    6
 +#define PLUGIN_TYPES    7
  
  #define PLUGIN_DATASET  255
  
@@@ -78,8 -77,6 +78,8 @@@ void boot_DynaLoader (PerlInterpreter *
  static XS (Collectd_plugin_register_ds);
  static XS (Collectd_plugin_unregister_ds);
  static XS (Collectd_plugin_dispatch_values);
 +static XS (Collectd_plugin_flush_one);
 +static XS (Collectd_plugin_flush_all);
  static XS (Collectd_plugin_dispatch_notification);
  static XS (Collectd_plugin_log);
  static XS (Collectd_call_by_name);
@@@ -133,8 -130,6 +133,8 @@@ static struct 
        { "Collectd::plugin_register_data_set",   Collectd_plugin_register_ds },
        { "Collectd::plugin_unregister_data_set", Collectd_plugin_unregister_ds },
        { "Collectd::plugin_dispatch_values",     Collectd_plugin_dispatch_values },
 +      { "Collectd::plugin_flush_one",           Collectd_plugin_flush_one },
 +      { "Collectd::plugin_flush_all",           Collectd_plugin_flush_all },
        { "Collectd::plugin_dispatch_notification",
                Collectd_plugin_dispatch_notification },
        { "Collectd::plugin_log",                 Collectd_plugin_log },
@@@ -153,7 -148,6 +153,7 @@@ struct 
        { "Collectd::TYPE_SHUTDOWN",   PLUGIN_SHUTDOWN },
        { "Collectd::TYPE_LOG",        PLUGIN_LOG },
        { "Collectd::TYPE_NOTIF",      PLUGIN_NOTIF },
 +      { "Collectd::TYPE_FLUSH",      PLUGIN_FLUSH },
        { "Collectd::TYPE_DATASET",    PLUGIN_DATASET },
        { "Collectd::DS_TYPE_COUNTER", DS_TYPE_COUNTER },
        { "Collectd::DS_TYPE_GAUGE",   DS_TYPE_GAUGE },
@@@ -559,7 -553,7 +559,7 @@@ static int pplugin_dispatch_values (pTH
                list.host[DATA_MAX_NAME_LEN - 1] = '\0';
        }
        else {
-               strcpy (list.host, hostname_g);
+               sstrncpy (list.host, hostname_g, sizeof (list.host));
        }
  
        if (NULL != (tmp = hv_fetch (values, "plugin", 6, 0))) {
@@@ -761,12 -755,6 +761,12 @@@ static int pplugin_call_all (pTHX_ int 
  
                XPUSHs (sv_2mortal (newRV_noinc ((SV *)notif)));
        }
 +      else if (PLUGIN_FLUSH == type) {
 +              /*
 +               * $_[0] = $timeout;
 +               */
 +              XPUSHs (sv_2mortal (newSViv (va_arg (ap, int))));
 +      }
  
        PUTBACK;
  
@@@ -903,54 -891,6 +903,54 @@@ static XS (Collectd_plugin_dispatch_val
  } /* static XS (Collectd_plugin_dispatch_values) */
  
  /*
 + * Collectd::plugin_flush_one (timeout, name).
 + *
 + * timeout:
 + *   timeout to use when flushing the data
 + *
 + * name:
 + *   name of the plugin to flush
 + */
 +static XS (Collectd_plugin_flush_one)
 +{
 +      dXSARGS;
 +
 +      if (2 != items) {
 +              log_err ("Usage: Collectd::plugin_flush_one(timeout, name)");
 +              XSRETURN_EMPTY;
 +      }
 +
 +      log_debug ("Collectd::plugin_flush_one: timeout = %i, name = \"%s\"",
 +                      (int)SvIV (ST (0)), SvPV_nolen (ST (1)));
 +
 +      if (0 == plugin_flush_one ((int)SvIV (ST (0)), SvPV_nolen (ST (1))))
 +              XSRETURN_YES;
 +      else
 +              XSRETURN_EMPTY;
 +} /* static XS (Collectd_plugin_flush_one) */
 +
 +/*
 + * Collectd::plugin_flush_all (timeout).
 + *
 + * timeout:
 + *   timeout to use when flushing the data
 + */
 +static XS (Collectd_plugin_flush_all)
 +{
 +      dXSARGS;
 +
 +      if (1 != items) {
 +              log_err ("Usage: Collectd::plugin_flush_all(timeout)");
 +              XSRETURN_EMPTY;
 +      }
 +
 +      log_debug ("Collectd::plugin_flush_all: timeout = %i", (int)SvIV (ST (0)));
 +
 +      plugin_flush_all ((int)SvIV (ST (0)));
 +      XSRETURN_YES;
 +} /* static XS (Collectd_plugin_flush_all) */
 +
 +/*
   * Collectd::plugin_dispatch_notification (notif).
   *
   * notif:
@@@ -1118,7 -1058,7 +1118,7 @@@ static c_ithread_t *c_ithread_create (P
  
        aTHX = t->interp;
  
 -      if (NULL != base) {
 +      if ((NULL != base) && (NULL != PL_endav)) {
                av_clear (PL_endav);
                av_undef (PL_endav);
                PL_endav = Nullav;
@@@ -1255,25 -1195,6 +1255,25 @@@ static int perl_notify (const notificat
        return pplugin_call_all (aTHX_ PLUGIN_NOTIF, notif);
  } /* static int perl_notify (const notification_t *) */
  
 +static int perl_flush (const int timeout)
 +{
 +      dTHX;
 +
 +      if (NULL == perl_threads)
 +              return 0;
 +
 +      if (NULL == aTHX) {
 +              c_ithread_t *t = NULL;
 +
 +              pthread_mutex_lock (&perl_threads->mutex);
 +              t = c_ithread_create (perl_threads->head->interp);
 +              pthread_mutex_unlock (&perl_threads->mutex);
 +
 +              aTHX = t->interp;
 +      }
 +      return pplugin_call_all (aTHX_ PLUGIN_FLUSH, timeout);
 +} /* static int perl_flush (const int) */
 +
  static int perl_shutdown (void)
  {
        c_ithread_t *t = NULL;
        plugin_unregister_init ("perl");
        plugin_unregister_read ("perl");
        plugin_unregister_write ("perl");
 +      plugin_unregister_flush ("perl");
  
        ret = pplugin_call_all (aTHX_ PLUGIN_SHUTDOWN);
  
@@@ -1441,7 -1361,7 +1441,7 @@@ static int init_pi (int argc, char **ar
                log_err ("init_pi: pthread_key_create failed");
  
                /* this must not happen - cowardly giving up if it does */
 -              exit (1);
 +              return -1;
        }
  
  #ifdef __FreeBSD__
  
        if (0 != perl_parse (aTHX_ xs_init, argc, argv, NULL)) {
                log_err ("init_pi: Unable to bootstrap Collectd.");
 -              exit (1);
 +
 +              perl_destruct (perl_threads->head->interp);
 +              perl_free (perl_threads->head->interp);
 +              sfree (perl_threads);
 +
 +              pthread_key_delete (perl_thr_key);
 +              return -1;
        }
  
        /* Set $0 to "collectd" because perl_parse() has to set it to "-e". */
        plugin_register_read ("perl", perl_read);
  
        plugin_register_write ("perl", perl_write);
 +      plugin_register_flush ("perl", perl_flush);
        plugin_register_shutdown ("perl", perl_shutdown);
        return 0;
  } /* static int init_pi (const char **, const int) */
@@@ -1524,9 -1437,7 +1524,9 @@@ static int perl_config_loadplugin (pTHX
                return (1);
        }
  
 -      init_pi (perl_argc, perl_argv);
 +      if (0 != init_pi (perl_argc, perl_argv))
 +              return -1;
 +
        assert (NULL != perl_threads);
        assert (NULL != perl_threads->head);
  
diff --combined src/rrdtool.c
@@@ -195,7 -195,7 +195,7 @@@ static int rra_get (char ***ret, const 
                span = rts[i];
  
                if ((span / ss) < rrarows)
 -                      continue;
 +                      span = ss * rrarows;
  
                if (cdp_len == 0)
                        cdp_len = 1;
@@@ -285,7 -285,7 +285,7 @@@ static int ds_get (char ***ret, const d
  
                if (isnan (d->min))
                {
-                       strcpy (min, "U");
+                       sstrncpy (min, "U", sizeof (min));
                }
                else
                {
  
                if (isnan (d->max))
                {
-                       strcpy (max, "U");
+                       sstrncpy (max, "U", sizeof (max));
                }
                else
                {
@@@ -947,20 -947,6 +947,20 @@@ static int rrd_write (const data_set_t 
        return (status);
  } /* int rrd_write */
  
 +static int rrd_flush (const int timeout)
 +{
 +      pthread_mutex_lock (&cache_lock);
 +
 +      if (cache == NULL) {
 +              pthread_mutex_unlock (&cache_lock);
 +              return (0);
 +      }
 +
 +      rrd_cache_flush (timeout);
 +      pthread_mutex_unlock (&cache_lock);
 +      return (0);
 +} /* int rrd_flush */
 +
  static int rrd_config (const char *key, const char *value)
  {
        if (strcasecmp ("CacheTimeout", key) == 0)
@@@ -1166,6 -1152,5 +1166,6 @@@ void module_register (void
                        config_keys, config_keys_num);
        plugin_register_init ("rrdtool", rrd_init);
        plugin_register_write ("rrdtool", rrd_write);
 +      plugin_register_flush ("rrdtool", rrd_flush);
        plugin_register_shutdown ("rrdtool", rrd_shutdown);
  }
diff --combined src/users.c
  #include "common.h"
  #include "plugin.h"
  
 +#if HAVE_STATGRAB_H
 +# include <statgrab.h>
 +#endif /* HAVE_STATGRAB_H */
 +
  #if HAVE_UTMPX_H
  # include <utmpx.h>
  /* #endif HAVE_UTMPX_H */
@@@ -49,8 -45,8 +49,8 @@@ static void users_submit (gauge_t value
        vl.values = values;
        vl.values_len = 1;
        vl.time = time (NULL);
-       strcpy (vl.host, hostname_g);
-       strcpy (vl.plugin, "users");
+       sstrncpy (vl.host, hostname_g, sizeof (vl.host));
+       sstrncpy (vl.plugin, "users", sizeof (vl.plugin));
  
        plugin_dispatch_values ("users", &vl);
  } /* void users_submit */
@@@ -93,16 -89,6 +93,16 @@@ static int users_read (void
        users_submit (users);
  /* #endif HAVE_GETUTENT */
  
 +#elif HAVE_LIBSTATGRAB
 +      sg_user_stats *us;
 +
 +      us = sg_get_user_stats ();
 +      if (us == NULL)
 +              return (-1);   
 +
 +      users_submit ((gauge_t) us->num_entries);
 +/* #endif HAVE_LIBSTATGRAB */
 +
  #else
  # error "No applicable input method."
  #endif
diff --combined src/utils_cmd_putval.c
  #include "common.h"
  #include "plugin.h"
  
 +#define print_to_socket(fh, ...) \
 +      if (fprintf (fh, __VA_ARGS__) < 0) { \
 +              char errbuf[1024]; \
 +              WARNING ("handle_putval: failed to write to socket #%i: %s", \
 +                              fileno (fh), sstrerror (errno, errbuf, sizeof (errbuf))); \
 +              return -1; \
 +      }
 +
  static int parse_value (const data_set_t *ds, value_list_t *vl,
                const char *type,
                FILE *fh, char *buffer)
@@@ -44,7 -36,7 +44,7 @@@
        char *value_str = strchr (time_str, ':');
        if (value_str == NULL)
        {
 -              fprintf (fh, "-1 No time found.\n");
 +              print_to_socket (fh, "-1 No time found.\n");
                return (-1);
        }
        *value_str = '\0'; value_str++;
@@@ -84,7 -76,7 +84,7 @@@
                                "Number of values incorrect: "
                                "Got %i, expected %i. Identifier is `%s'.",
                                i, vl->values_len, identifier);
 -              fprintf (fh, "-1 Number of values incorrect: "
 +              print_to_socket (fh, "-1 Number of values incorrect: "
                                "Got %i, expected %i.\n",
                                i, vl->values_len);
                return (-1);
@@@ -138,9 -130,10 +138,9 @@@ int handle_putval (FILE *fh, char **fie
        {
                DEBUG ("cmd putval: Wrong number of fields: %i",
                                fields_num);
 -              fprintf (fh, "-1 Wrong number of fields: Got %i, "
 +              print_to_socket (fh, "-1 Wrong number of fields: Got %i, "
                                "expected at least 3.\n",
                                fields_num);
 -              fflush (fh);
                return (-1);
        }
  
        if (status != 0)
        {
                DEBUG ("cmd putval: Cannot parse `%s'", fields[1]);
 -              fprintf (fh, "-1 Cannot parse identifier.\n");
 -              fflush (fh);
 +              print_to_socket (fh, "-1 Cannot parse identifier.\n");
                sfree (identifier_copy);
                return (-1);
        }
                        || ((type_instance != NULL)
                                && (strlen (type_instance) >= sizeof (vl.type_instance))))
        {
 -              fprintf (fh, "-1 Identifier too long.\n");
 -              fflush (fh);
 +              print_to_socket (fh, "-1 Identifier too long.\n");
                sfree (identifier_copy);
                return (-1);
        }
  
-       strcpy (vl.host, hostname);
-       strcpy (vl.plugin, plugin);
+       sstrncpy (vl.host, hostname, sizeof (vl.host));
+       sstrncpy (vl.plugin, plugin, sizeof (vl.plugin));
        if (plugin_instance != NULL)
-               strcpy (vl.plugin_instance, plugin_instance);
+               sstrncpy (vl.plugin_instance, plugin_instance, sizeof (vl.plugin_instance));
        if (type_instance != NULL)
-               strcpy (vl.type_instance, type_instance);
+               sstrncpy (vl.type_instance, type_instance, sizeof (vl.type_instance));
  
        ds = plugin_get_ds (type);
        if (ds == NULL) {
        vl.values = (value_t *) malloc (vl.values_len * sizeof (value_t));
        if (vl.values == NULL)
        {
 -              fprintf (fh, "-1 malloc failed.\n");
 -              fflush (fh);
 +              print_to_socket (fh, "-1 malloc failed.\n");
                sfree (identifier_copy);
                return (-1);
        }
                {
                        if (parse_option (&vl, fields[i]) != 0)
                        {
 -                              fprintf (fh, "-1 Error parsing option `%s'\n",
 +                              print_to_socket (fh, "-1 Error parsing option `%s'\n",
                                                fields[i]);
                                break;
                        }
        /* Done parsing the options. */
  
        if (i == fields_num)
 -              fprintf (fh, "0 Success\n");
 -      fflush (fh);
 +              print_to_socket (fh, "0 Success\n");
  
        sfree (vl.values); 
        sfree (identifier_copy);