[asterisk-dev] [Code Review] Data Store Destruction moved to ast_hangup

David Vossel reviewboard at asterisk.org
Mon Mar 14 15:47:41 CDT 2011



> On 2011-03-14 15:15:53, Russell Bryant wrote:
> >

I don't understand how chanspy prevents channels from being destroyed.  Is it possible for them to exist indefinitely?  If so this sounds more like a memory leak.


- David


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/1136/#review3242
-----------------------------------------------------------


On 2011-03-14 12:44:27, jrose wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/1136/
> -----------------------------------------------------------
> 
> (Updated 2011-03-14 12:44:27)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Summary
> -------
> 
> Changes channel.c so that the call hangup function (instead of channel being closed) is responsible for destroying application data stores and signalling to the application thread that it can continue it's destruction event.
> 
> This was necessary because applications like chanspy could prevent channels from closing under certain conditions, causing the threads to become stuck and never complete.
> 
> 
> This addresses bug https://issues.asterisk.org/view.php?id=18742.
>     https://issues.asterisk.org/view.php?id=https://issues.asterisk.org/view.php?id=18742
> 
> 
> Diffs
> -----
> 
>   /trunk/main/channel.c 310588 
> 
> Diff: https://reviewboard.asterisk.org/r/1136/diff
> 
> 
> Testing
> -------
> 
> Rudimentary testing was done using dialplans similar to what was suggested in the noted bug report.  In addition, initial response to the patch suggests it works.
> 
> 
> Thanks,
> 
> jrose
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20110314/cd8e1bc5/attachment-0001.htm>


More information about the asterisk-dev mailing list