modbus plugin: fix 2 leaks on error paths

Merge pull request #1380 from rubenk/csv-plug-leak-on-error-path

Csv plug leak on error path

Merge pull request #1379 from rubenk/ceph-close-fd-on-error

ceph plugin: close fd on error

Merge pull request #1378 from rubenk/bind-dont-free-invalid-pointer

bind: tmp doesn't point to start of allocated memory

Merge pull request #1377 from rubenk/mqtt-fix-out-of-bounds-write

mqtt: fix an out-of-bounds write

Merge pull request #1376 from rubenk/amqp-fix-buffer-size

amqp: use the right buffer size for sstrncpy

Merge pull request #1375 from rubenk/redis-unlock-lock

write_redis: don't return with lock held

Merge pull request #1374 from rubenk/powerdns-fix-array-initialization

powerdns plugin: fix array initialization

csv plugin: remove trailing whitespace

csv plugin: plug memory leak on error

CID 38015

ceph plugin: close fd on error

bind: tmp doesn't point to start of allocated memory

mqtt: fix an out-of-bounds write

CID 37990

amqp: use the right buffer size for sstrncpy

buffer is a char *, not an array, so size_of returns the size
of the pointer.

write_redis: don't return with lock held

powerdns plugin: fix array initialization

Merge pull request #1371 from rubenk/mqtt-plug-leak

mqtt: plug leak on error path

Fix build with older version of libprotobuf-c

Revert "Set bugreport url and homepage url"

The net-snmp public headers are generated by autoconf
so they also define PACKAGE_URL (and lots more).

Apparently this is completely normal and documented at
http://www.net-snmp.org/tutorial/tutorial-5/toolkit/autoconf/index.html

This reverts commit e16e805eae42f911a6f9ab81420fcaf1a957810f.

Merge pull request #1199 from rubenk/remove-gcrypt-exec-prefix-check

Remove gcrypt exec prefix check

Merge pull request #1210 from rubenk/protoc

Protoc

mqtt: plug leak on error path

Merge pull request #958 from rubenk/fix-version-gen-sh

Only run git describe when we're in a git repo

Only run git describe when we're in a git repo

This fixes an issue I see when extracting a collectd
tarball into another git repo (the Fedora collectd rpm one).
In that case version-gen.sh picked up the last tag from the Fedora
repo, which is the wrong one.

Merge pull request #1219 from rubenk/version-gen

Some more version-gen.sh tweaks

Merge pull request #1220 from rubenk/set-package-bugreport-and-homepage

Set bugreport url and homepage url

protoc: check for headers in new location as well

Newer versions of protobuf-c have their headers in
/usr/include/protobuf-c, and a symlink from
/usr/include/google/protobuf-c/.

These symlinks will eventually disappear however, so be prepared.

protoc: check for libprotobuf-c as well

protoc: simplify configure check

protoc: slightly nicer output in silent mode

make[1]: Entering directory '/home/ruben/src/collectd/src'
  PROTOC-C     pinba.pb-c.c
  PROTOC-C     riemann.pb-c.c

protoc: log when compiler is missing

In the case where we do have the required headers
but don't have the compiler, this makes it more clear
that we're missing the latter.

Silence protoc-c in silence mode

Instead of showing the full command line, this shows:
  GEN      pinba.pb-c.c
  GEN      riemann.pb-c.c

write_riemann: only compile .proto files when enabled

If the write_riemann plugin is not enabled there's no
need to compile the protobuf files.

Pinba: only compile .proto files when enabled

If the pinba plugin is not enabled there's no
need to compile the protobuf files.

write_kafka: fix indentation

It was all over the place.
I went with 4 spaces since that's what the majority
of code used.

the swrite interface is 'non-zero'

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

Merge remote-tracking branch 'github/pr/1366'

snmp plugin: plug leak in error path.

Partial back port of #1368.

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

snmp plugin: plug leak in error path.

Partial back port of #1368.

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

processes.c: plug memory leak in error path

Found with Infer (http://fbinfer.com/)

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

Travis: switch to new Trusty environment

https://docs.travis-ci.com/user/trusty-ci-environment

statsd plugin: Implement the "CounterSum" option.

Issues: #929, #1282, #1311

Merge branch 'pr/1345'

mqtt plugin: Add TLS options to the example config and improve documentation.

mqtt plugin: Rename "TLSprotocol" to "TLSProtocol".

mqtt plugin: Rename "CACertificateFile" to "CACert".

This way the option name is consistent with the name used by the cURL
plugins.

Sorry for previous comments in #1345 that pushed towards copying Apache
HTTPd's naming schema -- being consistent within collectd is more
important.

Merge commit 'refs/pull/1360/head' of github.com:collectd/collectd

Merge remote-tracking branch 'github/pr/1355'

utils_mount: use reentrant getmntent_r when we can

Fixes #1162

(cherry picked from commit a916ae72b89d0bcb5fe28d55def7327e8a1f6ce0)

Allow MQTT payloads without NULL string termination

Merge pull request #1163 from rubenk/fix-data-race-in-df-and-cgroup-plugins

utils_mount: use reentrant getmntent_r when we can

table plugin: log_warn is better than log_err

Merge branch 'collectd-5.5'

Merge branch 'collectd-5.4' into collectd-5.5

c_avl_pick needs to update the 'size' field of the tree when it deletes
a node.

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

utils_avltree_test.c: Extend unit test to test c_avl_pick().

This demonstrates the bug that c_avl_pick() does not decrease the AVL
tree's size.

Issue: #1353

Remove duplicate call to pthread_mutex_destroy.

statsd plugin: Correctly free metrics at shutdown.

The previous shutdown handler leaked "timer" and "set" metrics. This is
not a huge problem in itself, as the daemon is exiting shortly after
anyway, but it makes debugging of memory leaks very hard.

Issue: #997

Enable to disconnect at shutdown, changed default values, fixed time unit issue, fixed a typo and display precision.

gps: merge manpage into main collectd.conf.pod

Also mention plugin in README and attribute Nicolas in AUTHORS file.

gps: make plugin compatible with older libgps versions

Fixed time to ms instead of us, added a possibility to reconnect if gpsd server restarted.

Merging with changes made by Florian Foster.

gps plugin: Use cf_util_get_service() to read the "Port" option.

This allows users to provide unquoted numeric ports, too.

gps plugin: Store timeout and pause as cdtime_t.

This allows to specify sub-second precision in the usual manner in the
config file.

gps plugin: Coding style cleanup.

* Include <gps.h> and <pthread.h> unconditionally -- the code doesn't
  compile without these headers.
* Convert all data in cgps_data_t to gauge_t.
* Rename the "gps_data_read" variable to "data", protected by "data_lock".
* Handle errors and continue, allowing the following code to be outdented.
* Add "satellites-visible" in addition to "satellites-used".
* Remove newlines from log messages. Unify prefix to "gps plugin:".
* Unify function and type names to use the "cgps_" prefix.
* Don't check for NULL when calling free().

Applied comments from Florian FOSTER, added more documentation, pause, fixed some typos.

apache plugin: DRY: use macro to determine number of fields.

Applied comments from Florian FOSTER, added more documentation, pause, fixed some typos.

Add TLS support
addresses #1265
protect for newer libmosquitto
address octo's comments

Merge branch 'GPSd'

apache: parse whole response, not only the first 16 lines

mod_status recently started reporting more statistics. According to
http://www.apache.org/dist/httpd/CHANGES_2.4 `mod_proxy, mod_ssl,
mod_cache_socache, mod_socache_*: Support machine readable server-status
produced when using the "?auto" query string.` for version 2.4.13
onwards.

As we can't assume everything we need will be included in the first 16
lines of output anymore, we now simply scan through the whole output.

Remove POLLRDNORM (equivalent to POLLIN)

Checking for closed peer connection before send

restores support for TypeInstance in ts_create()

Initial support of GPSd in collectd.

Merge branch 'collectd-5.5'

Add utils_ignorelist back to daemon (#1323)

commit 216c6246b73645ac093de15b87aedc9abc6ebc80 moves the collectd
daemon files into the src/daemon/ directory, but also silently drops
utils_ignorelist. Dropping utils_ignorelist from the daemon has the
drawback that it cannot be used by plugins any more and that 16 plugins
need to bundle ignorelist.

Thus add utils_ignorelist back to the collectd daemon.

Merge branch 'collectd-5.4' into collectd-5.5

Fix clang complaining about extraneous parenthesis.

ASN_GAUGE (SNMP-SMIv2 Gauge32) is unsigned, not signed.

Start write threads after plugin initialization.

Some plugins such as "network" create own threads from within their
init callbacks which can then start submitting data to the queue
right away, even if the read threads haven't been started yet.

If write threads are started before plugin initialization, this can
result in a race where a plugin's write callback gets called before
that plugin's init callback has completed.

To fix this, delay starting the write threads until after all plugins
have been initialized.

Merge branch 'collectd-5.4' into collectd-5.5

statsd: provide a gauge for counters, holding the absolute count.

When not using StoreRates false, this allows statsd aggregators
to hold on to the absolute count, if needed.

Merge branch 'collectd-5.5'

Merge branch 'collectd-5.4' into collectd-5.5

battery: add missing capacity type to types.db

For the record, "capacity" was introduced by a2e732f8.

Also fix a bogus example in collectd.conf.

Thanks to @ryan-williams for reporting the problem !

Fix #1338

notify_nagios plugin: Add plugin for writing Nagios passive check results.

varnish: remove unused variable

This was used in a DEBUG statement I didn't check in.

varnish: rework v3 and v4 support

By using VSC_Iter() to loop over the list of metrics exposed by varnish,
we can pick those we're interested *if they are found*.

Not explicitly referring to metrics in the VSC_C_main struct makes the
plugin more resilient to small differences between minor varnish
versions.

It also opens the possibility to monitor non-MAIN metrics, such as
per-backend or per-storage engine stats.

This patch should be compatible with the previous way of doing, from the
user point of view.

Fix #1302

varnish: isolate varnish v2 code

Segregating v2-specific code will allow reworking the v3 and v4 parts,
while ensuring backwards compatibility with v2.

The downside is that this leads to a large chunk of mostly duplicate
code. That said, my suggestion would be to drop support for varnish v2
in collectd 5.6.

NB: this commit breaks v3 and v4 support.

apcups: prefix plugin name in INFO and DEBUG statements

Fixes #1329

Merge pull request #1311 from pyr/fix/1282

statsd: provide a gauge for counters, holding the absolute count.

Merge pull request #1309 from baryonix/time_fix

Fix TIMESPEC_TO_CDTIME_T() on 32-bit arch.

write_redis plugin: Add support for StoreRates option

write_redis plugin: Improve ZREMRANGEBYRANK command debug message

write_redis plugin: Fix ZREMBYRANK command debug message

write_redis plugin: Add option to limit sorted set size

Merge branch 'collectd-5.5'

write_redis: fix a few typos in manpage

Also correct the maximum length of the node name.
The callback name is "write_redis/%s" so the maximum
length of a node name is DATA_MAX_NAME_LEN - strlen("write_redis") -1.