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

Florian Forster octo at verplant.org
Wed Aug 28 15:22:55 CEST 2013


 src/cgroups.c     |   19 ++++++++++++-------
 src/utils_mount.c |    3 ---
 2 files changed, 12 insertions(+), 10 deletions(-)

New commits:
commit 9875cce31c8aee20cae090d319f9885334bcf237
Merge: 57bd4c5 c94e448
Author: Florian Forster <octo at collectd.org>
Date:   Wed Aug 28 15:22:37 2013 +0200

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

commit 57bd4c578e337319d4a190311bb08ef98d0d7358
Author: Florian Forster <octo at collectd.org>
Date:   Wed Aug 28 15:19:48 2013 +0200

    cgroups plugin: Use cu_mount_checkoption().
    
    cu_mount_checkoption() behaves a bit like strstr() in that it returns a pointer
    into the buffer. cu_mount_getoptionvalue() allocated memory that was never
    freed, slowly leaking memory (although very little).
    
    Github: #412

commit 3dcfbf12d5ec0b62bc540ac59944ddf50c37c1f2
Author: Florian Forster <octo at collectd.org>
Date:   Wed Aug 28 15:18:11 2013 +0200

    src/utils_mount.c: Fix wrong behavior in cu_mount_getoptionvalue().
    
    Github: #412

commit c94e448d41d1852c2dd6ca4dc402ca6e225b5f37
Author: Marc Fournier <marc.fournier at camptocamp.com>
Date:   Wed Aug 21 16:54:10 2013 +0200

    correct typo

commit 4b4eee900b167cfdf17528e70c9c0659eb0329b0
Author: Marc Fournier <marc.fournier at camptocamp.com>
Date:   Wed Aug 21 16:22:32 2013 +0200

    cgroups: don't fail parsing when colon not found
    
    The format of cpuacct.stat seems to not always have a colon as separator.
    
    It appears to *not* have the colon at least on debian's kernel 3.2.41 and
    3.2.35, as well as on RHEL6 with 2.6.32. All of them have decent support for
    cgroups/lxc, and the cgroups plugin fails to work with them without this patch.
    
    Also, looking at other implementations shows that they *don't* expect a colon:
    https://bitbucket.org/dotcloud/liblxcstats/src/2558b4fbbf589c609895b0badbfc7d413466d716/probes/cpuacct.c?at=default#cl-71
    https://github.com/BrightcoveOS/Diamond/blob/master/src/collectors/cpuacct_cgroup/cpuacct_cgroup.py#L55




More information about the collectd-changes mailing list