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

hugpages: fix compilation error

Fix error: format '%lu' expects type 'long unsigned int', but
argument 6 has type 'derive_t'. For line 93.

Change-Id: I4f164d238cd4fa6f8a69d78b2906042aa3c5da72
Signed-off-by: Maryam Tahhan <maryam.tahhan@intel.com>

formatting changes in cpusleep

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

hugepages plugin: a stats plugin for hugepages

Plugin that allows stats for hugepages to be collected.

Change-Id: Ide5ff31c54206cf5ee3d6155899bd53e8edae7bd
Signed-off-by: Jaroslav Safka <jaroslavx.safka@intel.com>
Signed-off-by: Kim Jones <kim-marie.jones@intel.com>
Signed-off-by: Maryam Tahhan <maryam.tahhan@intel.com>

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

add ActiveInterfaceOnly option to interface plugin

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

adding configure check for clock boottime and monotonic

typo in error message fixed

adding CPU sleep plugin

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.

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

This replaces 09fb81e130ad70bf0547cec3d3c349ce8cb092cf which provided a
specific solution for the `PIDFile' option. The new solution is generic and
more nicely abstracted without the need for global variables across multiple
modules.

Fixes #366

Merge pull request #1747 from rubenk/write_graphite_check_fd

Write graphite check fd

wg_send_buffer: return error if fd < 0

CID 116310

write_graphite: remove dead assignment

status is immediately overwritten in the next line.

Merge pull request #1746 from rubenk/swrite-check-for-open-fd

swrite: return error if fd is negative

swrite: return error if fd is negative

Merge pull request #1743 from rubenk/apcups-coverity

apcups: ignore result of swrite in net_shutdown

apcups: ignore result of swrite in net_shutdown

CID 37945

Add test logs to .gitignore

automake: disable gzipped tarballs

We only use bz2.

Fixes make distcheck on FreeBSD-11-CURRENT

test -n ""  || find "collectd-5.5.1.1086.g41e7c34" -type d ! -perm -755  -exec chmod u+rwx,go+rx {} \; -o  ! -type d ! -perm -444 -links 1 -exec chmod a+r {} \; -o  ! -type d ! -perm -400 -exec chmod a+r {} \; -o  ! -type d ! -perm -444 -exec /bin/sh /home/ruben/src/collectd/libltdl/config/install-sh -c -m a+r {} {} \;  || chmod -R a+r "collectd-5.5.1.1086.g41e7c34"
tardir=collectd-5.5.1.1086.g41e7c34 && tar --format=posix -chf - "$tardir" | BZIP2=${BZIP2--9} bzip2 -c >collectd-5.5.1.1086.g41e7c34.tar.bz2
tardir=collectd-5.5.1.1086.g41e7c34 && tar --format=posix -chf - "$tardir" | GZIP=--best gzip -c >collectd-5.5.1.1086.g41e7c34.tar.gz
if test -d "collectd-5.5.1.1086.g41e7c34"; then  find "collectd-5.5.1.1086.g41e7c34" -type d ! -perm -200 -exec chmod u+w {} ';'  && rm -rf "collectd-5.5.1.1086.g41e7c34"  || { sleep 5 && rm -rf "collectd-5.5.1.1086.g41e7c34"; };  else :; fi
case 'collectd-5.5.1.1086.g41e7c34.tar.gz collectd-5.5.1.1086.g41e7c34.tar.bz2' in  *.tar.gz*)  GZIP=--best gzip -dc collectd-5.5.1.1086.g41e7c34.tar.gz | tar -xf - ;; *.tar.bz2*)  bzip2 -dc collectd-5.5.1.1086.g41e7c34.tar.bz2 | tar -xf - ;; *.tar.lz*)  lzip -dc collectd-5.5.1.1086.g41e7c34.tar.lz | tar -xf - ;; *.tar.xz*)  xz -dc collectd-5.5.1.1086.g41e7c34.tar.xz | tar -xf - ;; *.tar.Z*)  uncompress -c collectd-5.5.1.1086.g41e7c34.tar.Z | tar -xf - ;; *.shar.gz*)  GZIP=--best gzip -dc collectd-5.5.1.1086.g41e7c34.shar.gz | unshar ;; *.zip*)  unzip collectd-5.5.1.1086.g41e7c34.zip ;; esac
tar: Parse error: SCHILY.acl.default
tar: Parse error: SCHILY.acl.default
tar: Error exit delayed from previous errors.
*** Error code 1

Stop.
make: stopped in /usr/home/ruben/src/collectd

src/Makefile: Add valgrind.FreeBSD.suppress to the dist tarball.

Merge pull request #1742 from tokkee/sh/suppress

Valgrind: Suppress a bogus invalid read on FreeBSD.

Merge pull request #1741 from rubenk/fix-leak-in-disk-plugin

Plug a memory leak in disk plugin

Valgrind: Suppress a bogus invalid read on FreeBSD.

Valgrind reports an invalid read in common.c:parse_value. The error is
supposed to happen in strlen() on a just previously strdup()ed string.

Possibly, this is some sort of alignment issue. The read size is reported to
be 4 on a buffer of size 3.

https://bugzilla.redhat.com/show_bug.cgi?id=678518 may also be related.

Plug a memory leak in disk plugin

Introduced by bdbc066 so no need to backport.
CID 116311

Makefile: Add testwrapper.sh to dist tarballs.

Merge pull request #1740 from rubenk/correct-am_cflags

Append -Werror to the right flags

chrony plugin: constify 2 function arguments

Fixes a couple of dozen of these warnings:
chrony.c:889:20: warning: passing 'const char [14]' to parameter of type
'char *' discards qualifiers
[-Wincompatible-pointer-types-discards-qualifiers]
  chrony_push_data("clock_stratum", "chrony", (__extension__ ({ unsigned
short int __v, __x = (unsigned short int)
(chrony_resp.body.tracking.f_stratum); if (__builtin_constant_p (__x))
__v = ((unsigned short int) ((((__x) >> 8) & 0xff) | (((__x) & 0xff) <<
8))); else __asm__ ("rorw $8, %w0" : "=r" (__v) : "0" (__x) : "cc");
__v; })));
                   ^~~~~~~~~~~~~~~

chrony plugin: fix clang warning

chrony.c:745:11: warning: declaration shadows a local variable
[-Wshadow]
      int fh = open("/dev/random", 00);
          ^

collectd.spec: add missing %endif

configure.ac: fix autoreconf warnings

configure.ac:804: warning: AC_LANG_CONFTEST: no AC_LANG_SOURCE call
detected in body
../../lib/autoconf/lang.m4:193: AC_LANG_CONFTEST is expanded from...
configure.ac:804: the top level

Append -Werror to the right flags

testwrapper.sh: Use /bin/sh instead of /bin/bash.

We don't use any bashisms.

Merge pull request #1739 from tokkee/sh/memcheck

unit tests: Run all tests through Valgrind's memcheck, if available.

Actually add the testwrapper.sh script.

… it's now used for running all tests.

configure: Drop unneeded AC_SUBST() calls for AC_PATH_PROG() results.

AC_PATH_PROG calls AC_SUBST for us.

unit tests: Run all tests through Valgrind's memcheck, if available.

collectd.spec: add new gRPC plugin

Disable it for now as the version of protobuf-compiler
in RHEL and Fedora is too old.

ceph_test: Free the yajl handle.

utils_mount_test: Free dynamically allocated strings.

testing: Evaluate expressions only once in EXPECT_EQ_STR().

utils_vl_lookup: Free regexen when destroying class lists.

Else, we'll leak any memory used by the regex.

Merge pull request #1738 from tokkee/sh/silent

src/Makefile: Support silent build rules for pod2man.

src/Makefile: Support silent build rules for pod2man.

Merge pull request #1699 from tokkee/sh/grpc

gRPC server plugin

gRPC doc: Fixed typos and mention C++11 requirement.

src/Makefile: Don't unnecessarily set plugin specific CXXFLAGS.

Also, drop unneeded -lphread / -ldl from grpc's libraries.