diff options
| author | Matthew Jordan <mjordan@digium.com> | 2013-05-17 17:43:58 +0000 |
|---|---|---|
| committer | Matthew Jordan <mjordan@digium.com> | 2013-05-17 17:43:58 +0000 |
| commit | d04f1fd60a5f5a6e358fbe5a59ae8f42d21cbbdf (patch) | |
| tree | fd3e01c6b7b34f35f5d020bbfa641898de104147 /main/dial.c | |
| parent | b90bba7a303bf57c3c874a1c8f506d39d4e78a9c (diff) | |
Publish the outbound channel's application/data when dialing
This patch does two things:
* It fixes a bug where the outbound channel's application/data set by the
dialing API/app_dial is not communicated until the channel is hung up.
If that happens, AMI would incorrectly send a NewExten event immediately
after a Hangup. This isn't really AMI's fault, as the dialing APIs never
communicated the 'helpful' app/data on the outbound channel until it was
hungup.
* It makes public sending a stasis message about a change in channel state.
This is useful enough that - for now at least - it should be public. If
operations on a channel go to being more coarse-grained, this function
could be made private again.
Review: https://reviewboard.asterisk.org/r/2548
Note that this problem was found and reported by Matt DiMeo.
git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@388976 65c4cc65-6c06-0410-ace0-fbb531ad65f3
Diffstat (limited to 'main/dial.c')
| -rw-r--r-- | main/dial.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/main/dial.c b/main/dial.c index f8b0c9787..da02f7a23 100644 --- a/main/dial.c +++ b/main/dial.c @@ -287,6 +287,7 @@ static int begin_dial_channel(struct ast_dial_channel *channel, struct ast_chann ast_channel_appl_set(channel->owner, "AppDial2"); ast_channel_data_set(channel->owner, "(Outgoing Line)"); + ast_publish_channel_state(channel->owner); memset(ast_channel_whentohangup(channel->owner), 0, sizeof(*ast_channel_whentohangup(channel->owner))); /* Inherit everything from he who spawned this dial */ |