Hi, All.
I have some problems with pkg_memory leak.
After each call used private memory of the "udp receiver" process increases at 2500 bytes. On test environment I hav set it to 2Mbyte, but it is full after 700 calls.
I need to modify SDP in invite and other packets. When I do in script
====================================================
if(has_body("application/sdp")) set_body_multipart(); if (msg_apply_changes()) { xlog("L_INFO", "ISUP 1 Changes Applied Succesfully"); } }
record_route(); dlg_manage route(RELAY); ==== I make test this on kamailio 4.3.5 and 4.4.0 - same issue.
After discussion in mailing list with Daniel-Constantin Mierla, I have done debugging memory usage by process and found 3 points where memory hase not freeing after execution.
This patch solves it for me. You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/587
-- Commit Summary --
* Update textops.c * Update msg_translator.c
-- File Changes --
M modules/textops/textops.c (2) M msg_translator.c (7)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/587.patch https://github.com/kamailio/kamailio/pull/587.diff
--- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/587
Thanks, the patches were applied manually 2b427a33937c6a86ab231de39d5a4574309dbfc6 61435f40f0f4ca25521f7f1b8515e47954a69b2d to format properly the commit message and also use tabs for indentation of the code.
For future, it would be good to use tabs and format the commit messages like:
* https://www.kamailio.org/wiki/devel/github-contributions#commit_message_form...
It will make the merge straightforward.
--- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/587#issuecomment-215557875
Closed #587.
--- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/587#event-644877513