Hello,

you don't have to generate the readme files anymore, they are generated automatically on the server after committing the xml files.

For creating the pull requests, just search the web/github about how to do it for github -- there are some good manuals in place hosted by github.com, but I don't have the links at hand right now.

Cheers,
Daniel


On 07/11/16 10:42, Jurijs Ivolga wrote:
Hi Daniel,

I found modules what are impacted by this leak:

http_client
utils
xcap
http_async_client
auth_identity
xcap_client

I would like to update documentations, but this is first time when I'm updating documentation.

I believe I need to update xml file for module. Should I then regenerate readme file as stated here:

https://www.kamailio.org/wiki/devel/how-to/module-readme

Or not?

Then I believe to create pull request with my changes. Correct?

Maybe you have manual for this?

With kind regards,

Jurijs

On Wed, Oct 12, 2016 at 4:30 PM, Daniel-Constantin Mierla <miconda@gmail.com> wrote:

Hello,

ok, so that was...

Maybe it would be good to add a note to the docs of the module about this issue so people become aware of it. I guess the other http_* modules are affected. Pull requests or other suggestions are welcome, of course!

Cheers,
Daniel


On 12/10/16 15:04, Jurijs Ivolga wrote:
Hi Daniel,

Thank you a lot, it looks that issue is solved, after updating libcurl.

I was using following manual for updating libcurl, in case if somebody will have same issue.

https://www.digitalocean.com/community/questions/how-to-upgrade-curl-in-centos6

With kind regards,

Jurijs

On Tue, Oct 11, 2016 at 2:43 PM, Jurijs Ivolga <jurijs.ivolga@gmail.com> wrote:
Hi Daniel,

You are correct we are using heavily http_query.

I found following bug report:

https://bugs.centos.org/view.php?id=9391

I will try to update to libcurl 7.44 and check if this help.

Thank you a lot Daniel!

With kind regards,

Jurijs

On Tue, Oct 11, 2016 at 10:55 AM, Daniel-Constantin Mierla <miconda@gmail.com> wrote:

Hello,

from the logs, it seems to be related to curl library, I see many reports like:

==16459== 189,318 bytes in 167 blocks are possibly lost in loss record 681 of 683
==16459==    at 0x4C26FEF: calloc (vg_replace_malloc.c:711)
==16459==    by 0x104BB699: ??? (in /usr/lib64/libnsspem.so)
==16459==    by 0x104AA537: ??? (in /usr/lib64/libnsspem.so)
==16459==    by 0x104AB81E: ??? (in /usr/lib64/libnsspem.so)
==16459==    by 0x104B0B88: ??? (in /usr/lib64/libnsspem.so)
==16459==    by 0x104B77E1: ??? (in /usr/lib64/libnsspem.so)
==16459==    by 0xB71ABC9: ??? (in /usr/lib64/libnss3.so)
==16459==    by 0xB71AE62: PK11_CreateGenericObject (in /usr/lib64/libnss3.so)
==16459==    by 0xA0674DF: ??? (in /usr/lib64/libcurl.so.4.1.1)
==16459==    by 0xA067666: ??? (in /usr/lib64/libcurl.so.4.1.1)
==16459==    by 0xA069141: ??? (in /usr/lib64/libcurl.so.4.1.1)
==16459==    by 0xA0601C4: Curl_ssl_connect (in /usr/lib64/libcurl.so.4.1.1)

That's like almost 200KB lost in this report.

From the list of the modules, I see you have utils and I guess you use http query function from there, is it?

Cheers,
Daniel


On 10/10/16 12:06, Jurijs Ivolga wrote:
Hi Daniel,

I left valgrind running for little while, not sure if this will be enough.

Please find attached log file.

Thank you a lot for your help!

With kind regards,

Jurijs

On Fri, Oct 7, 2016 at 7:15 PM, Daniel-Constantin Mierla <miconda@gmail.com> wrote:

Hello,

that's the way it was done for older versions of kamailio.

In master and 4.4 the memory debugging is turned on and it is reflected by the presence of DBG_SR_MEMORY in the output of 'kamailio -v'.

Anyhow, what you reported is not a leak inside kamailio memory manager, but a leak of using system memory, so it is not affected by DBG_SR_MEMORY and cannot be troubleshooted using the mechanisms for pkg and shm managers.

Cheers,
Daniel



-- 
Daniel-Constantin Mierla
http://twitter.com/#!/miconda - http://www.linkedin.com/in/miconda
Kamailio Advanced Training, Berlin, Oct 24-26, 2016 - http://www.asipto.com
-- 
Daniel-Constantin Mierla
http://twitter.com/#!/miconda - http://www.linkedin.com/in/miconda
Kamailio Advanced Training, Berlin, Oct 24-26, 2016 - http://www.asipto.com
-- 
Daniel-Constantin Mierla
http://twitter.com/#!/miconda - http://www.linkedin.com/in/miconda
Kamailio Advanced Training, Berlin, Nov 28-30, 2016 - http://www.asipto.com