virt: Fix memory handling for domains data

Memory with data for inactive domains was wrongly
freed earlier in refresh_lists() function and crash
was generated. Memory for active domains have
had the same problems with unnecessary free.
Also added handling for freeing inactive domain in
case of error.

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

virt: Fix code style in unit tests

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

virt: Fix memory leak when using libvirt <= 1.0

Cleanup was missing for libvirt-allocated virDomainPtr.

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

virt: Replace malloc with calloc for array allocations

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

src/virt.c: Decouple persistent notifications from refreshInterval

When persistentNotification is true, if refreshInterval value is larger than
(read) Interval, the notifications are not issued every read interval as should
be. The fix consist in decoupling persistent notifications from refreshInterval.

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

src/virt_test.c: Fix build error on ubuntu:trusty

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

src/virt_test.c: Fix memory leaks detected by valgrind

src/virt.c: The same memory leaks are also fixed.

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

src/virt.c: Fix build error on ubuntu:trusty

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

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>

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>

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>

Merge pull request #2687 from jeis2497052/master

Propose fix some typos

Propose fix some typos

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

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

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>

Merge branch 'collectd-5.8'

Merge branch 'collectd-5.7' into collectd-5.8

Makefile.am: Fix out-of-tree builds.

Fixes: #2602

processes plugin: Implement the "CollectDelayAccounting" option.

src/utils_taskstats.[ch]: Add library for Linux Delay Accounting.

Auto-Merge pull request #2593 from octo/ff/grpc

Automatically merged due to "Automerge" label

configure: Be more verbose about why the grpc plugin cannot be built.

configure: Improve detection of grpc_cpp_plugin.

When the `--with-libgrpc++=<path>` command line argument is given, add
`<path>/bin` to the search path for grpc_cpp_plugin.

Merge branch 'master' into pri

Replace zu with PRIu64 and llu with new macro, PRIsz, which will make it easier to make the code platform-independent.

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

contrib/format.sh src/grpc.cc

grpc plugin: Implement the VerifyPeer option for servers.

Merge branch 'collectd-5.8'

.travis.yml: Send notifications to the changes ML.

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.8'

Merge branch 'collectd-5.8'

.travis.yml: Call coverity scan from job 1 only.

The logic was inverted.

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 #2584 from octo/coverity

Automatically merged due to "Automerge" label

.travis.yml: Try to work around travis-ci/travis-ci#1975.

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

.travis.yml: Import list of dependencies from collectd-ci.

.travis.yml: Add coverity addon.

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.8'

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>