<p>Kevin Harwell <strong>uploaded patch set #2</strong> to this change.</p><p><a href="https://gerrit.asterisk.org/c/asterisk/+/15274">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">app_mixmonitor: cleanup datastore when monitor thread fails to launch<br><br>launch_monitor_thread is responsible for creating and initializing<br>the mixmonitor, and dependent data structures. There was one off<br>nominal path after the datastore gets created that triggers when<br>the channel being monitored is hung up prior to monitor starting<br>itself.<br><br>If this happened the monitor thread would not "launch", and the<br>mixmonitor object and associated objects are freed, including the<br>underlying datastore data object. However, the datastore itself was<br>not removed from the channel, so when the channel eventually gets<br>destroyed it tries to access the previously freed datastore data<br>and crashes.<br><br>This patch removes and frees datastore object itself from the channel<br>before freeing the mixmonitor object thus ensuring the channel does<br>not call it when destroyed.<br><br>ASTERISK-28947 #close<br><br>Change-Id: Id4f9e958956d62473ed5ff06c98ae3436e839ff8<br>---<br>M apps/app_mixmonitor.c<br>1 file changed, 22 insertions(+), 1 deletion(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/74/15274/2</pre><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/15274">change 15274</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.asterisk.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.asterisk.org/c/asterisk/+/15274"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: 18 </div>
<div style="display:none"> Gerrit-Change-Id: Id4f9e958956d62473ed5ff06c98ae3436e839ff8 </div>
<div style="display:none"> Gerrit-Change-Number: 15274 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: Kevin Harwell <kharwell@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-MessageType: newpatchset </div>