diff options
| author | Richard Mudgett <rmudgett@digium.com> | 2014-04-18 16:27:31 +0000 |
|---|---|---|
| committer | Richard Mudgett <rmudgett@digium.com> | 2014-04-18 16:27:31 +0000 |
| commit | cbe7f656749f5cf7e4d8321ef74a99c9281a2284 (patch) | |
| tree | 08eaecfa4a7d3e1693aa6c18e808b069f1e6aea0 /main/stasis_channels.c | |
| parent | 9a85fc0aa0f471fe3ea5bfa1496239044d8d8de6 (diff) | |
app_dial and app_queue: Make lock the forwarding channel while taking the channel snapshot.
* Fixed ast_channel_publish_dial_forward() not locking the forwarded
channel when taking the channel snapshot.
* Fixed app_dial.c:do_forward() using the wrong channel to get the
original call forwarding string.
* Removed unnecessary locking when calling ast_channel_publish_dial() and
ast_channel_publish_dial_forward() in app_dial and app_queue. Holding
channel locks when calling ast_channel_publish_dial_forward() with a
forwarded channel could result in pausing the system while the stasis bus
completes processsing a forwarded channel subscription.
Review: https://reviewboard.asterisk.org/r/3451/
........
Merged revisions 412579 from http://svn.asterisk.org/svn/asterisk/branches/12
git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@412580 65c4cc65-6c06-0410-ace0-fbb531ad65f3
Diffstat (limited to 'main/stasis_channels.c')
| -rw-r--r-- | main/stasis_channels.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/main/stasis_channels.c b/main/stasis_channels.c index eefb7a9f8..3522fb291 100644 --- a/main/stasis_channels.c +++ b/main/stasis_channels.c @@ -336,7 +336,9 @@ void ast_channel_publish_dial_forward(struct ast_channel *caller, struct ast_cha ast_multi_channel_blob_add_channel(payload, "peer", peer_snapshot); if (forwarded) { + ast_channel_lock(forwarded); forwarded_snapshot = ast_channel_snapshot_create(forwarded); + ast_channel_unlock(forwarded); if (!forwarded_snapshot) { return; } |