[asterisk-dev] [Code Review] another take on the asyc_goto lock inversion issue.

David Vossel reviewboard at asterisk.org
Fri Jun 17 10:46:37 CDT 2011

This is an automatically generated e-mail. To reply, visit:

Review request for Asterisk Developers and kkm.


This is in response to review 1274.  kkm identified an issue with async_goto, and as I was reviewing it I realized the problem is much deeper than I can easily comment and give direction on in the review.  It wasn't much code so I just wrote what I was thinking instead.

We can not hold the "chan" lock while doing the masquerade, the explicit goto on the tmp chan, or the channel alloc.  Nearly the entire function is wrong.  Instead we need to get the channel lock, store off information about the channel that we need, and then let the channel lock go for the remainder of the function.

This addresses bug ASTERISK-18031.


  /branches/1.8/include/asterisk/pbx.h 324047 
  /branches/1.8/main/pbx.c 324047 

Diff: https://reviewboard.asterisk.org/r/1275/diff


I have not tested this.



-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20110617/4a75211d/attachment.htm>

More information about the asterisk-dev mailing list