[asterisk-commits] kmoore: branch 1.8 r364786 - /branches/1.8/apps/app_confbridge.c

[SVN commits to the Asterisk project]

Author: kmoore
Date: Tue May  1 14:03:17 2012
New Revision: 364786

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=364786
Log:
Play conf-placeintoconf message to the correct channel

Correct the code in app_confbridge to play the conf-placeintoconf message to
the marked user entering the bridge instead of to the conference while the
marked user hears silence.

(closes issue ASTERISK-19641)
Reported-by: Mark A Walters

Modified:
    branches/1.8/apps/app_confbridge.c

Modified: branches/1.8/apps/app_confbridge.c
URL: http://svnview.digium.com/svn/asterisk/branches/1.8/apps/app_confbridge.c?view=diff&rev=364786&r1=364785&r2=364786
==============================================================================
--- branches/1.8/apps/app_confbridge.c (original)
+++ branches/1.8/apps/app_confbridge.c Tue May  1 14:03:17 2012
@@ -280,11 +280,12 @@
 
 		/* Next play the audio file stating they are going to be placed into the conference */
 		if (!ast_test_flag(&conference_bridge_user->flags, OPTION_QUIET)) {
-			ao2_unlock(conference_bridge);
-			ast_autoservice_start(conference_bridge_user->chan);
-			play_sound_file(conference_bridge, "conf-placeintoconf");
-			ast_autoservice_stop(conference_bridge_user->chan);
-			ao2_lock(conference_bridge);
+			if (play_prompt_to_channel(conference_bridge,
+				conference_bridge_user->chan,
+				"conf-placeintoconf")) {
+				/* user hungup while the sound was playing */
+				return -1;
+			}
 		}
 
 		/* Finally iterate through and unmute them all */