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

Florian Forster octo at verplant.org
Wed Dec 9 16:51:32 CET 2009


 contrib/examples/MyPlugin.pm |   13 +++++++------
 src/protocols.c              |    2 +-
 src/utils_cmd_listval.c      |    6 ++++++
 3 files changed, 14 insertions(+), 7 deletions(-)

New commits:
commit 973b70c411688212c081f5e90b89bd657785b81c
Merge: 8579aab25eae16e266b03e9b623285e56036c922 bc42a61b1f8e518c7decb824dc61e85d4e9f02b8
Author: Florian Forster <octo at leeloo.lan.home.verplant.org>
Date:   Wed Dec 9 16:50:42 2009 +0100

    Merge branch 'collectd-4.7' into collectd-4.8

commit bc42a61b1f8e518c7decb824dc61e85d4e9f02b8
Author: Ben Knight <bkk at nerdboy.net.au>
Date:   Tue Dec 1 09:03:27 2009 +0100

    src/utils_cmd_listval.c: Free memory returned by `uc_get_names'.
    
    We've run into a memory leak in collectd, triggered by usage of 'listval'
    via the unixsock plugin.
    
    When making a 'listval' call, utils_cmd_listval.c:handle_listval() calls
    utils_cache.c:uc_get_names() to retrieve a list of active value names from the
    internal cache. uc_get_names() uses realloc() to allocate memory in which to
    store the list, and returns pointers.
    
    handle_listval() does not perform a free() on the returned memory. Each time
    listval is called, some memory is leaked. handle_getval() does not suffer from
    the same problem - a free() is called in that case.

commit 014985dbc0967cd281bd2e2b88ad16faf21838ae
Author: Florian Forster <octo at leeloo.lan.home.verplant.org>
Date:   Mon Nov 23 21:45:33 2009 +0100

    contrib/examples/MyPlugin.pm: Updated to new interface.

commit d82efb9e42492c32d26604a97c92928e4af0368f
Author: Sebastian Harl <sh at tokkee.org>
Date:   Sat Nov 21 13:07:16 2009 +0100

    protocols plugins: Fixed a typo in an error message.
    
    Thanks to lintian(1) for finding that :-)




More information about the collectd-changes mailing list