tokkee.org

treewide: fix invocation of c_avl_create

Fixes the following warning on Solaris:

|c_avl_tree_t *c_avl_create (int (*compare) (const void *, const
void *));
|              ^  line 54, utils_avltree.h
|                 included in line 34, utils_vl_lookup.c
|
|  obj->by_type_tree = c_avl_create ((void *) strcmp);
|                                     ^  line 567,
utils_vl_lookup.c
E_ARG_INCOMPATIBLE_WITH_ARG_L, argument #1 is incompatible with
prototype:
        prototype: pointer to function(pointer to const void, pointer to
const void) returning int : "src/daemon/utils_avltree.h", line 54
        argument : pointer to void

I'll look into writing a generic function to compare avl keys so
we don't need to do all the casting.

email plugin: fix lint warnings

set but not used in function
    (468) err in open_connection
    (574) err in email_init

utils_llist.c: remove unneccesary include

write_kafka plugin: remove unneccesary includes

threshold plugin: remove unneccesary include

mqtt plugin: remove unused include

battery plugin: remove unused include

memcachec plugin: correct error message

curl plugin: correct error message

modbus plugin: unsigned value can't be negative

collectd-tg: remove dead code

fscache plugin: unsigned value can't be negative

uuid plugin: fix inclusion of libhal.h

Follow up of 34db18b

configure.ac: always use pkg-config to detect hal

Hal needs dbus, but we need pkg-config to tell us that.
Now that pkg-config is not optional anymore, rewrite the
check to always use it. Also add a check for the header.

Fixes building the uuid plugin on Solaris 11.3

ganglia plugin: unsigned value can't be negative

treewide: remove unused includes

Ignore lint files

apache plugin: fix compiler warning on Solaris

memcached plugin: fix compiler warning on Solaris

|  ud.free_func = (void *) memcached_free;
|               ^  line 559, memcached.c
assignment type mismatch:
pointer to function(pointer to void) returning void "=" pointer to void

Casting a function pointer to a void pointer is undefined behaviour
in C (it is defined in POSIX however)

mysql plugin: remove unneccesary cast

network plugin: unsigned variables can't be negative

utils_dns.c: unsigned variable can't be negative

utils_dns.c: fix strange linebreak

rrdtool plugin: unsigned value can't be negative

ted plugin: 0xAA doesn't fit in a signed char

openldap plugin: rc is only used once

interface plugin: val is only used once

Make chrony compile with Solaris Studio compiler

Ideally we'll have autoconf checks for supported attributes,
will look into that later.

utils_dns.c: remove dead code

zone plugin: remove unused code

collectctl: remove dead code

teamspeak plugin: fix indentation

curl_xml plugin: fix indentation

utils_curl_stats.c: fix order of calloc arguments

libcollectdclient: remove unneccesary cast

processes plugin: fix indentation

log_logstash plugin: fix indentation

curl_json plugin: fix indentation

conntrack plugin: fix indentation

utils_cmd_putval.c: fix indentation

utils_threshold.c: fix indentation

collectd.h: guard define of __attribute__

It might have already been defined to a no-op in <sys/cdefs.h>

collectd.h: remove unused defines

collectd.h: fix indentation

Merge branch 'collectd-5.5'

Merge branch 'collectd-5.4' into collectd-5.5

network plugin: Fix heap overflow in parse_packet().

Emilien Gaspar has identified a heap overflow in parse_packet(), the
function used by the network plugin to parse incoming network packets.

This is a vulnerability in collectd, though the scope is not clear at
this point. At the very least specially crafted network packets can be
used to crash the daemon. We can't rule out a potential remote code
execution though.

Fixes: CVE-2016-6254

Merge pull request #1765 from vincentbernat/fix/kafka-partition-selection

write_kafka: ensure key is properly hashed to select partition

Merge pull request #1785 from rinigus/master_issue1781

disk plugin: release udev-based name. Fixes #1781

Merge pull request #1794 from swt2c/smart_use_serial

smart plugin: add option to store data using serial number

Merge pull request #1796 from tokkee/sh/configure

configure: Use AC_MSG_RESULT to print the config summary.

configure: Use AC_MSG_RESULT to print the config summary.

This way, it will show up in config.log as well.

configure: Add missing end fold marker (for Vim).

Merge pull request #1783 from matwey/ntp

ntpd: Check STA_NANO in the server status

ntpd: Check STA_NANO in the server status

When STA_NANO is set in ik->status, then results are
expressed in nanoseconds (instead of microseconds by default).

Signed-off-by: Matwey V. Kornilov <matwey.kornilov@gmail.com>

zookeeper plugin: remove unneccesary cast

ntpd plugin: remove unneccesary casts

collectd-nagios: fix some whitespace

virt plugin: fix some whitespace

Merge pull request #1793 from rubenk/fix-riemann-plugin-on-32bit-solaris

Fix write_riemann plugin build on 32-bit Solaris

Fix write_riemann plugin build on 32-bit Solaris

Fixes #1782

CC       write_riemann_la-write_riemann.lo
In file included from ./daemon/collectd.h:31:0,
                 from write_riemann.c:34:
./config.h:1664:0: error: "_FILE_OFFSET_BITS" redefined [-Werror]
 #define _FILE_OFFSET_BITS 64
 ^
In file included from /usr/include/limits.h:17:0,
                 from
/opt/csw/lib/gcc/i386-pc-solaris2.10/5.2.0/include-fixed/limits.h:168,
                 from
/opt/csw/lib/gcc/i386-pc-solaris2.10/5.2.0/include-fixed/syslimits.h:7,
                 from
/opt/csw/lib/gcc/i386-pc-solaris2.10/5.2.0/include-fixed/limits.h:34,
                 from /opt/csw/include/protobuf-c/protobuf-c.h:200,
                 from /opt/csw/include/riemann/proto/riemann.pb-c.h:7,
                 from /opt/csw/include/riemann/attribute.h:21,
                 from /opt/csw/include/riemann/riemann-client.h:23,
                 from write_riemann.c:31:
/opt/csw/lib/gcc/i386-pc-solaris2.10/5.2.0/include-fixed/sys/feature_tests.h:196:0:
note: this is the location of the previous definition
 #define _FILE_OFFSET_BITS 32
 ^
cc1: all warnings being treated as errors
Makefile:4645: recipe for target 'write_riemann_la-write_riemann.lo'
failed
gmake[3]: *** [write_riemann_la-write_riemann.lo] Error 1
gmake[3]: Leaving directory
'/home/dam/mgar/pkg/collectd/trunk/work/solaris10-i386/build-isa-pentium_pro/collectd-5.5.1.1126.g37fe166/src'
Makefile:4879: recipe for target 'all-recursive' failed
gmake[2]: *** [all-recursive] Error 1
gmake[2]: Leaving directory
'/home/dam/mgar/pkg/collectd/trunk/work/solaris10-i386/build-isa-pentium_pro/collectd-5.5.1.1126.g37fe166/src'
Makefile:3209: recipe for target 'all' failed
gmake[1]: *** [all] Error 2
gmake[1]: Leaving directory
'/home/dam/mgar/pkg/collectd/trunk/work/solaris10-i386/build-isa-pentium_pro/collectd-5.5.1.1126.g37fe166/src'
Makefile:572: recipe for target 'all-recursive' failed
gmake: *** [all-recursive] Error 1
gmake: Leaving directory
'/home/dam/mgar/pkg/collectd/trunk/work/solaris10-i386/build-isa-pentium_pro/collectd-5.5.1.1126.g37fe166'
/home/dam/mgar/pkg/.buildsys/v2/gar//gar.lib.mk:874: die Regel für Ziel
„build-work/solaris10-i386/build-isa-pentium_pro/collectd-5.5.1.1126.g37fe166/Makefile“
scheiterte

smart plugin: add option to store data using serial number

Since a device name (e.g., /dev/sda) can change, allow using the serial number
to store SMART data as this will ensure the data for a given drive is always
kept together.

Merge pull request #1770 from swt2c/smart_ignore_sleep

smart plugin: add IgnoreSleepMode option to ignore sleeping disks

smart plugin: add IgnoreSleepMode option to ignore sleeping disks

This option enables the smart plugin to use disks that libatasmart mistakenly
reports as asleep.  This happens because libatasmart has not been updated to
incorporate support for newer idle states in the ATA spec.

Merge pull request #1773 from skob/master

Add slow queries statistics to mysql module

Add slow queries statistics to mysql module

Merge pull request #1774 from mmckinst/rdkafka_now_libkafka

rdkafka is called librdkafka

disk plugin: release udev-based name. Fixes #1781

rdkafka is called librdkafka

The [spec file for
librdkafka](https://github.com/edenhill/librdkafka/blob/master/packaging/rpm/librdkafka.spec)
calls its package 'librdkafka-devel', not 'rdkafka-devel'. Searching the
web for 'rdkafka-devel RPM' turns up nothing, all the RPMs are named
'librdkafka-devel'.

write_kafka: ensure key is properly hashed to select partition

Otherwise, partition is selected solely on the first letter of the
key. If all your hosts are starting with the same letter, everything
ends up in a single partition.

README: add reference to libxen

Merge branch 'collectd-5.5'

README: mention 3 missing plugins

AUTHORS: mention new contributors and update plugin attributions

Merge branch 'collectd-5.4' into collectd-5.5

src/*.pod: fix minor warnings spotted by podchecker

collectd.conf.pod: fix minor typos in ntpd plugin section

doc: add chronyd section to manpage

Merge pull request #1004 from tokkee/sh/curl

Add support for detailed cURL statistics

Language and typo fixes.

utils_curl_stats: Drop the nonnull attribute from curl_stats_from_config.

It's not overly useful after all but generates false compiler warnings /
errors on Xenial using CLANG, since we also check the parameter in the
function (which is what we want in the end and there's no risk in actually
calling it with a NULL value).

curl stats: Use cf_util_get_boolean().

curl stats: Simplified some error checks and removed unused code.

configure / curl stats: Check for CURLINFO_APPCONNECT_TIME.

It's available since cURL 7.19 but RHEL5 ships an older version.

collectd.conf(5): Briefly explain MeasureResponseTime vs TotalTime.

curl stats: Use CamelCase identifiers instead of using underscores.

This is more consistent with the usual collectd style but still close enough
to the names used in cURL.

collectd.conf(5): Add documentation for cURL statistics.

Add request specific statistics to all CURL-based plugins.

All metrics supported by curl_easy_getinfo can be retrieved and dispatched for
each request done through CURL. All statistics are optional and disabled by
default.

configure.ac: indent and sort some conditionals

configure.ac: remove obsolete macro

We don't need it anymore. See
http://gnu-autoconf.7623.n7.nabble.com/Bug-389558-autoconf-AC-PROG-GCC-TRADITIONAL-throws-error-in-comparision-td1885.html

chrony plugin: use ANSI function definitions

chrony.c:392:1: warning: function declaration isn’t a prototype
[-Wstrict-prototypes]
 chrony_set_timeout()
 ^
chrony.c: In function ‘chrony_set_timeout’:
chrony.c:392:1: warning: old-style function definition
[-Wold-style-definition]
chrony.c: At top level:

Merge pull request #1758 from rubenk/README

README: use https for collectd.org link

Merge pull request #1759 from rubenk/ansi-function-declaration

uc_get_iterator: use ansi function declaration

Merge branch 'collectd-5.5'

Merge branch 'collectd-5.4' into collectd-5.5

Bug#1059 : Remove any white spaces in ethstats stat names (affects VMXNet3 interface)

uc_get_iterator: use ansi function declaration

In file included from common.c:37:0:
utils_cache.h:76:1: warning: function declaration isn’t a prototype
[-Wstrict-prototypes]
 uc_iter_t *uc_get_iterator ();
 ^

README: use https for collectd.org link

Merge pull request #1750 from tokkee/sh/cli

daemon: Let all command line options override global options from the config.

configfile: Log errors when trying to access unknown global options.