diff options
author | Benny Prijono <bennylp@teluu.com> | 2007-03-15 21:15:16 +0000 |
---|---|---|
committer | Benny Prijono <bennylp@teluu.com> | 2007-03-15 21:15:16 +0000 |
commit | a3095ed444a54cf42b5035a66187bc48493b8e53 (patch) | |
tree | e0dec4b5952b79be72b0b1016ebbd04c0449ab27 /pjmedia/src | |
parent | 9e7cba424b089b419f0eef754ca2958644a90a97 (diff) |
Fixed ticket #182: SDP negotiation failed when local does not specify telephone-event (thanks Esbjorn Dominique)
git-svn-id: http://svn.pjsip.org/repos/pjproject/trunk@1070 74dad513-b988-da41-8d7b-12977e46ad98
Diffstat (limited to 'pjmedia/src')
-rw-r--r-- | pjmedia/src/pjmedia/sdp_neg.c | 15 |
1 files changed, 4 insertions, 11 deletions
diff --git a/pjmedia/src/pjmedia/sdp_neg.c b/pjmedia/src/pjmedia/sdp_neg.c index aab409e3..40918f7c 100644 --- a/pjmedia/src/pjmedia/sdp_neg.c +++ b/pjmedia/src/pjmedia/sdp_neg.c @@ -803,28 +803,21 @@ static pj_status_t match_offer(pj_pool_t *pool, } /* See if all types of offer can be matched. */ -#if 1 if (offer_has_codec && !found_matching_codec) { return PJMEDIA_SDPNEG_NOANSCODEC; } + /* If this comment is removed, negotiation will fail if remote has offered + telephone-event and local is not configured with telephone-event + if (offer_has_telephone_event && !found_matching_telephone_event) { return PJMEDIA_SDPNEG_NOANSTELEVENT; } + */ if (offer_has_other && !found_matching_other) { return PJMEDIA_SDPNEG_NOANSUNKNOWN; } -#else - PJ_TODO(FULL_MATCHING_WITH_TELEPHONE_EVENTS); - if (!found_matching_codec && - !found_matching_telephone_event && - !found_matching_other) - { - /* Some of the payload in the offer has no matching local sdp */ - return PJ_FALSE; - } -#endif /* Seems like everything is in order. * Build the answer by cloning from local media, but rearrange the payload |