Auto-Merge pull request #3063 from octo/issue/3062

Automatically merged due to "Automerge" label

strjoin(): Fix behavior if output buffer is NULL.

Additionally, test the (NULL, 0) output buffer for all test cases rather
than an individual, isolated test case.

Fixes: #3062

Merge pull request #3059 from octo/sock_cloexec

write_prometheus plugin: Only use SOCK_CLOEXEC if it is defined.

write_prometheus plugin: Only use SOCK_CLOEXEC if it is defined.

Fixes compilation issue on Mac OS X. See also #3055.

Merge pull request #2988 from octo/ff/struct-init-syntax

log_logstash plugin: Fix non-portable struct initialization with "{}".

log_logstash plugin: Fix non-portable struct initialization with "{}".

Merge pull request #2975 from octo/issue/2954

zfs_arc: Ignore the first two lines of the statistics file on Linux (5.7 cherry-pick)

contrib/format.sh src/zfs_arc.c

zfs_arc: Ignore the first two lines of the statistics file on Linux.

The first two lines of the statistics file on Linux contain information about
the rest of the file. They do not contain any usable statistics.
See module/spl/spl-kstat.c of the spl module of ZFS on Linux for more
information.

Merge pull request #2962 from octo/utils_mount

src/utils_mount.c: Add code path for HAVE_GETMNTENT_R.

src/utils_mount.c: Add code path for HAVE_GETMNTENT_R.

I don't know when this got lost or why it is triggering now, but the
build fails on my machine without this now.

Auto-Merge pull request #2952 from octo/issue/2951

Automatically merged due to "Automerge" label

disk plugin: Support Linux kernels with additional fields.

Linux 4.19 will add additional counters, which caused this plugin to
bail out because it checked the number of fields exactly instead of
ensuring that there are *at least* the number of fields expected.

Fixes: #2951

powerdns plugin: no-packet-error counts number of erroneous packets; it should be returned as counter (like other query/hit/miss counters), not gauge

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

Auto-Merge pull request #2939 from octo/pr2937-backport

Automatically merged due to "Automerge" label

exec plugin: dynamically allocate grname buffer.

Backport of a fix in master.

Fixes: #2696

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

Automatically merged due to "Automerge" label

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

Closes: #2732

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"

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>

liboconfig: Coding style changes.

Mostly whitespace.

liboconfig: Improve error handling.

  * Use the YYERROR macro instead of exit(3).
  * Remove newlines from strings passed to yyerror().
  * Change return valud or yyerror() from int to void.

liboconfig: Check return values of calloc() and realloc().

CID: 38123

email plugin: Avoid strtok_r to tokenize input.

Coverity (wrongly) thinks that "type" might have been NULL (since its
first argument is non-NULL, this cannot happen). It has a point about
insufficient NULL-ness checks, though, as "tmp" may very well be NULL and
we're passing it to atoi() which will likely not take this well.

CID: 37988

table plugin: Add explicit NULL checks to clear functions.

CID: 37966

exec plugin: Check return value of plugin_thread_create().

CID: 37942, 37943

rrdtool plugin: Check return value of c_avl_get().

CID: 37940

Auto-Merge pull request #2589 from octo/cid/37960

Automatically merged due to "Automerge" label

Auto-Merge pull request #2588 from octo/cid/37968

Automatically merged due to "Automerge" label

contrib/format.sh src/perl.c

powerdns plugin: Remove the unused ret_buffer_size argument …

… from powerdns_get_data().

perl plugin: Fix exporting notification meta data.

Previously, the while loop ensured that meta was NULL, causing the for
loop to never been executed. This introduces a new temporary variable to
avoid changing meta before the for loop.

CID: 37960

powerdns plugin: Declare variables on first use.

powerdns plugin: Ensure powerdns_get_data() returns a non-NULL buffer.

Coverity is concerned that if buffer is NULL, strtok_r() is called with
two NULL arguments, causing a NULL pointer dereference.

CID: 37968

Auto-Merge pull request #2580 from octo/cid/179235

Automatically merged due to "Automerge" label

contrib/format.sh src/aggregation.c

aggregation plugin: Check return value of config functions.

CID: 179235

aggregation plugin: Declare variables on initialization.

Auto-Merge pull request #2575 from octo/cid/37934

Automatically merged due to "Automerge" label

ping plugin: Lock the mutex later.

Coverity gets confused if holding the lock while allocating pingobj.

CID: 37934

Auto-Merge pull request #2568 from octo/cid/157645

Automatically merged due to "Automerge" label

set target: Fix memory leak.

CID: 157645

Auto-Merge pull request #2565 from octo/cid/37966

Automatically merged due to "Automerge" label

table plugin: Refactor realloc() calls.

Increase the number of elements in the successful path only, instead of
increasing it immediately and decreasing on error.

table plugin: Assert that result_num is zero when result is NULL.

The Coverity check gets confused by the `tbl->results == NULL` check:
due to the check, is assumes that tbl->results *may* be NULL. It then
discovers the dereference in tbl_clear() without realizing that "result"
and "result_num" are linked and that situation can't happen. This
assertion makes this link a bit easier to conclude.

CID: 37966

Auto-Merge pull request #2563 from octo/cid/179225

Automatically merged due to "Automerge" label

Auto-Merge pull request #2561 from octo/cid/157643

Automatically merged due to "Automerge" label

Auto-Merge pull request #2560 from octo/cid/179237

Automatically merged due to "Automerge" label

Auto-Merge pull request #2559 from octo/cid/179236

Automatically merged due to "Automerge" label

gps plugin: Properly unlock mutex after waiting.

How did this ever work?!

CID: 158522

write_tsdb plugin: Unlock mutex before destroying it.

CID: 179224

write_graphite plugin: Unlock mutex before destroying it.

CID: 179225

set target: Ensure that int being cast to size_t is positive or zero.

CID: 157643

write_sensu plugin: Unlock mutex before destroying it.

CID: 179237

src/utils_format_json.c: Check return values of yajl_gen_* functions.

CID: 179228, 179236, 179239

Auto-Merge pull request #2554 from octo/cid/179227

Automatically merged due to "Automerge" label

write_riemann plugin: Unlock mutex before destroying it.

CID: 179227

Auto-Merge pull request #2547 from octo/cid/81390

Automatically merged due to "Automerge" label

gps plugin: Fix double unlock.

CID: 179241

snmp plugin: Ensure that index is within bounds.

CID: 81389, 81390

Auto-Merge pull request #2545 from octo/turbostat

Automatically merged due to "Automerge" label

contrib/format.sh src/msr-index.h src/turbostat.c

msr-index: include GPLv2 header

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

turbostat: import msr-index header from linux

Since Linux 4.12, the kernel does not expose msr-index.h to the userland
anymore. This commit pulls and hardcode the required definitions.

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

src/daemon/meta_data.c: Fix double unlock.

CID: 179245

Fix documentation typo for PostgreSQL plugin

In collectd/src/postgresql_default.conf the name for the backends
query is plural, yet in the documentation it is singular. This caused
me some lost time when running this plugin.

It is also wrongly listed on the wiki:

https://collectd.org/wiki/index.php/Plugin:PostgreSQL
Signed-off-by: Florian Forster <octo@collectd.org>

postgresql: fix last sum() related parse errors

These queries return empty values when a database doesn't have any
tables yet, which results in collectd logging this error every Interval:

```
db query utils: udb_result_submit: Parsing `' as derive failed.
```

Related to #1905

Auto-Merge pull request #2518 from octo/issue/2510

Automatically merged due to "Automerge" label

netlink plugin: Improve error reporting for mnl_attr_validate2() and mnl_socket_recvfrom().

Issue: #2510

Merge branch 'collectd-5.6' into collectd-5.7

[2475] write_prometheus plugin: Fix port listening

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

write_prometheus plugin: Implement logging callback.

memcached: Fix CPU usage reporting

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

statsd: Document collectd behaviour when receiving the 0.000 timer value

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

collectd-python(5): Fix documentation of Notification.dispatch().

Cherry-picked from PR #2135
Signed-off-by: Florian Forster <octo@collectd.org>

Merge branch 'collectd-5.6' into collectd-5.7

Bump version to 5.6.3; Update ChangeLog.

Merge remote-tracking branch 'github/pr/2466' into collectd-5.7

write_prometheus plugin: Don't construct struct sockaddr_in6 manually.

On FreeBSD, the struct is an incomplete type.

write_prometheus plugin: Add support for libmicrohttpd < 0.9.0.

0.9.0 was released in July 2010, yet Precise, which we're still building for,
ships 0.4.6. *sigh*

write_prometheus plugin: Fall back to IPv4 socket if IPv6 fails.

Unfortunately, for libmicrohttpd "dual stack" means "enforce IPv6 (but
allow IPv4, too)". That means that on systems without IPv6,
libmicrohttpd'd initialization fails (when the dual stack flag is set).
To work around this limitation we open the socket ourselves and pass it
to libmicrohttpd to use.

Fixes: #2149

Merge branch 'collectd-5.6' into collectd-5.7

dbi plugin: Fix additional memory leaks.

dbi plugin: Fix memory leak.

Some memory was leaked when shutting down.

This is a partial back-port of 5ee365a.

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

Merge remote-tracking branch 'github/pr/2451' into collectd-5.6

snmp plugin: Style fix

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

snmp plugin: Fix double free of request PDU

snmp_sess_synch_response() always frees request PDU, in both case of request
error and success. If error condition occurs inside of `while (status == 0)`
loop, double free of `req` happens.

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

email, exec and unixsock plugins: Use _SC_GETPW_R_SIZE_MAX.

Fixes: #2208

Merge branch 'collectd-5.6' into collectd-5.7

perl plugin: Improve error message.

Merge remote-tracking branch 'github/pr/2391' into collectd-5.6

Merge pull request #2417 from bluca/dpdk_pkgconfig_backport

configure.ac: dpdk: use pkg-config - 5.7

collectd.spec: avoid accidentally picking an ancient/unsupported java version

ntpd: Correct loop offset and error calculation

Stop the scaling calculation dropping the sign of the underlying value

Fixes: #2188