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

[Florian Forster]

 README                     |    2 +-
 src/daemon/utils_avltree.c |   14 +++++++++++---
 src/daemon/utils_llist.c   |    3 +++
 src/liboconfig/oconfig.c   |    1 +
 src/liboconfig/scanner.l   |    6 ++++++
 src/network.c              |    4 ++++
 src/perl.c                 |    1 -
 src/target_notification.c  |    2 +-
 src/utils_rrdcreate.c      |   28 +++++++++++++++++++++-------
 9 files changed, 48 insertions(+), 13 deletions(-)

New commits:
commit cf33da9ad00ac94931a2c87563f8d007a996f1ad
Merge: 4f70f3c 7909156
Author: Florian Forster <octo at collectd.org>
Date:   Wed Jun 17 09:12:26 2015 +0200

    Merge branch 'collectd-5.4' into collectd-5.5

commit 7909156126916e6fd6ad97fa45b2a8391c78ba22
Merge: 707fa6c 33819a4
Author: Florian Forster <octo at collectd.org>
Date:   Wed Jun 17 09:12:08 2015 +0200

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

commit 707fa6cb057b53846aaa27a505fe05226612ecd3
Author: Florian Forster <octo at collectd.org>
Date:   Wed Jun 17 09:09:17 2015 +0200

    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;

commit 86ef26818f3a1af5afb4842e9c97c34ffe98bafa
Author: Florian Forster <octo at collectd.org>
Date:   Wed Jun 17 08:55:42 2015 +0200

    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 :(

commit 6efc94f59c38e86309078fb9e20c78a2e9c8ff59
Author: Florian Forster <octo at collectd.org>
Date:   Wed Jun 17 07:50:46 2015 +0200

    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.

commit b37ddefb01607272698e10c1e84a62ccbb8a5036
Author: Florian Forster <octo at collectd.org>
Date:   Wed Jun 17 07:34:57 2015 +0200

    notification target: Pass void** to tn_destroy().
    
    The previous call was bad, resulting in errors in free().

commit 94e5829f3f2a6ea5eef5455cf3f39a28f7f8cdaa
Author: Florian Forster <octo at collectd.org>
Date:   Wed Jun 17 07:26:39 2015 +0200

    perl plugin: Fix invalid free().
    
    notification_meta_t.name is a char[], which we must not free.

commit 33819a49ab40ec2f54f5ee12b3b88ab4205873eb
Author: Florian Forster <octo at collectd.org>
Date:   Tue Jun 16 23:07:34 2015 +0200

    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.

commit 53be6eefcb3d53634e04ffb1ca3db7fd7f064c34
Author: Florian Forster <octo at collectd.org>
Date:   Tue Jun 16 23:06:56 2015 +0200

    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 ();
               ^

commit 84f5f5923d8109366b2c3caf66ad7c2d89be38bd
Author: Manuel Luis Sanmartín Rozada <manuel.luis at gmail.com>
Date:   Wed May 20 17:10:41 2015 +0200

    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.

commit 0944585208ec2427fc434881e99228e9f870eaef
Author: Manuel Luis Sanmartín Rozada <manuel.luis at gmail.com>
Date:   Wed May 20 22:06:18 2015 +0200

    Allow empty Plugin blocks.

commit df3e412ed81290689b3af9b2e83c19a245d0e978
Author: Kelly Thomas Kline <kellytk at users.noreply.github.com>
Date:   Mon Jun 15 14:05:51 2015 -0700

    Corrected text in README