filecount plugin: mark function as static

Fixes a build warning:

CC       src/filecount.lo
src/filecount.c:66:6: warning: no previous prototype for function 'fc_free_dir' [-Wmissing-prototypes]
void fc_free_dir(fc_directory_conf_t *dir) {
     ^
1 warning generated.

Merge branch 'master' into endian

Include config.h early or _FILE_OFFSET_BITS is wrongly defined and bails out on Solaris 32 bit

Merge pull request #2716 from dago/kstat

Include kstat.h when available

Merge pull request #2688 from dstathis/master

change HAVE_UDEV_H to HAVE_LIBUDEV_H for bug 2651

Merge pull request #2731 from elfiesmelfie/ovs_fixes

Ovs Fixes

Merge remote-tracking branch 'origin/collectd-5.8'

Auto-Merge pull request #2736 from rpv-tomsk/collectd-collectd-5.8

Automatically merged due to "Automerge" label

uuid plugin: Fixed hostname setting

Closes: #2723
References: #2467

Merge pull request #2722 from elfiesmelfie/increase_buffer

dpdk: increase buffer size for parsing lcores

Merge pull request #2721 from elfiesmelfie/update_README

README: Include compiler defenses suggestion

Merge pull request #2703 from elfiesmelfie/fix_ipmi_bugfix_0.2

Fixes for ipmi plugin

Merge pull request #2704 from dhrupadb/patch-1

Properly cleanup dropped MySQL connections.

Merge pull request #2735 from rpv-tomsk/collectd-master

Tree wide: Replace sstrerror() with STRERRNO.

Auto-Merge pull request #2734 from rpv-tomsk/fix-2732

Automatically merged due to "Automerge" label

Tree wide: Replace sstrerror() with STRERRNO.

References: #2519

perl plugin: Fix compile fail with message "void function cannot return value"

Closes: #2732

utils_ovs: fix resource leaks

In ovs_db_init(), upon failure of either ovs_db_event_thread_init()
and/or ovs_db_poll_thread_init(), ovs_db_destroy() is invoked, in
order to release OVSDB data resources.

However, in the event that ovs_db_destroy() returns an error value,
some OVSDB resources are never released, and are consequently lost.

Resolve this issue by checking the return value of ovs_db_destroy()
and manually releasing the appropriate resources accordingly.

Fixes: 5a90d0e ("ovs_events: Address PR comments")
Signed-off-by: Kavanagh, Mark B <mark.b.kavanagh@intel.com>

utils_ovs: fix potential NULL

In ovs_utils_get_map_value(), YAJL_GET_ARRAY may return a NULL
pointer, which is subsequently dereferenced. Ensure that the
pointer is non-NULL before proceeding.

Fixes: 12f6497 ("ovs_events: Add external ids as metadata")
Signed-off-by: Mark Kavanagh <mark.b.kavanagh@intel.com>

utils_ovs: fix potential NULL-string dereference

In ovs_utils_get_map_value(), a potential NULL-string, returned
by YAJL_GET_STRING(), is passed to strcmp, and subsequently
dereferenced. Ensure that said string is non-NULL.

Fixes: cb59d85 ("ovs_events: Fix plugin collectd config file")
Signed-off-by: Mark Kavanagh <mark.b.kavanagh@intel.com>

ovs_stats: fix potential NULL array dereference

In ovs_stats_update_bridge(), YAJL_GET_ARRAY() may return a NULL
pointer. Ensure that a non-NULL pointer is returned, before
attempting to dererence same.

Fixes: 4256753 ("ovs_events: fix scan-build warnings #2205")
Signed-off-by: Mark Kavanagh <mark.b.kavanagh@intel.com>

utils_ovs: fix potential NULL dereference

In ovs_utils_get_map_value(), YAJL_GET_ARRAY may return NULL,
which is subsequently dereferenced.

Ensure that the returned pointer is non-NULL before attempting
to dereference it.

Fixes: 12f6497 ("ovs_events: Add external ids as metadata")
Signed-off-by: Mark Kavanagh <mark.b.kavanagh@intel.com>

ovs_stats: fix memory leak

In ovs_stats_plugin_config(), memory may be allocated from the heap,
but is not subsequently freed in the case of failure later in the
function.

Fixes: bcf9c48 ("ovs_stats plugin: Fix a memory leak.")
Signed-off-by: Mark Kavanagh <mark.b.kavanagh@intel.com>

ovs_stats: fix potential NULL-string dereference

The YAJL_GET_STRING macro may return NULL; ensure that the returned
string is non-NULL, before passing it to ovs_stats_new_port(), within
the context of the ovs_stats_update_bridge() function.

Since this change introduces another return path, a single 'failure'
return point is introduced to ovs_stats_bridge().

Signed-off-by: Mark Kavanagh <mark.b.kavanagh@intel.com>

utils_ovs: fix potential strcmp() NULL ptr param

YAJL_GET_STRING(array_values[0]) may return NULL if
array_values[0]->string is NULL; passing a NULL parameter to
strcmp() is not recommended, since the resultant behaviour may
be undefined.

Resolve the issue by adding a NULL check for the string returned
by YAJL_GET_STRING.

Fixes: cbq59d853 ("ovs_events: Fix plugin collectd config file")
Signed-off-by: Mark Kavanagh <mark.b.kavanagh@intel.com>

ovs_stats: fix potential NULL dereference

ovs_stats_update_iface() passes a pointer to a potentially-NULL
string to sstrncpy(); this is obviously problematic.

Add a check to ensure that the relevant string is non-NULL.

Fixes: 1cc7599 ("ovs_stats plugin: Fix null dereference of "port".)
Signed-off-by: Mark Kavanagh <mark.b.kavanagh@intel.com>

ovs_stats: fix potential NULL dereference

ovs_stats_new_port() accepts a character pointer, uuid, as a parameter,
and copies it into port->uuid. Later, this value is dereferenced in
ovs_stats_update_bridge(). If uuid was NULL, then a SEGV will occur.

Resolve this issue by checking if uuid is NULL in ovs_stats_new_port().

Fixes: 481984e ("ovs_stats: Implement OVS statistics plugin.")
Signed-off-by: Mark Kavanagh <mark.b.kavanagh@intel.com>

ovs_events: fix potential NULL pointer dereference

A potentially-NULL pointer to ovs_events_iface_info_t is defererenced
by an invocation of sizeof(), within ovs_events_get_iface_info().

Move the offending line, such that it is executed after the
NULL-pointer check.

Fixes: f6adec45 ("ovs_events: Fix notification metadata garbage.")
Signed-off-by: Mark Kavanagh <mark.b.kavanagh@intel.com>

dpdk: fix utils_dpdk.c clang issue

This commit resolves the clang issue in src/utils_dpdk.c

Signed-off-by: Kevin Laatz <kevin.laatz@intel.com>

Keep clang-format happy

Add fallback for endianess conversion

Include kstat.h when available

README: Include compiler defenses suggestion

This commit adds text in the README to advise users of the DPDK plugins to
use compiler defenses such as -fstack-protector.

Signed-off-by: Kevin Laatz <kevin.laatz@intel.com>

dpdk: increase buffer size for parsing lcores

This commit increases the size of the "high_str" buffer,
which is later used by the strncpy() function. Static analysis
showed that there was a potential issue in accessing this string
if the buffer is smaller.

Signed-off-by: Harry van Haaren <harry.van.haaren@intel.com>

Docs: use "its" vice "it's" where appropriate

Merge pull request #2711 from dago/kstat

Include kstat.h if available to provide kstat_ctl_t

virt: Fix unit tests build with libvirt versions < 1.3

Previously too much code was disabled in ifdefs, and during
compilation with old libvirt (version < 1.3.x) warning was shown
because of unused test variables.

Change-Id: I631c9446add4ccef068a3c4585f8750a5930f406
Signed-off-by: Radoslaw Jablonski <radoslawx.jablonski@intel.com>

plugin_mock: Fix code formatting after virt-plugin related changes

Change-Id: I759724b3b6ae9e2f62a66f82e59ca4ca29cc1db9
Signed-off-by: Radoslaw Jablonski <radoslawx.jablonski@intel.com>

src/virt.c: Enable support for libvirt-1.3.1

Several macros are added to the definition of map_domain_event_detail_to_reason.

One unnecessary #ifdef macro is removed.

Change-Id: I5357816caa11009fbfee076524e03cdf2eb06109
Signed-off-by: Antoine Naud <antoinex.naud@intel.com>

Fix suggest by test clang-format

Include kstat.h if available to provide kstat_ctl_t

Revert "ipmi plugin: Ignore events from ignored sensors"

This reverts commit cd660c5b07b6f09d165435f240ca17d118a1e452.

Revert "ipmi plugin: Remove duplicate/unused code and unneeded brackets."

This reverts commit 8c9d64bf7615875b5030127b4df68a620e084f0d.

grpc plugin: Fixed typo in an error message.

ipmi plugin: Remove duplicate/unused code and unneeded brackets.

Change-Id: Ia4298f75f59e697df25d05091adbb556b64869e1
Signed-off-by: Mariusz Szafranski <mariuszx.szafranski@intel.com>

Properly cleanup dropped MySQL connections.

ipmi plugin: Threshold event handler registration fix

This patch fixes registration of threshold event handler for
specific sensor types (e.g. 'System Airflow')

Change-Id: I437f3fe68fecf264adb32a4e2ef4ac214a7e87a4
Signed-off-by: Mariusz Szafranski <mariuszx.szafranski@intel.com>

ipmi plugin: Ignore events from ignored sensors

Reading is disabled for ignored sensors. This fix it to also
ignore events from s4ensors on ignore list.

Change-Id: I705593c79274a00456a786698452cd3c1bae9872
Signed-off-by: Mariusz Szafranski <mariuszx.szafranski@intel.com>

ipmi plugin: Fix crash in ipmi.so (interval < 0.5)

Fixes crash (divide by 0) in ipmi.so when plugin related interval
set to 0.45 or less.

Change-Id: I3ac2282efdc005edefe92f870ac30417451c202c
Signed-off-by: Mariusz Szafranski <mariuszx.szafranski@intel.com>

src/virt.c: Fix security issue detected by klocwork related to strncat

The fix consist in replacing all strncat invocations
with safer macro SSTRNCAT.

Change-Id: I4faee764762f8881d7f7539db6359685b910da44
Signed-off-by: Antoine Naud <antoinex.naud@intel.com>

virt: Apply formatting rules on plugin code

Code formatted with contrib/format.sh script.

Change-Id: I49ad8bc48af11a09779027df7855e042e8d9de37
Signed-off-by: Radoslaw Jablonski <radoslawx.jablonski@intel.com>

virt: Fix build error

Change-Id: I6b1111aded6391775a40911fdede288b3f0380d9
Signed-off-by: Mozejko, MarcinX <marcinx.mozejko@intel.com>

virt: Add domain state metrics dispatch

Previously information about Virtual Machine domains state
(e.g. VIR_DOMAIN_RUNNING, VIR_DOMAIN_PAUSED..) was not
dispatched (e.g. file with domain states was not created by
CSV plugin).
Now current VM state information with corresponding reason is
dispatched as ‘domain_state’.

Change-Id: I781f11f53adb9ca0abcf6565d5717c6495f853b7
Signed-off-by: Mozejko, MarcinX <marcinx.mozejko@intel.com>

build.sh: fix a few cosmetic issues

build.sh: use consistent indentation

build.sh: look in m4 dir for macro's

This fixes #2674

ping plugin: support specifying the address family

This closes #2478.

Merge branch 'collectd-5.8'

Merge branch 'collectd-5.7' into collectd-5.8

src/virt.c: State notifications for all persistent domains

When virt option PersistentNotification is enabled, the notifications of VMs
states are sent only for running VMs (whereas they should be sent for all
persistent VMs, including running, suspended, shut off VMs). This behaviour is
due to the fact that VM states are notified while fetching all other metrics,
which is executed only for running VMs.

This commit fixes this issue by decoupling the notifications of VMs states from
fetching and dispatching metrics values. In this purpose, a new function named
persistent_domains_state_notification is added to fetch the states (and reasons)
of persistent VMs and notify them. This function is called at each read interval
before fetching and dispatching the metrics from running VMs.

Unit Tests for the new functions are implemented.

Change-Id: Ic2befb3e7826184696344b037916bc603ec01341
Signed-off-by: Antoine Naud <antoinex.naud@intel.com>

src/virt.c: add lifecycle transition notifications

Notifications about VM state are received on every read interval, so the same
notification about specific VM state is received several times within a minute,
which lead to growing log files. This issue is fixed by generating notifications
only for VM state changes (denoted also here lifecycle transitions).

This commit adds a binary option PersistentNotification to virt plugin. When
this option is set to false (default value), only VM state changes are notified
as soon as they occur. When this option is set to true, VM states are notified
on every read interval.

Change-Id: I46ebe2d2f1b72432e7b458530c4cc0eac3e26c6a
Signed-off-by: Antoine Naud <antoinex.naud@intel.com>

Merge pull request #2692 from shastah/fix-readiness-spelling

s/readyness/readiness/

s/readyness/readiness/

Signed-off-by: Jakub Jankowski <shasta@toxcorp.com>

spec: fix mbmon/mcelog typo

Fix a typo in collectd.spec which manifested when building with
with_mcelog set to false, invalidating with_mbmon setting.

Signed-off-by: Jakub Jankowski <shasta@toxcorp.com>

change HAVE_UDEV_H to HAVE_LIBUDEV_H for bug 2651

Merge pull request #2687 from jeis2497052/master

Propose fix some typos

Propose fix some typos

Merge pull request #2676 from elfiesmelfie/fix_intel_pmu_crash

intel_pmu: fix for possible null pointer dereference

intel_pmu: fix for possible null pointer dereference

Null pointer dereference could occur for invalid first hardware event.
In pmu_add_hw_events if the first event was not resolved, on the second
iteration the eventlist_last could be dereferenced to mark previous
event as group leader.

Change-Id: Ic2a2a3572d6835ca892d71f3e66812f356592d9e
Signed-off-by: Kamil Wiatrowski <kamilx.wiatrowski@intel.com>

Merge pull request #2684 from rpv-tomsk/fix-2673

write_prometheus: Set SO_REUSEADDR on listening socket

Merge pull request #2613 from elfiesmelfie/update_dpdk_note

docs: update dpdkstats documentation

write_prometheus: Set SO_REUSEADDR on listening socket

Otherwise Collectd fails to bind the socket after restart.

Thanks to Richard Weinberger and Brandon Hume for reporting.

Closes: #2570
Closes: #2673

Auto-Merge pull request #2683 from rpv-tomsk/fix-2607

Automatically merged due to "Automerge" label

write_kafka: 'rd_kafka_errno2err' is deprecated, fix failed builds

Since librdkafka-0.11.0 applications should use rd_kafka_last_error()
to retrieve the error code from the legacy APIs.

Thanks to hikdo.

Closes: #2607
Closes: #2628
Closes: #2640

Update BUILD.dpdkstat.md

fixed trailing word "Version"

Merge pull request #2660 from pbostley/master

Adding support for CDAB endian 32-bit modbus polls

Adding documentation for the Modbus little endian modes where 32 bit values have thier registers swapped

Adding support for CDAB endian 32-bit modbus polls

Auto-Merge pull request #2629 from rpv-tomsk/fix-2626

Automatically merged due to "Automerge" label

ipmi plugin: Fix broken 'Sensor' configuration option

Merge pull request #2512 from Stackdriver/pri

Replace zu with PRIsz and llu with PRIu64.

Merge branch 'collectd-5.8'

Auto-Merge pull request #2611 from rpv-tomsk/fix-2587

Automatically merged due to "Automerge" label

tail plugin, latency: Fix regressions after #2535

 - Remove two fixed decimal places from type instance of percentiles
 - Recover accuracy of bounds in type instance of buckets
 + Allow (compared to state before #2535) to report percentiles with fractional values
 * Changed maximum length of reported type and type instance to 50 chars

References: #2535
Closes: #2587

docs: dpdk extended the initial note

docs: update dpdkstats documentation

Update the dpdkstats documentation to state that it shouldn't be used with
OVS-with-DPDK. The Open vSwitch plugins is what should be used.

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

Auto-Merge pull request #2612 from rpv-tomsk/collectd-5.8-memcached

Automatically merged due to "Automerge" label

memcached: Check for closed connection to avoid deadloop

Check for 'recv(...) == 0' condition was missing.

Auto-Merge pull request #2610 from rpv-tomsk/fix-2609

Automatically merged due to "Automerge" label

processes plugin: Fix compilation when ps_delay() not used.

Remove unused ps_delay() function when compiled without libtaskstats/libmnl

Closes: #2609

Merge pull request #2598 from octo/ff/delayacct

processes plugin: Add support for Linux Delay Accounting.

processes plugin: Use the new "delay_rate" type for Delay Accounting.

processes plugin: Add the "delay-" prefix to type instances.

This fixes a regression introduced in
17b81d4fa4eabbb7c029ccf424f43e4d89c27e35.

processes plugin: Fix error message.

processes plugin: Make delay metric reporting less repetitive.

contrib/systemd.collectd.service: Add the processes plugin.

processes plugin: Check for the CAP_NET_ADMIN capability.

This allows us to print helpful error messages to the user if something
goes wrong.

Auto-Merge pull request #2605 from maryamtahhan/feat_turbostat_report_skl

Automatically merged due to "Automerge" label

turbostat: Add support of reporting GPU power on SKL in turbostat

Change-Id: Icb57cf5ab116069b1e4d9580e248af1207e3282b

df: fix memory leak in error case

Because the memory of the STATANYFS list is not released
when the read callback function error occurs,
a memory leak may occur.
With this change, the memory is always released
by not returning in the loop.

Signed-off-by: Florian Forster <octo@collectd.org>