<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Oct 3, 2022 at 11:41 AM mayamatakeshi <<a href="mailto:mayamatakeshi@gmail.com">mayamatakeshi@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Sep 26, 2022 at 8:32 PM Daniel-Constantin Mierla <<a href="mailto:miconda@gmail.com" target="_blank">miconda@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hello,<br>
<br>
as I am not a user of dialog variables, I am turning to community to ask<br>
for help testing the current master branch with configurations that make<br>
use of dialog variables and acc dialog-based cdr generation.<br>
<br>
With a few reports of issues related to dialog modules and unexpected<br>
crashes, I looked over the code and noticed that the access of the value<br>
for dialog variables was not protected, making them vulnerable of<br>
invalid memory access in case of the variable was updated by another<br>
process or dialog was terminated.<br>
<br>
I introduced a couple of new functions to try to cover different use<br>
cases of getting the dlg variable values, dialog management code was not<br>
affected, but given that these commits need to be backported to stable<br>
branch (5.6), I want to get proper feedback from community that things<br>
work fine.<br>
<br>
A previous attempt of a simpler fix was not enough, having side effects<br>
to acc module for dialog-based cdr generation, because it was keeping<br>
referenced to many dlg variables at the same time.<br>
<br>
In short, it would be appreciated any feedback on testing dialog and acc<br>
with dialog-based cdr generation using git master branch.<br><br></blockquote><div><br></div><div>I have started 4 load test environments today with latest commit 6f400a8074fe60916867596431ca26dff00435d1.</div><div>I usually leave a commit load test running for 2 months before consider it ready for production release.</div><div>I will report any crash/problem.</div></div></div></blockquote><div><br></div><div>After a few hours of load test, all 4 load test environments start to log memory allocation problems:</div><div><br></div>[root@lab002107-flip-server ~]$ grep memory /var/log/kamailio/kamailio.log |head<br>2022-10-13T01:36:10.429809+09:00 lab002107-flip-server /usr/local/src/git/kamailio-master/src/kamailio[3286370]: ERROR: rtpengine [rtpengine.c:2721]: rtpp_function_call(): out of memory - bencode failed<br>2022-10-13T01:36:12.923609+09:00 lab002107-flip-server /usr/local/src/git/kamailio-master/src/kamailio[3286370]: ERROR: rtpengine [rtpengine.c:2721]: rtpp_function_call(): out of memory - bencode failed<br>2022-10-13T01:36:12.961677+09:00 lab002107-flip-server /usr/local/src/git/kamailio-master/src/kamailio[3286370]: ERROR: acc [acc_extra.c:234]: extra2strar(): could not allocate private memory from pkg pool<br>2022-10-13T01:36:14.983281+09:00 lab002107-flip-server /usr/local/src/git/kamailio-master/src/kamailio[3286370]: ERROR: rtpengine [rtpengine.c:2721]: rtpp_function_call(): out of memory - bencode failed<br>2022-10-13T01:36:14.983537+09:00 lab002107-flip-server /usr/local/src/git/kamailio-master/src/kamailio[3286370]: ERROR: acc [acc_extra.c:234]: extra2strar(): could not allocate private memory from pkg pool<br>2022-10-13T01:36:14.983665+09:00 lab002107-flip-server /usr/local/src/git/kamailio-master/src/kamailio[3286370]: ERROR: acc [acc_extra.c:234]: extra2strar(): could not allocate private memory from pkg pool<br>2022-10-13T01:36:20.861558+09:00 lab002107-flip-server /usr/local/src/git/kamailio-master/src/kamailio[3286370]: ERROR: rtpengine [rtpengine.c:2721]: rtpp_function_call(): out of memory - bencode failed<br>2022-10-13T01:36:20.864388+09:00 lab002107-flip-server /usr/local/src/git/kamailio-master/src/kamailio[3286370]: ERROR: acc [acc_extra.c:234]: extra2strar(): could not allocate private memory from pkg pool<br>2022-10-13T01:36:20.878469+09:00 lab002107-flip-server /usr/local/src/git/kamailio-master/src/kamailio[3286370]: ERROR: acc [acc_extra.c:234]: extra2strar(): could not allocate private memory from pkg pool<br><div>2022-10-13T01:36:23.174159+09:00 lab002107-flip-server /usr/local/src/git/kamailio-master/src/kamailio[3286370]: ERROR: rtpengine [rtpengine.c:2721]: rtpp_function_call(): out of memory - bencode failed </div><div><br></div><div><br></div><div>I reverted 2 of the load test envs to previous kamailio 5.6 commit 61e86a1f502388ffd4dce6e52811ba640337c813 and restarted the load tests, then again, kamailio master commit 6f400a8074fe60916867596431ca26dff00435d1 started to write the above logs but this doesn't happen with 5.6.</div><div><br></div><div><br></div><div><br></div></div></div>