Merge branch 'collectd-5.5'

Merge branch 'collectd-5.4' into collectd-5.5

docs: mention regexps for the interface plugin

It is very useful to collect data (or ignore) from a group of interfaces
that are similarly named - especially if the list is dynamic and it's
not feasible to update configuration each time an interface shows up or
disappears. It's possible to use regexps to specify interfaces to
collect data from (or ignore), but it's not clearly stated anywhere in
the documentation.
This is an attempt to improve documentation in this area.

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

exec plugin: Revert file mode to 0644.

This fixes a regression from 94e562250296b1435e3e48cd302055d89c4c702b.

virt plugin: make indentation explicit

Fixes this warning raised by gcc-6's new "misleading-indentation"
option:

```
virt.c: In function 'lv_init':
virt.c:331:2: error: statement is indented as if it were guarded by... [-Werror=misleading-indentation]
  return 0;
  ^~~~~~

virt.c:328:5: note: ...this 'if' clause, but it is not
     if (virInitialize () != 0)
     ^~
```

df: remove legacy references to ReportReserved

The ReportReserved option was removed in fcb3cbf. This patch cleans up a
couple of references which got left behind.

Merge branch 'collectd-5.5'

Merge branch 'collectd-5.4'

Merge branch 'collectd-5.4' into collectd-5.5

write_redis: don't return with lock held

src/utils_latency.c: Fix format string error.

src/utils_latency.c: Ensure that LLONG_MAX is defined.

EPEL5 doesn't have it in <limits.h> apparently.

src/utils_latency.c: Store "bin_width" as cdtime_t.

This solves the integer overflow when passing huge latency values to
latency_counter_add(). In addition to fixing the overflow the function
will now ignore values that are larger than LLONG_MAX, i.e. the longest
possible latency is 272 years. As a nice side-effect, the precission of
latency_counter_get_percentile() is improved.

Issue: #1131

src/utils_latency.c: Fix change_bin_width().

A missing cast when calculating "width_change_ratio" caused all metrics to
be moved to bucket #0.

varnish: fix leak on read

Since VSM_Close doesn't free the object we leak a few bytes
every interval

openvpn plugin: Fix "Dead assignment" warning.

modbus plugin: Make sure variable is initialized.

This was introduced by the merge
(09c6a320f3cb36b5dbb2c2ce43858f33be7acf9b).

tail plugin: Make sure variable is initialized.

This was introduced by the merge
(09c6a320f3cb36b5dbb2c2ce43858f33be7acf9b).

Fix header condition

The dependency is either UTMPX or UTMP or Statgrab. The inclusion above hasn't statgrab.h in the elif-chain, that means compilation fails when only libstatgrab is available. The line can safely be taken out as the same condition is checked in #L120.

Fixed rrdtool sigsegv (datadir==NULL ptr dereferencing)

processes: correct a typo

Merge branch 'collectd-5.5'

Merge branch 'collectd-5.4' into collectd-5.5

write_redis: don't return with lock held

redis plugin: make sure buffer is null-terminated

CID 37938

perl: unlock mutex on error

CID 37970

openvpn: free resource on error

openvpn: fix error message

openvpn: check return value of malloc

CID 37987

curl_json: remove dead code

CID 37963

snmp plugin: don't read from uninitialized pointer

if the first snmp_pdu_create inside the loop returns an error,
we exit the loop with res uninitialized and then call snmp_free_pdu on it.

CID #38037

libcollectdclient: don't close socket twice

CID #38039

email plugin: fix check for result of malloc

CID #38010

Fixed rrdtool sigsegv (datadir==NULL ptr dereferencing)

openldap plugin: fix build on OSX El Capitan

OpenLDAP is deprecated in OSX El Capitan.
Apple would like us to use the OpenDirectory Framework instead.

Treat these deprecation warnings as warnings, not errors.

collectd.conf.pod: mention IPv6 option to the iptables plugin

Thanks to @michaelsalmon for pointing this out !

Fix #1496

remove the addressof in the sizeof

quick fix for issue 1501?
the new warning for pointer access introduced in gcc 4.8 helped to find
this one.

disk plugin: Declare alt_name only when libudev is available.

Coverity whines about "output_name = alt_name" being dead code because
it sees alt_name as beeing NULL always.

CID: 48417

network plugin: Coding style fixes.

This fixes a number of sizeof() operations which did not follow our best
practice of using the variable rather than its type. It also removes
unnecessary (void *) casts.

network plugin: Check range of pkg_numval.

This is essentially the same as the check using exp_size a couple lines
earlier, but that appears to be too indirect for Coverity to understand.
This hopefully fixes the resulting "tainted_data_return" errors.

CID: 48420

Merge branch 'collectd-5.5'

Merge branch 'collectd-5.4' into collectd-5.5

perl plugin: Check "return value" of PERL_GET_CONTEXT.

CID: 37985

email plugin: Don't call type_list_incr() with type == NULL.

CID: 37988

write_http plugin: Check return status of cf_util_get_*().

CID: 37949

gmond plugin: Inline the plugin_dispatch_values() call.

Since version 5.3, a write queue is used. This means that the
plugin_dispatch_values() call will create a copy of the value list and
return quickly. Creating yet another copy here is therefore not as useful
anymore.

CID: 38114

gmond plugin: Check return value of sendto(2).

CID: 38113

gmond plugin: Check return value of setsockopt(2).

CID: 38112

zfs_arc plugin: Rewrite Linux parsing logic.

CID: 38028

Merge branch 'collectd-5.5'

Merge branch 'collectd-5.4' into collectd-5.5

zfs_arc plugin: Plug memory leak on Linux.

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

configure.ac: warn when pkg-config is not installed

It hasn't been optional for a while now.

Fixes:
checking for rrd.h... no
./configure: 25157: Syntax error: word unexpected (expecting ")")

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

barometer plugin: cleanup whitespace

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

collectd.conf(5): Document all metrics created by CollectInternalStats.

collectd.conf(5): Improve documentation of the CollectdInternalStats option.

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

Merge branch 'pr/1469'

Merge branch 'pr/1393'

drbd plugin: no need to link with libpthread

Merge pull request #1466 from rubenk/write_riemann-remove-unneeded-include

write_riemann: remove unneeded include

Merge pull request #1463 from rubenk/redis-fix-build-warning

redis plugin: fix build error on NetBSD

write_riemann: remove unneeded include

Fixes build on NetBSD:

Making all in daemon
  CC       write_riemann_threshold.lo
write_riemann_threshold.c:36:18: fatal error: ltdl.h: No such file or directory
 #include <ltdl.h>
                  ^
compilation terminated.
*** Error code 1

Stop.
make[3]: stopped in /home/ruben/src/collectd/src
*** Error code 1

Stop.
make[2]: stopped in /home/ruben/src/collectd/src
*** Error code 1

Stop.
make[1]: stopped in /home/ruben/src/collectd/src
*** Error code 1

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

redis plugin: fix build error on NetBSD

Making all in daemon
  CC       redis_la-redis.lo
redis.c: In function 'redis_handle_info':
redis.c:307:5: error: array subscript has type 'char' [-Werror=char-subscripts]
     for(i=0;(*str && (isdigit(*str) || *str == '.'));i++,str++)
     ^
cc1: all warnings being treated as errors
*** Error code 1

Stop.
make[3]: stopped in /home/ruben/src/collectd/src
*** Error code 1

Stop.
make[2]: stopped in /home/ruben/src/collectd/src
*** Error code 1

Stop.
make[1]: stopped in /home/ruben/src/collectd/src
*** Error code 1

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

Merge pull request #1462 from rubenk/email-plugin-fix-malloc-check

email plugin: fix check for result of malloc

email plugin: fix check for result of malloc

CID #38010

Merge pull request #1461 from rubenk/fix-java-plugin-on-freebsd

java: link with libpthread on FreeBSD

java: link with libpthread on FreeBSD

Merge branch 'collectd-5.5'

Merge branch 'collectd-5.4' into collectd-5.5

Merge pull request #1460 from rubenk/mark-lvm2app-linux-only

lvm2 plugin: mark as Linux only

Merge pull request #1459 from rubenk/remove-unneccesary-check-for-ip-compat-h

Remove check for ip_compat.h

Merge pull request #1458 from rubenk/pthread-detection

Pthread detection

Merge pull request #1457 from rubenk/java-plugin-fixups

Java plugin fixups

Merge pull request #1455 from rubenk/configure-ac-remove-dead-code

configure.ac: remove commented out check

Merge pull request #1454 from rubenk/libi2c-dev-mark-as-linux-only

libi2c only exists on Linux

Merge pull request #1453 from rubenk/configure-dont-output-pkgconfig-errors

libnotify: Don't output multiple lines of errors

Merge pull request #1452 from rubenk/configure-silence-configure-check

configure.ac: send output of check to /dev/null

Merge pull request #1442 from rubenk/omnios-fix-make-check

Fix make check on OmniOS

lvm2 plugin: mark as Linux only

Merge pull request #1456 from rubenk/onewire-fixup-flag-handling

onewire plugin: fixup flag handling

Remove check for ip_compat.h

Solaris needed this at one point but since
a621ced we check for ip6_ext without including this
header and that seems to succeed to.

Clarify error message

Stop hardcoding -lpthread

Move check for libpthread up

So we can use the result in plugins later on.

Java plugin: allow setting JAVA_LIBS as well

Java plugin: remove unused variables

Java plugin: clarify error message

It is possible that we've found libjvm, but it doesn't have
the right function.

onewire plugin: fixup flag handling

Split flags in _LDFLAGS and _LIBS
Allows the plugin to be detected and to be build on FreeBSD

configure.ac: remove commented out check

libi2c only exists on Linux

libnotify: Don't output multiple lines of errors

The errors pkg-config returns are not very helpful and they
break our nicely formatted list of found and absent libraries.

Just say that the package can't be found, like we do for the rest
of the packages we need.

configure.ac: send output of check to /dev/null

The output is quite a long line:
kern.cp_times: 10336 0 9848 499639 7613098 18834 0 11963 115 8101970 16444 0 14368 1672 8100398 17584 0 14149 1358 8099791

And we have no use for the result, we just want to know if the sysctl exists.

src/utils_ignorelist.c: Refactoring / code style fixes.

One of the goals was to remove uses of smalloc().

swap plugin: don't exit when malloc fails

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

src/daemon/utils_time.c: Rewrite format_zone() using strftime().

The "extern long timezone", while specified by X/Open, is not available
on FreeBSD. There, timezone() is a function.

.gitignore: Ignore all files starting with "test_".

Merge branch 'collectd-5.5'

Merge branch 'collectd-5.4' into collectd-5.5

curl_json plugin: Rewrite logic to add a path to db->tree.

CID: 37991

curl_xml plugin: Correctly free xpath object.

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