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

jrose reviewboard at asterisk.org
Mon Mar 14 12:43:46 CDT 2011


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

Review request for Asterisk Developers.


Summary
-------

Changes channel.c so that the call hangup function 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/1466d090/attachment.htm>


More information about the asterisk-dev mailing list