I can take a look this evening. Assuming nobody has already started? Best,
Charles On 2 Oct 2013 20:23, "Daniel-Constantin Mierla" <[email protected]> wrote: > Hello, > > there is (still) a memory leak in memcached module, discovered on a report > by Dragos Oancea. > > The pkg usage logs are like: > > 0(24328) NOTICE: qm_status: 19010. N address=0x7fb23683bc98 > frag=0x7fb23683bc68 size=8 used=1 > 0(24328) NOTICE: qm_status: alloc'd from memcached: > ../../parser/../ut.h: pkg_str_dup(733) > 0(24328) NOTICE: qm_status: start check=f0f0f0f0, end check= > c0c0c0c0, abcdefed > 0(24328) NOTICE: qm_status: 19011. N address=0x7fb23683bd00 > frag=0x7fb23683bcd0 size=48 used=1 > 0(24328) NOTICE: qm_status: alloc'd from memcached: > memcached.c: mcd_malloc(127) > 0(24328) NOTICE: qm_status: start check=f0f0f0f0, end check= > c0c0c0c0, abcdefed > 0(24328) NOTICE: qm_status: 19012. N address=0x7fb23683bd90 > frag=0x7fb23683bd60 size=8 used=1 > 0(24328) NOTICE: qm_status: alloc'd from memcached: > ../../parser/../ut.h: pkg_str_dup(733) > 0(24328) NOTICE: qm_status: start check=f0f0f0f0, end check= > c0c0c0c0, abcdefed > 0(24328) NOTICE: qm_status: 19013. N address=0x7fb23683bdf8 > frag=0x7fb23683bdc8 size=48 used=1 > 0(24328) NOTICE: qm_status: alloc'd from memcached: > memcached.c: mcd_malloc(127) > 0(24328) NOTICE: qm_status: start check=f0f0f0f0, end check= > c0c0c0c0, abcdefed > 0(24328) NOTICE: qm_status: 19014. N address=0x7fb23683be88 > frag=0x7fb23683be58 size=8 used=1 > 0(24328) NOTICE: qm_status: alloc'd from memcached: > memcached.c: mcd_malloc(127) > 0(24328) NOTICE: qm_status: start check=f0f0f0f0, end check= > c0c0c0c0, abcdefed > 0(24328) NOTICE: qm_status: 19015. N address=0x7fb23683bef0 > frag=0x7fb23683bec0 size=16 used=1 > 0(24328) NOTICE: qm_status: alloc'd from memcached: > ../../parser/../ut.h: pkg_str_dup(733) > 0(24328) NOTICE: qm_status: start check=f0f0f0f0, end check= > c0c0c0c0, abcdefed > 0(24328) NOTICE: qm_status: 19016. N address=0x7fb23683bf60 > frag=0x7fb23683bf30 size=8 used=1 > 0(24328) NOTICE: qm_status: alloc'd from memcached: > memcached.c: mcd_malloc(127) > 0(24328) NOTICE: qm_status: start check=f0f0f0f0, end check= > c0c0c0c0, abcdefed > 0(24328) NOTICE: qm_status: 19017. N address=0x7fb23683bfc8 > frag=0x7fb23683bf98 size=24 used=1 > 0(24328) NOTICE: qm_status: alloc'd from memcached: > ../../parser/../ut.h: pkg_str_dup(733) > 0(24328) NOTICE: qm_status: start check=f0f0f0f0, end check= > c0c0c0c0, abcdefed > > The one related to pkg_str_dup() should be fixed by the commit: > > - http://git.sip-router.org/cgi-**bin/gitweb.cgi/sip-router/?a=**commit;h= > **6faf12653c1db9f011b1826061824c**831bda3f58<http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=6faf12653c1db9f011b1826061824c831bda3f58> > > The other one is related to mcd_malloc(), which I guess it is due to the > function that returns the value from memchache, memcached_get() used in > pv_get_mcd_value_helper() -- the returned object has to be freed by > calling code, according to: > > - > http://docs.libmemcached.org/**memcached_get.html<http://docs.libmemcached.org/memcached_get.html> > > Since the libmemcached was initialized with wrappers around pkg > malloc/free, I expect the respective free function has to be used to free > the result. > > Can any of memcached devs check my commit and investigate further the > second leak? > > Cheers, > Daniel > > -- > Daniel-Constantin Mierla - http://www.asipto.com > http://twitter.com/#!/miconda - > http://www.linkedin.com/in/**miconda<http://www.linkedin.com/in/miconda> > Kamailio Advanced Trainings - Berlin, Nov 25-28; Miami, Nov 18-20, 2013 > - more details about Kamailio trainings at http://www.asipto.com - > > > ______________________________**_________________ > sr-dev mailing list > [email protected] > http://lists.sip-router.org/**cgi-bin/mailman/listinfo/sr-**dev<http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev> > -- www.sipcentric.com Follow us on twitter @sipcentric <http://twitter.com/sipcentric> Sipcentric Ltd. Company registered in England & Wales no. 7365592. Registered office: Unit 10 iBIC, Birmingham Science Park, Holt Court South, Birmingham B7 4EJ.
_______________________________________________ sr-dev mailing list [email protected] http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev
