tokkee.org

Merge branch 'collectd-5.5'

Merge branch 'collectd-5.4' into collectd-5.5

modbus plugin: Make sure variable is initialized.

This was introduced by the merge
(09c6a320f3cb36b5dbb2c2ce43858f33be7acf9b).

stats plugin: Link with libm (-lm).

Fixes a bug reported by @Tenzer:

    symbol lookup error: /usr/lib/collectd/statsd.so: undefined symbol: log

tail plugin: Make sure variable is initialized.

This was introduced by the merge
(09c6a320f3cb36b5dbb2c2ce43858f33be7acf9b).

Fix "Undefined allocation of 0 bytes (CERT MEM04-C; CWE-131)" warnings.

threshold plugin: Fix "Dead increment" warnings.

Fix "Argument with 'nonnull' attribute passed null" warnings.

Merge branch 'collectd-5.5'

Merge branch 'collectd-5.4' into collectd-5.5

threshold plugin: Fix "Dead assignment" warning.

ted plugin: Fix "Dead assignment" warning.

tail_csv plugin: Fix "Dead assignment" warning.

tail plugin: Fix "Dead assignment" warning.

snmp plugin: Fix "Dead assignment" warning.

powerdns plugin: Fix "Dead assignment" warning.

ping plugin: Fix "Dead assignment" warning.

onewire plugin: Fix "Dead assignment" warning.

openvpn plugin: Fix "Dead assignment" warning.

ntpd plugin: Fix "Dead assignment" warning.

network plugin: Fix "Dead assignment" warning.

modbus plugin: Fix "Dead assignment" warning.

onewire plugin: Fix "Dead assignment" warning.

nfs plugin: Fix handling of servers with 42 or 44 fields.

battery plugin: Fix file descriptor leak.

src/libcollectdclient/client.c: Fix "Dead assignment" warning.

gmond plugin: Fix handling of COUNTER and ABSOLUTE data sources.

email plugin: Fix "Dead assignment" warning.

src/daemon/filter_chain.c: Fix "Dead assignment" warning.

curl_json plugin: Fix "Dead assignment" warning.

amqp plugin: Fix "Dead assignment" warning.

src/utils_avltree.c: Add assertions to rotate_{left,right}().

clang's static code analysis thought that x->right / x->left could be NULL,
reporting false positives. Let's see if this fixes it.

snmp plugin: Fix off-by-one array access.

exec plugin: Don't assign variable that is never read.

Merge branch 'collectd-5.5'

Merge branch 'collectd-5.4' into collectd-5.5

src/utils_db_query.c: Fix use-after-free.

"r_area->next" was evaluated after "r_area" was freed.

barometer plugin: Initialize output arrays.

src/daemon/common.c: Implement strjoin() with memcpy().

The previous implementation used strncat() which has the unfortunate and
unintuitive behavior of copying n+1 bytes to the buffer.

src/utils_cache.c: Remove incorrect free.

"ce" is not updated / used in this loop, so it would be theoretically
possible for "ce" to be freed twice.

Merge branch 'collectd-5.5'

fix openldap segfatult when ldapi is used  v2

fix openldap segfatult when ldapi is used

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

Merge branch 'collectd-5.5'

Merge branch 'collectd-5.4' into collectd-5.5

network plugin: Avoid "status may be used uninitialized" warning.

liboconfig: Declare yylex().

email plugin: Fix freeing linked lists.

The previous code essentially did:

    for (…; …; ptr = ptr->next)
      free (ptr);

The "ptr->next" is a use-after-free.

network plugin: Make sure all memory is freed when the receive thread fails.

Not all that useful (your receive thread just died …) but hopefully
makes the static analysis happy.

Merge branch 'collectd-5.5'

Merge branch 'collectd-5.4' into collectd-5.5

Merge remote-tracking branch 'github/pr/1088' into collectd-5.4

src/utils_llist.c: Handle unlikely corner case.

This handles the following (unlikely) case:
    (l->head == NULL) && (e == NULL)

In this case, the following code will dereference a NULL pointer:
    if (l->head == e)
      l->head = e->next;

src/utils_avltree.c: Rewrite checks in c_avl_pick().

The previous code made the (correct) assumption that "height" is always
greater than zero. This tripped up clang's "scan-build".

This confuses the static analysis in two more places in this file, which
are not as easy to fix :(

ceph plugin: Fix allocation size.

barometer plugin: Fix allocation size.

network plugin: Explicitly initialize "fd_num" to zero.

I have the feeling that initialization via memset() and unions in the sockent
structure confuse clang's scan-build.

notification target: Pass void** to tn_destroy().

The previous call was bad, resulting in errors in free().

perl plugin: Fix invalid free().

notification_meta_t.name is a char[], which we must not free.

liboconfig: Disable some clang warnings.

Hopefully fixes:

scanner.c:4128:17: error: unused function 'yyunput' [-Werror,-Wunused-function]
    static void yyunput (int c, register char * yy_bp )
                ^
scanner.c:4173:16: error: function 'input' is not needed and will not be emitted [-Werror,-Wunneeded-internal-declaration]
    static int input  (void)
               ^
2 errors generated.

liboconfig: Declare yyparse().

Should hopefully fix:

oconfig.c:68:12: error: implicit declaration of function 'yyparse' is invalid in C99 [-Werror,-Wimplicit-function-declaration]
  status = yyparse ();
           ^

Merge branch 'collectd-5.5'

Merge branch 'collectd-5.4' into collectd-5.5

add support for simple authentication

In rrd synchronous file creation use lock_file
to prevent the creation of the same file at the same time multiple times
and obtain a corrupt rrd file.

network plugin: Simplify config handling.

Remove network_config_set_{boolean,string} and replace them with
cf_util_get_{boolean,string}. The other, more specialized, config handling
functions were also simplified.

Allow empty Plugin blocks.

Allow empty Plugin blocks.

use counter type for Innodb_buffer_pool_pages_flushed

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

Fix 64 bit check issue, running the preprocessor does not honour CFLAGS which holds the relevant -m64

use counter type for Innodb_buffer_pool_pages_flushed

Merge branch 'collectd-5.5'

Corrected text in README

Merge branch 'collectd-5.4' into collectd-5.5

Conflicts:
src/write_redis.c

Revert "contrib/redhat/collectd.spec: added bison and flex"

This reverts commit 38203b1c8d8a8e5e567d7f6783dd8b88a2487070.

A misunderstanding about which tarball to use with the RPM specfile led
us here. See #1079 for the whole conversation.

configure: turbostat: check for MSR_PKG_C10_RESIDENCY symbol

Older versions of `asm/msr-index.h` don't have all the symbol needed to
build the turbostat plugin. Checking for MSR_PKG_C10_RESIDENCY seems to
guarantee we have a recent enough `asm/msr-index.h`.

Fixes #1075

collectd.conf.pod: update libyajl URL

Fixes #1081

Thanks to @msalmonse for pointing this out!

Fix a libtool warning

libtool: warning: '-version-info/-version-number' is ignored for
convenience libraries

Merge remote-tracking branch 'github/pr/1065' into collectd-5.5

Merge branch 'collectd-5.5'

contrib/redhat/collectd.spec: added bison and flex

Current .spec does not work due to missing bison and flex dependencies.
This fix lets install bison and flex into the build environment as well as
define YACC and YFLAGS environment variables.

write_redis plugin: Check the return value of format_values().

write_redis: Increase parsability by using format_values from common.h

write_redis: Increase parsability of multi-valued keys by insterting delimiting characters

configure.ac: Move all checks for "struct ip6_ext" into one place.

Fixes: #348

Automatically add -DSOLARIS2=8 if required for ip6_ext

Fixes: #348

processes plugin: Initialize the cswitch_{in,}vol fields.

Merge branch 'collectd-5.5'

Makefile.am: Link tests with required libraries, too.

"make check" fails on Solaris because libraries are missing:

      CCLD     test_common
    Undefined first referenced
     symbol   in file
    kc                          ./.libs/libcommon.a(common.o)
    kstat_data_lookup           ./.libs/libcommon.a(common.o)
    kstat_lookup                ./.libs/libcommon.a(common.o)
    kstat_read                  ./.libs/libcommon.a(common.o)
    getaddrinfo                 ./.libs/libcommon.a(common.o)
    freeaddrinfo                ./.libs/libcommon.a(common.o)
    gai_strerror                ./.libs/libcommon.a(common.o)
    ld: fatal: symbol referencing errors. No output written to test_common
    collect2: error: ld returned 1 exit status

configure.ac: Ask users to build 64-bit Solaris binaries.

By default, the compilers will build 32-bit binaries on Solaris. This is
sub-optimal for us, for example reading /proc entries for 64-bit
processes doesn't work when collectd is 32-bit.

Fixes: #1077

zone plugin: Fix strerror -> sstrerror.

zone plugin: Minor code cleanups.

zone plugin: Undefine _FILE_OFFSET_BITS when building on 32bit hosts.

Fixes: #1077

swap plugin: Report swap sizes in bytes.

Regression from 43aa1c5cc9ab69004d0fe6de480c78815224111b.

Merge branch 'collectd-5.5'

Merge branch 'collectd-5.4' into collectd-5.5

dns plugin: Check for "struct ip6_ext".

Solaris appears to declare the struct, but only when The Right defines
are specified at compile time. As a quick and dirty fix, only build with
IPv6 support when the struct is usable.

Fixes: #348

smart: KNF