Merge pull request #2590 from maryamtahhan/fix_collectd_ovs_deadlock

Fix collectd ovs deadlock

ceph plugin: Fix #2572

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

Merge branch 'collectd-5.7' into collectd-5.8

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

utils_ovs: fix #2574

Poll thread was trying to reconnect after event thread was terminated.

Change-Id: I48bf196302139ddf99bb96b49a6c3b4ede345240
Signed-off-by: Mytnyk, Volodymyr <volodymyrx.mytnyk@intel.com>

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

Merge branch 'collectd-5.7' into collectd-5.8

Auto-Merge pull request #2586 from octo/issue/2583

Automatically merged due to "Automerge" label

network_parse_test.c: Only test security features if libgcrypt is available.

Fixes: #2583

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

Automatically merged due to "Automerge" label

Auto-Merge pull request #2579 from octo/cid/179238

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.

virt plugin: Remove `do {} while(0)` loop around macro.

The `continue` in the macro was supposed to continue the `for` loop, but
was (unexpectedly) exiting the do…while loop instead.

CID: 179238

Auto-Merge pull request #2576 from octo/collectd-5.8

Automatically merged due to "Automerge" label

contrib/format.sh: Switch to custom domain name.

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 #2566 from octo/pr/2556

Automatically merged due to "Automerge" label

Merge branch 'collectd-5.7' into collectd-5.8

rrd plugin: Fix writing of long values.

The buffer provided for value_list_to_string() was not long enough to
fit a timestamp and a negative value using exponent and maximum
precision of the gauge format.

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

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 #2562 from octo/cid/179233

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

src/utils_ovs.c: Initialize pdb->sock to -1 earlier.

ovs_db_destroy() was called before the field was initialized, leading to
close(0) being called.

src/utils_ovs.c: Assign variable only once.

CID: 179233

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

Update version to 5.8.0.

Merge branch 'collectd-5.7' into collectd-5.8

Auto-Merge pull request #2555 from octo/cid/179231

Automatically merged due to "Automerge" label

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

Automatically merged due to "Automerge" label

Auto-Merge pull request #2553 from octo/cid/179230

Automatically merged due to "Automerge" label

ovs_stats plugin: Fix null dereference of "port".

This also refactors the ovs_stats_update_iface_ext_ids() function to
handle errors, instead of many nested if blocks.

CID: 179231

write_riemann plugin: Unlock mutex before destroying it.

CID: 179227

processes plugin: Fix check for context switch counters.

CID: 179230

Auto-Merge pull request #2550 from octo/cid/179234

Automatically merged due to "Automerge" label

Auto-Merge pull request #2549 from octo/cid/179240

Automatically merged due to "Automerge" label

Auto-Merge pull request #2548 from octo/cid/42796

Automatically merged due to "Automerge" label

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

Automatically merged due to "Automerge" label

ovs_stats plugin: Minor code cleanup.

ovs_stats plugin: Add missing unlock.

CID: 179234

src/utils_ovs.c: Random coding style improvements.

src/utils_ovs.c: Don't set pthread_t to -1, use pthread_equal() to compare.

The assumption that pthread_t is a scalar is not portable.

src/utils_ovs.c: Unlock pdb->mutex before calling ovs_db_callback_remove_all().

CID: 179240

gps plugin: Fix double unlock.

CID: 179241

zfs_arc plugin: Allocate ksp later to avoid memory leak.

CID: 42796

Auto-Merge pull request #2546 from octo/cid/179229

Automatically merged due to "Automerge" label

snmp plugin: Ensure that index is within bounds.

CID: 81389, 81390

ovs_stats plugin: Fix a memory leak.

br_name would leak when jumping to cleanup_fail.

CID: 179229

Auto-Merge pull request #2540 from octo/cid/141009

Automatically merged due to "Automerge" label

Auto-Merge pull request #2539 from octo/cid/179243

Automatically merged due to "Automerge" label

Auto-Merge pull request #2538 from octo/cid/179244

Automatically merged due to "Automerge" label

libcollectdclient: Ensure vl->values_len is non-zero.

That size_t is then used to declare Variable Length Arrays. Hopefully
this helps to convince Coverity that pkg_values and pkg_values_types are
actually fully initialized.

CID: 141009, 141010

ovs_events plugin: Replace strncpy() with sstrncpy().

CID: 179243

snmp_agent plugin: Fix memory leak.

Allocate "entry" later so that the error handling blocks don't leak it.

CID: 179244

Merge branch 'collectd-5.7' into collectd-5.8

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>

Auto-Merge pull request #2535 from mfournier/collectd-5.8

Automatically merged due to "Automerge" label

mcelog: filter through contrib/format.sh

Merge branch 'collectd-5.7' into collectd-5.8

Fixed code styling

Signed-off-by: Taras Chornyi <tarasx.chornyi@intel.com>

Fixed gcc 7 format-truncation errors #2423

Signed-off-by: Taras Chornyi <tarasx.chornyi@intel.com>

Auto-Merge pull request #2517 from trustchk/aix

Automatically merged due to "Automerge" label

libcollectdclient/network_parse.c: Implement be{16,64}toh() if not provided by the system.

Auto-Merge pull request #2527 from mfournier/varnish_52_compat_on_collectd_58

Automatically merged due to "Automerge" label

varnish plugin: Code cleanup:

*   Remove legacy rindex() with the POSIX strrchr().
*   Check return value of strrchr().
*   Chose non-arbitrary buffer size.
*   Use sstrncpy() instead of strcpy().

varnish plugin: Avoid unused variable even harder.

;-)

varnish plugin: avoid unused variable in V5 block

varnish plugin: fix build against pre varnish 5.2

Older Varnish versions don't know about VSM_Destroy()

running contrib/format.sh on src/varnish.c

varnish-cache-5.2 compatibility

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 #2522 from mfournier/fix-collectd-5.8-build

Automatically merged due to "Automerge" label

Merge branch 'collectd-5.7' into collectd-5.8

Conflicts:
src/netlink.c