mirror of
https://github.com/asterisk/asterisk.git
synced 2025-11-03 04:16:43 +00:00
Merged revisions 157306 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk ................ r157306 | mmichelson | 2008-11-18 12:31:08 -0600 (Tue, 18 Nov 2008) | 20 lines Merged revisions 157305 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r157305 | mmichelson | 2008-11-18 12:25:55 -0600 (Tue, 18 Nov 2008) | 12 lines Fix a crash in the end_bridge_callback of app_dial and app_followme which would occur at the end of an attended transfer. The error occurred because we initially stored a pointer to an ast_channel which then was hung up due to a masquerade. This commit adds a "fixup" callback to the bridge_config structure to allow for end_bridge_callback_data to be changed in the case that a new channel pointer is needed for the end_bridge_callback. ........ ................ git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.6.1@157308 65c4cc65-6c06-0410-ace0-fbb531ad65f3
This commit is contained in:
@@ -1282,6 +1282,10 @@ static void end_bridge_callback(void *data)
|
||||
ast_channel_unlock(chan);
|
||||
}
|
||||
|
||||
static void end_bridge_callback_data_fixup(struct ast_bridge_config *bconfig, struct ast_channel *originator, struct ast_channel *terminator) {
|
||||
bconfig->end_bridge_callback_data = originator;
|
||||
}
|
||||
|
||||
static int dial_exec_full(struct ast_channel *chan, void *data, struct ast_flags64 *peerflags, int *continue_exec)
|
||||
{
|
||||
int res = -1; /* default: error */
|
||||
@@ -1962,7 +1966,8 @@ static int dial_exec_full(struct ast_channel *chan, void *data, struct ast_flags
|
||||
|
||||
config.end_bridge_callback = end_bridge_callback;
|
||||
config.end_bridge_callback_data = chan;
|
||||
|
||||
config.end_bridge_callback_data_fixup = end_bridge_callback_data_fixup;
|
||||
|
||||
if (moh) {
|
||||
moh = 0;
|
||||
ast_moh_stop(chan);
|
||||
|
||||
@@ -892,6 +892,11 @@ static void end_bridge_callback(void *data)
|
||||
ast_channel_unlock(chan);
|
||||
}
|
||||
|
||||
static void end_bridge_callback_data_fixup(struct ast_bridge_config *bconfig, struct ast_channel *originator, struct ast_channel *terminator)
|
||||
{
|
||||
bconfig->end_bridge_callback_data = originator;
|
||||
}
|
||||
|
||||
static int app_exec(struct ast_channel *chan, void *data)
|
||||
{
|
||||
struct fm_args targs;
|
||||
@@ -1012,6 +1017,7 @@ static int app_exec(struct ast_channel *chan, void *data)
|
||||
ast_set_flag(&(config.features_caller), AST_FEATURE_AUTOMON);
|
||||
config.end_bridge_callback = end_bridge_callback;
|
||||
config.end_bridge_callback_data = chan;
|
||||
config.end_bridge_callback_data_fixup = end_bridge_callback_data_fixup;
|
||||
|
||||
ast_moh_stop(caller);
|
||||
/* Be sure no generators are left on it */
|
||||
|
||||
Reference in New Issue
Block a user