Merge branch 'collectd-5.6'
authorFlorian Forster <octo@collectd.org>
Mon, 7 Nov 2016 07:52:38 +0000 (08:52 +0100)
committerFlorian Forster <octo@collectd.org>
Mon, 7 Nov 2016 07:52:38 +0000 (08:52 +0100)
1  2 
configure.ac
src/apcups.c
src/modbus.c
src/write_kafka.c

diff --cc configure.ac
Simple merge
diff --cc src/apcups.c
@@@ -414,13 -415,21 +415,15 @@@ static int apcups_config (oconfig_item_
        return (0);
  } /* int apcups_config */
  
- static void apc_submit_generic (const char *type, const char *type_inst, double value)
+ static void apc_submit_generic (const char *type, const char *type_inst, gauge_t value)
  {
 -      value_t values[1];
--      value_list_t vl = VALUE_LIST_INIT;
 -
+       if (isnan (value))
+               return;
  
 -      values[0].gauge = value;
 -
 -      vl.values = values;
++      value_list_t vl = VALUE_LIST_INIT;
 +      vl.values = &(value_t) { .gauge = value };
        vl.values_len = 1;
 -      sstrncpy (vl.host, hostname_g, sizeof (vl.host));
        sstrncpy (vl.plugin, "apcups", sizeof (vl.plugin));
 -      sstrncpy (vl.plugin_instance, "", sizeof (vl.plugin_instance));
        sstrncpy (vl.type, type, sizeof (vl.type));
        sstrncpy (vl.type_instance, type_inst, sizeof (vl.type_instance));
  
diff --cc src/modbus.c
@@@ -470,12 -471,9 +471,11 @@@ static int mb_read_data (mb_host_t *hos
    }
    else if (host->conntype == MBCONN_TCP)
    {
-     status = getpeername (modbus_get_socket (host->connection),
-         (struct sockaddr *) &(struct sockaddr_storage) { 0 },
-         &(socklen_t) { sizeof (struct sockaddr_storage) });
-     if (status != 0)
 +    /* getpeername() is used only to determine if the socket is connected, not
 +     * because we're really interested in the peer's IP address. */
+     if (getpeername (modbus_get_socket (host->connection),
+           (void *) &(struct sockaddr_storage) {0},
+           &(socklen_t) {sizeof(struct sockaddr_storage)}) != 0)
        status = errno;
    }
  
Simple merge