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

Florian Forster octo at verplant.org
Tue Dec 1 09:13:49 CET 2009


 src/utils_cmd_listval.c |    6 ++++++
 1 files changed, 6 insertions(+), 0 deletions(-)

New commits:
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.




More information about the collectd-changes mailing list