<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">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>