[collectd-changes] collectd, the system statistics collection daemon: Changes to 'master'

[Florian Forster]

 src/amqp1.c      |    2 +-
 src/dpdkevents.c |    4 ++--
 src/ipmi.c       |    2 +-
 src/snmp.c       |    1 -
 src/turbostat.c  |    6 +++---
 5 files changed, 7 insertions(+), 8 deletions(-)

New commits:
commit 6eb269f494c039f1ff94f4771ac7ebf9f2783cba
Author: Ruben Kerkhof <ruben at rubenkerkhof.com>
Date:   Tue May 29 11:29:52 2018 +0200

    dpdkevents plugin: fix sign compare warnings
    
    src/dpdkevents.c: In function ‘dpdk_helper_link_status_get’:
    src/dpdkevents.c:431:21: warning: comparison of integer expressions of different signedness: ‘int’ and ‘uint32_t’ {aka ‘unsigned int’} [-Wsign-compare]
       for (int i = 0; i < ec->nb_ports; i++) {
                         ^
    src/dpdkevents.c: In function ‘dpdk_events_link_status_dispatch’:
    src/dpdkevents.c:500:21: warning: comparison of integer expressions of different signedness: ‘int’ and ‘uint32_t’ {aka ‘unsigned int’} [-Wsign-compare]
       for (int i = 0; i < ec->nb_ports; i++) {
                         ^

commit c4aa1d3762ad3465a247a581a79a7c9be18bd421
Author: Ruben Kerkhof <ruben at rubenkerkhof.com>
Date:   Tue May 29 11:25:10 2018 +0200

    ipmi plugin: fix sign compare warning
    
      CC       src/ipmi_la-ipmi.lo
    src/ipmi.c: In function ‘sensor_unit_to_type’:
    src/ipmi.c:360:21: warning: comparison of integer expressions of different signedness: ‘int’ and ‘long unsigned int’ [-Wsign-compare]
       for (int i = 0; i < STATIC_ARRAY_SIZE(ipmi_db_type_map); i++)
                         ^

commit 990752086e35df7e47f46771e82a7e968b40c218
Author: Ruben Kerkhof <ruben at rubenkerkhof.com>
Date:   Tue May 29 11:23:25 2018 +0200

    amqp1 plugin: fix sign compare warning
    
      CC       src/amqp1_la-amqp1.lo
    src/amqp1.c: In function ‘amqp1_send_out_messages’:
    src/amqp1.c:138:29: warning: comparison of integer expressions of different signedness: ‘size_t’ {aka ‘long unsigned int’} and ‘int’ [-Wsign-compare]
           if (DEQ_SIZE(to_send) == link_credit)
                                 ^~

commit 74eda959ddf653b7d20779d84f0a140e85376726
Author: Ruben Kerkhof <ruben at rubenkerkhof.com>
Date:   Tue May 29 10:37:18 2018 +0200

    turbostat plugin: free the right variable
    
    scan-build: Using '/usr/bin/clang-6.0' for static analysis
    make  all-am
    make[1]: Entering directory '/home/ruben/src/collectd'
      CC       src/turbostat.lo
    src/turbostat.c:1292:5: warning: Argument to free() is the address of the global variable 'thread_delta', which is not memory allocated by malloc()
        sfree(threads);
        ^~~~~~~~~~~~~~
    ./src/daemon/common.h:41:5: note: expanded from macro 'sfree'
        free(ptr);                                                                 \
        ^~~~~~~~~
    src/turbostat.c:1299:5: warning: Argument to free() is the address of the global variable 'core_delta', which is not memory allocated by malloc()
        sfree(cores);
        ^~~~~~~~~~~~
    ./src/daemon/common.h:41:5: note: expanded from macro 'sfree'
        free(ptr);                                                                 \
        ^~~~~~~~~
    2 warnings generated.
      CCLD     turbostat.la

commit 6111d6c23af6e3a4cde8830dc79a117aeac2e4de
Author: Ruben Kerkhof <ruben at rubenkerkhof.com>
Date:   Tue May 29 10:29:14 2018 +0200

    snmp plugin: fix scan-build warning
    
     CC       src/snmp_la-snmp.lo
    src/snmp.c:592:5: warning: Value stored to 'status' is never read
        status = 0;
        ^        ~
    1 warning generated.