<p>Patch set 2:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #d4ffd4;">Code-Review +1</span></p><p><a href="https://gerrit.asterisk.org/10382">View Change</a></p><p>2 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.asterisk.org/#/c/10382/2/res/res_smdi.c">File res/res_smdi.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/#/c/10382/2/res/res_smdi.c@a246">Patch Set #2, Line 246:</a> <code style="font-family:monospace,monospace"></code></p><p style="white-space: pre-wrap; word-wrap: break-word;">In 2014 I replaced astobj with ao2 and failed to remove the invalid call to free here.  This was first released in 13.0.0 and nobody reported this crash.  Not sure if this means that it's impossible for interfaces to be cleaned or if it means that literally nobody uses this module.</p><p style="white-space: pre-wrap; word-wrap: break-word;">https://reviewboard.asterisk.org/r/3758/</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/#/c/10382/2/res/res_smdi.c@249">Patch Set #2, Line 249:</a> <code style="font-family:monospace,monospace">            ast_module_unref(ast_module_info->self);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">I think the interface/thread should not hold a reference to the module, we should just use ast_module_shutdown_ref in load_module.  This would allow graceful shutdown without allowing the module to be dlclose'd with active threads.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Honestly I don't care about this module and think it should be considered for deletion so feel free to ignore this comment.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/10382">change 10382</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/10382"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: Ic288db51b58e395d6a2fc3847f77176c16988784 </div>
<div style="display:none"> Gerrit-Change-Number: 10382 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: Richard Mudgett <rmudgett@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Corey Farrell <git@cfware.com> </div>
<div style="display:none"> Gerrit-Reviewer: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 (1000185) </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 03 Oct 2018 12:29:32 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: Yes </div>