mirror of
				https://github.com/asterisk/asterisk.git
				synced 2025-10-31 02:37:10 +00:00 
			
		
		
		
	audiohooks: Reevaluate the bridge technology when an audiohook is added or removed.
Adding a mixmonitor to a channel causes the bridge to change technologies from native to simple_bridge so the call can be recorded. However, when the mixmonitor is stopped the bridge does not switch back to the native technology. * Added unbridge requests to reevaluate the bridge when a channel audiohook is removed. * Moved the unbridge request into ast_audiohook_attach() ensure that the bridge reevaluates whenever an audiohook is attached. This simplified the mixmonitor and chan_spy start code as well. * Added defensive code to stop_mixmonitor_full() in case additional arguments are ever added to the StopMixMonitor application. * Made ast_framehook_detach() not do an unbridge request if the framehook does not exist. * Made ast_framehook_list_fixup() do an unbridge request if there are any framehooks. Also simplified the loop. ASTERISK-24195 #close Reported by: Jonathan Rose Review: https://reviewboard.asterisk.org/r/4046/ ........ Merged revisions 424506 from http://svn.asterisk.org/svn/asterisk/branches/12 ........ Merged revisions 424507 from http://svn.asterisk.org/svn/asterisk/branches/13 git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@424508 65c4cc65-6c06-0410-ace0-fbb531ad65f3
This commit is contained in:
		| @@ -498,21 +498,10 @@ static struct ast_generator spygen = { | ||||
|  | ||||
| static int start_spying(struct ast_autochan *autochan, const char *spychan_name, struct ast_audiohook *audiohook) | ||||
| { | ||||
| 	int res = 0; | ||||
|  | ||||
| 	ast_log(LOG_NOTICE, "Attaching %s to %s\n", spychan_name, ast_channel_name(autochan->chan)); | ||||
|  | ||||
| 	ast_set_flag(audiohook, AST_AUDIOHOOK_TRIGGER_SYNC | AST_AUDIOHOOK_SMALL_QUEUE); | ||||
| 	res = ast_audiohook_attach(autochan->chan, audiohook); | ||||
|  | ||||
| 	if (!res) { | ||||
| 		ast_channel_lock(autochan->chan); | ||||
| 		if (ast_channel_is_bridged(autochan->chan)) { | ||||
| 			ast_channel_set_unbridged_nolock(autochan->chan, 1); | ||||
| 		} | ||||
| 		ast_channel_unlock(autochan->chan); | ||||
| 	} | ||||
| 	return res; | ||||
| 	return ast_audiohook_attach(autochan->chan, audiohook); | ||||
| } | ||||
|  | ||||
| static void change_spy_mode(const char digit, struct ast_flags *flags) | ||||
|   | ||||
		Reference in New Issue
	
	Block a user