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

[jrose]

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

(Updated 2011-03-14 12:44:27.377616)


Review request for Asterisk Developers.


Summary (updated)
-------

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/9230d5a5/attachment.htm>