perl plugin: Fix potential double-free.

The av2notification_meta() function updated it's parameter by doing

    (*m) = malloc();

Afterwards, if an error occurred, it would call `free(*m);` and (potentially)
return, leaving an invalid pointer stored in `m`. The caller would then try to
free the returned pointer because it was non-NULL.

This patch fixes this by doing several code cleanups:

*   The meta argument is not updated unnecessarily. Instead, a local variable
    is allocated and used within the loop and only on success is the return
    pointer updated.
*   Introduce additional local variables instead of re-using the `tmp` variable
    multiple times.
*   Name the variable pointing to the end of the linked list appropriately
    `tail` and remove one level of indirection. Reading code that is using
    pointers to pointers is unnecessarily hard.

Remove parentheses around return arguments

They don't do any harm but they are not really needed either.
Contributors are not used to following this style, and reviewers have to
keep pointing it out in reviews. This takes up valuable time for both
the contributor and the reviewer and distracts from the more important
issues.

I used the following Coccinelle patch:

@@
expression e;
@@
- return (e);
+ return e;

spatch is having trouble with some files so I have cleaned up the rest
with a few regexes.

intel_pmu: formatting by clang-format

Signed-off-by: Serhiy Pshyk <serhiyx.pshyk@intel.com>

intel_pmu: Implement performance monitoring plugin

The intel_pmu plugin collects information about system performance
counters using kernel Linux perf interface.

Signed-off-by: Serhiy Pshyk <serhiyx.pshyk@intel.com>

Merge branch 'collectd-5.7'

Merge pull request #2275 from octo/ff/prometheus

write_prometheus plugin: Fix incorrect use of realloc().

Merge pull request #2274 from octo/ff/putval

src/utils_cmd_putval.c: Fix multi-value PUTVAL.

src/utils_cmd_putval.c: Fix multi-value PUTVAL.

vl.values was allocated outside of the loop and then copied into each
ret_putval->vl[]. This means that later values overwrote the values
stored in previous ret_putval->vl[].

write_prometheus plugin: Fix incorrect use of realloc().

Calling realloc(ptr, 0) is undefined. Call free() explicitly.

configure.ac: Remove empty else-block from AC_CHECK_LIB.

Fixes: #2272

Merge branch 'collectd-5.7'

Merge branch 'collectd-5.6' into collectd-5.7

bind plugin: Use timegm() to convert to time_t if available.

FreeBSD doesn't provide the "extern long timezone", presumably because
it's "only" an XSI extension, leading to a portability issue with the
previous approach.

timegm() is a non-standard function available in GNU's and BSD's libc
which is doing exactly what we need. The previous code is left as a
fallback.

Fixes: #1268

fix BIND timezone parsing issue

collectd.conf(5): Add section documenting the ignorelist functionality.

Fixes: #2267

Merge pull request #2265 from velotraveler/master

fix BIND timezone parsing issue

fix BIND timezone parsing issue

libcollectdclient: Include "collectd/types.h" instead of "collectd/client.h".

libcollectdclient: Use IPV6_JOIN_GROUP instead of IPV6_ADD_MEMBERSHIP.

IPV6_JOIN_GROUP appears to be more portable.

Merge pull request #2260 from bufadu/evol_redis

redis: adding support for keys count per database

redis: fix parser and comments

libcollectdclient: Include <netinet/in.h>.

We need to include this for a variety of symbols on FreeBSD.

redis: fixing maintainer requests

systemd.collectd.service: fix a typo

libcollectdclient: Fix BSD support.

FreeBSD needs <sys/endian.h> instead of <endian.h>.

redis: removing sprintf() calls

mcelog: code review updates

* Comparing variables with NULL explicilty instead of !var
* Removed unnecessary casts

Change-Id: I50a1691c42842c81ff71c2e91a8fb292524dc945
Signed-off-by: Tahhan, Maryam <maryam.tahhan@intel.com>

mcelog: only send notifications for new errors

Update the mcelog plugin to only distpatch notifications for memory corrected
and uncorrected errors when the value changes for each dimm.

Change-Id: I29ee6b8519e273bd0ef47eed1a0084d3b6d7cef0
Signed-off-by: Tahhan, Maryam <maryam.tahhan@intel.com>

mcelog: different notifications for memory errors

For mcelog's unix socked data source (default /var/run/mcelog-client), mcelog
plugin generates collectd notifications that merge both corrected and
uncorrected memory error info. This is incorrect behaviour. Separate
notifications should be created for each category and also severity should be
differentiated. This patch fixes this.

Change-Id: Ie9eb1dd6bbc7bbbdd9fc72f1f2e7c395031b3d51
Signed-off-by: Tahhan, Maryam <maryam.tahhan@intel.com>

redis: adding support for key count per database

libcollectdclient: Use GCRY_THREAD_OPTION_PTHREAD_IMPL with older gcrypt versions only.

Copied from src/network.c

libcollectdclient: Remove TODO.

No, we should not, because memory allocation is more annoying in C.

libcollectdclient: Improve documentation of lcc_listener_t.conn.

libcollectdclient: Replace TODOs with (hopefully) useful comments.

libcollectdclient: Write only metrics with the requested security level.

libcollectdclient: Add LCC_NETWORK_BUFFER_SIZE.

libcollectdclient: Add lcc_network_parse_options_t to lcc_listener_t.

This avoids listing the same fields twice.

libcollectdclient: Introduce a "parser" callback.

This will allow the network plugin to enqueue received network packets and
parse them using a thread pool.

libcollectdclient: Install the "collectd/types.h" header.

libcollectdclient: Fix off-by-one error when reading the username.

The allocated buffer includes a null byte, but the network packet doesn't.
That means we were reading one byte too many from the buffer, overwriting
the terminating null byte in the buffer.

Merge pull request #2255 from clemensg/sensors_support_current

sensors: support current values

sensors: support current values

Add support for current sensor data collection.

Signed-off-by: Clemens Gruber <clemensgru@gmail.com>

Merge pull request #2252 from jaroug/kairosdb_ttl_support

Kairosdb ttl support

fix spelling

Merge pull request #2253 from LionelCons/master

data-sets are in fact supported (issues#2244)

correct spelling

data-sets are in fact supported (issues#2244)

replace spaces by tabs

Add support for the new TTL parameter - kairosdb only

add documentation on the new TTL parameter

Merge branch 'collectd-5.7'

Merge branch 'collectd-5.6' into collectd-5.7

libcollectdclient: Separate module for lcc_network_parse().

libcollectdclient: Implement parsing of signed and encrypted network data.

libcollectdclient: Implement server code.

The implementation is intentionally similar to the Golang implementation.
Security features (signing, encryption) are still missing.

libcollectdclient: Include headers with the "collectd/" path prefix.

Tree-wide: remove last remnants of sizeof(char)

sizeof(char) is 1 by definition.

Merge pull request #2168 from maryamtahhan/feat_libvirt_upstream

virt: add support for new metrics

libcollectdclient: Move type definitions into their own header.

The "types.h" header is included by "client.h", so existing code does not
need to be updated.

Update mongodb write code to use latest API (Fixes: #492) (#2236)

Update mongodb write code to use latest API (Fixes: #492)

Remove utils_ignorelist.[ch] from daemon; link plugins directly.

The collectd daemon isn't using the "ignorelist" functionality itself --
it was only linked in to provide it to loaded plugins. Since the code is
under the GPL, this interfers with our intend of having an MIT licensed
daemon.

This patch changes the build system to link plugins with the ignorelist
directly instead of expecting the symbols to be in the daemon.

network plugin: Fix endless loop DOS in parse_packet()

When correct 'Signature part' is received by Collectd, configured without
AuthFile option, condition for endless loop occurs due to missing increase
of pointer to next unprocessed part.

This is a forward-port of #2233.

Fixes: CVE-2017-7401
Closes: #2174
Signed-off-by: Florian Forster <octo@collectd.org>

Merge pull request #2051 from dsilakov/virt_list_domains

virt plugin: get domains using virConnectListAllDomains

Merge pull request #1989 from rpv-tomsk/processes

processes plugin: Code cleanup + add per-process open files count

processes plugin: collect per-process open files count on Linux
Closes: #836

processes plugin: Update all counters via ps_update_counter()
Fix counting when matched processes are started and terminated often.

processes plugin: Code cleanup

Merge pull request #2199 from jaroug/support_attributes_kairosdb

Add support for 'Attribute' option on KAIROSDB output format

Merge pull request #2223 from maryamtahhan/intel_rdt_core_id_fix

intel_rdt: fix validation of configured core id values

Merge pull request #2225 from maryamtahhan/dpdstat_portmask_fix

dpdkstat: Enable all ports by default.

snmp_agent: generate table OID based on instance as a key in case index is not specfified.

Based on the SNMP rules when table column is represented as a key
of string type, key should be used while generating OID for each row.
When index is not specified, then it's assumed that instance value is
the key for the specified table.

Change-Id: I9f3fbd9aa018bc9037bd3202e2ed107a302fe1b3
Signed-off-by: Korynkevych, RomanX <romanx.korynkevych@intel.com>

dpdkstat: Enable all ports by default.

If "EnabledPortMask" option is not set
all DPDK ports will be monitored.

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

intel_rdt: fix validation of configured core id values

Change-Id: Ia009635c93e1cecd9852326b01c62fc6d2b090b5
Signed-off-by: Serhiy Pshyk <serhiyx.pshyk@intel.com>

Merge pull request #2215 from maryamtahhan/dpdkstats_upstream

dpdkstat: doc updated with initialization time notice

dpdkstat: doc updated with initialization time notice

Change-Id: Ia13c35e4112e74990896edb597d39c1f8ab14b6e
Signed-off-by: Krzysztof Matczak <krzysztofx.matczak@intel.com>

Merge pull request #2210 from rpv-tomsk/processes-fix-init

processes plugin: Fix broken compilation on AIX

Merge pull request #2213 from maryamtahhan/ovs_enable_platforms

ovs: enable only supported platforms (#2203)

ovs: enable only supported platforms

fixes GitHub issue #2203

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

Merge pull request #2211 from maryamtahhan/feat_ovs_events_change_defaults

ovs_events: change configuration option defaults

Merge pull request #2212 from maryamtahhan/ovs_events_scan_build

ovs_events: fixes issue #2205

virt: add support for new metrics

Add support for new libvirt metrics:
- Performance monitoring events
- Domain state & reason
- Percentage CPU usage
- CPU pinning (affinity)
- Disk errors
- File system info
- Job stats

Change-Id: I233fb18653d218c7e906a157743606c6818f9feb
Signed-off-by: Przemyslaw Szczerbik <przemyslawx.szczerbik@intel.com>

ovs_events: fix scan-build warnings #2205

src/utils_ovs.c:366:20: warning: Access to field 'type' results in a dereference of a null pointer (loaded from variable 'jval')
              (int)(jval)->type);
                   ^~~~~~~~~~~~
...

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

ovs_events: change configuration option defaults

Right now, the plugin dispatches values by default and doesn't send
any notification messages. The expected behaviour of the event plugin
is to send notification by default. So, changed default value of
SendNotification & DispatchValues options to opposite.

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

processes plugin: Fix broken compilation on AIX

After a bb0000acfc57e230fc1c26099bc7acb913680f1a an extra pse.cpu_system
statement was left in the AIX / elif HAVE_PROCINFO_H section, which caused
the compilation to fail.

Fixes: #1981

Merge pull request #2207 from maryamtahhan/fix_for_2201

ovs_stats: Increase devname buffer

Increase devname buffer buffer to ensure that snprintf output
would not be truncated before the last format character

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

collectd.spec: move new plugins to subpackages

collectd.spec: add ovs_stats plugin

virt plugin: fix build warning

CC       src/virt_la-virt.lo
src/virt.c:417:37: warning: comparison of integers of different signs: 'int' and 'unsigned long' [-Wsign-compare]
  if ((tag_index < 0) || (tag_index >= STATIC_ARRAY_SIZE(tags))) {
                          ~~~~~~~~~ ^  ~~~~~~~~~~~~~~~~~~~~~~~
1 warning generated.

Merge branch 'collectd-5.7'

Conflicts:
contrib/redhat/collectd.spec

Merge branch 'collectd-5.6' into collectd-5.7

Conflicts:
contrib/redhat/collectd.spec

collectd.spec: Don't enable XFS support on RHEL6

It is missing for i386.
If there's strong interest in having support for xfs in the df plugin in
RHEL6 we can do some macro magic to only BuildRequire xfsprogs-devel if
were not on rhel6 i386, but this was the easiest fix and unblocks our
package builds.

spec: Don't enable XFS support on RHEL6

It is missing for i386.

Add dpkdevents plugin to spec file

utils_cmd_flush.c: fix another build warning

src/utils_cmd_flush.c:140:11: warning: declaration shadows a local variable [-Wshadow]
      int status;
          ^
src/utils_cmd_flush.c:114:7: note: previous declaration is here
  int status;
      ^

utils_cmd_flush.c: fix build warning

CC       src/utils_cmd_flush.lo
src/utils_cmd_flush.c:139:12: warning: declaration shadows a local variable [-Wshadow]
      char buffer[1024];
           ^
src/utils_cmd_flush.c:108:47: note: previous declaration is here
cmd_status_t cmd_handle_flush(FILE *fh, char *buffer) {
                                              ^

ovs_stats plugin: fix some more build warnings

CC       src/ovs_stats_la-ovs_stats.lo
src/ovs_stats.c:158:8: warning: 'const' type qualifier on return type has no effect [-Wignored-qualifiers]
static const iface_counter ovs_stats_counter_name_to_type(const char *counter) {
       ^~~~~~
src/ovs_stats.c:356:29: warning: comparison of integers of different signs: 'int' and 'size_t' (aka 'unsigned long') [-Wsign-compare]
          for (int i = 0; i < ports_num; i++)
                          ~ ^ ~~~~~~~~~
src/ovs_stats.c:401:23: warning: comparison of integers of different signs: 'int' and 'size_t' (aka 'unsigned long') [-Wsign-compare]
    for (int i = 0; i < YAJL_GET_OBJECT(bridges)->len; i++) {
                    ~ ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/ovs_stats.c:415:23: warning: comparison of integers of different signs: 'int' and 'size_t' (aka 'unsigned long') [-Wsign-compare]
    for (int i = 0; i < YAJL_GET_OBJECT(bridges)->len; i++) {
                    ~ ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/ovs_stats.c:501:23: warning: comparison of integers of different signs: 'int' and 'size_t' (aka 'unsigned long') [-Wsign-compare]
    for (int i = 0; i < YAJL_GET_OBJECT(ports)->len; i++) {
                    ~ ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/ovs_stats.c:524:23: warning: comparison of integers of different signs: 'int' and 'size_t' (aka 'unsigned long') [-Wsign-compare]
    for (int i = 0; i < YAJL_GET_OBJECT(ports)->len; i++) {
                    ~ ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/ovs_stats.c:538:23: warning: comparison of integers of different signs: 'int' and 'size_t' (aka 'unsigned long') [-Wsign-compare]
    for (int i = 0; i < YAJL_GET_ARRAY(stats)->len; i++) {
                    ~ ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~
src/ovs_stats.c:558:23: warning: comparison of integers of different signs: 'int' and 'size_t' (aka 'unsigned long') [-Wsign-compare]
    for (int i = 0; i < YAJL_GET_ARRAY(ext_ids)->len; i++) {
                    ~ ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/ovs_stats.c:684:23: warning: comparison of integers of different signs: 'int' and 'size_t' (aka 'unsigned long') [-Wsign-compare]
    for (int i = 0; i < YAJL_GET_OBJECT(ports)->len; i++)
                    ~ ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~
9 warnings generated.

ovs_stats plugin: fix build warning

src/ovs_stats.c:158:8: warning: 'const' type qualifier on return type has no effect [-Wignored-qualifiers]
static const iface_counter ovs_stats_counter_name_to_type(const char *counter) {
       ^~~~~~

Merge branch 'collectd-5.7'

Conflicts:
configure.ac

Merge branch 'collectd-5.6' into collectd-5.7