From: Kinsey Moore <kmoore@digium.com>
Date: Fri, 12 Dec 2014 14:11:38 +0000 (+0000)
Subject: PJSIP: Allow use of 'inactive' streams for hold
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=8b34ef65cbc6e8f29ed5cb8eb4f9a9cff38c575b;p=thirdparty%2Fasterisk.git

PJSIP: Allow use of 'inactive' streams for hold

This allows use of the 'inactive' stream direction identifier to be
used for hold where 'sendonly' is normally used. Some Seimens phones
use 'inactive' and this change allows music on hold to operate
properly.

Review: https://reviewboard.asterisk.org/r/4252/
Reported by: Steve Pitts


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/12@429432 65c4cc65-6c06-0410-ace0-fbb531ad65f3
---

diff --git a/res/res_pjsip_sdp_rtp.c b/res/res_pjsip_sdp_rtp.c
index 191ff73e01..5049327e51 100644
--- a/res/res_pjsip_sdp_rtp.c
+++ b/res/res_pjsip_sdp_rtp.c
@@ -1144,7 +1144,8 @@ static int apply_negotiated_sdp_stream(struct ast_sip_session *session, struct a
 
 	if (ast_sockaddr_isnull(addrs) ||
 		ast_sockaddr_is_any(addrs) ||
-		pjmedia_sdp_media_find_attr2(remote_stream, "sendonly", NULL)) {
+		pjmedia_sdp_media_find_attr2(remote_stream, "sendonly", NULL) ||
+		pjmedia_sdp_media_find_attr2(remote_stream, "inactive", NULL)) {
 		if (!session_media->held) {
 			/* The remote side has put us on hold */
 			ast_queue_hold(session->channel, session->endpoint->mohsuggest);