[asterisk-commits] file: branch 12 r400256 - /branches/12/res/res_pjsip_session.c

[SVN commits to the Asterisk project]

Author: file
Date: Wed Oct  2 10:31:31 2013
New Revision: 400256

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=400256
Log:
Fix a random one way audio issue in PJSIP.

Due to the asynchronous design of the PJMEDIA SDP negotiator it was possible for
the SDP to be negotiated *after* a channel was created and after it was being wait
on by an application. It is only after negotiation occurs that the file descriptors
for RTP are placed on the channel. Since the channel was already being waited on
these file descriptors were not monitored, causing incoming media to never be read.

This change wakes up any application waiting on the channel so that added file
descriptors end up being monitored.

(closes issue AST-1227)
Reported by: John Bigelow

Modified:
    branches/12/res/res_pjsip_session.c

Modified: branches/12/res/res_pjsip_session.c
URL: http://svnview.digium.com/svn/asterisk/branches/12/res/res_pjsip_session.c?view=diff&rev=400256&r1=400255&r2=400256
==============================================================================
--- branches/12/res/res_pjsip_session.c (original)
+++ branches/12/res/res_pjsip_session.c Wed Oct  2 10:31:31 2013
@@ -472,6 +472,7 @@
 	successful = ao2_callback(session->media, OBJ_MULTIPLE, handle_negotiated_sdp_session_media, &callback_data);
 	if (successful && ao2_container_count(successful->c) == ao2_container_count(session->media)) {
 		/* Nothing experienced a catastrophic failure */
+		ast_queue_frame(session->channel, &ast_null_frame);
 		return 0;
 	}
 	return -1;