diff options
author | Nanang Izzuddin <nanang@teluu.com> | 2010-10-16 04:40:01 +0000 |
---|---|---|
committer | Nanang Izzuddin <nanang@teluu.com> | 2010-10-16 04:40:01 +0000 |
commit | 090871816ecb857b83e8f4381c7840a3647adb55 (patch) | |
tree | be09c15ff20ba7ec4980b85e357c46a4eb2712ca /pjmedia/src | |
parent | 2b502fb1a99fd8c73075594a2b40a05263f7b3ce (diff) |
Fixed #1148:
- Added check in processing answer, if media offer port is zero, just skip negotiation process.
- Added SIPp test scenario.
git-svn-id: http://svn.pjsip.org/repos/pjproject/trunk@3347 74dad513-b988-da41-8d7b-12977e46ad98
Diffstat (limited to 'pjmedia/src')
-rw-r--r-- | pjmedia/src/pjmedia/sdp_neg.c | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/pjmedia/src/pjmedia/sdp_neg.c b/pjmedia/src/pjmedia/sdp_neg.c index 974809d9..1087efc1 100644 --- a/pjmedia/src/pjmedia/sdp_neg.c +++ b/pjmedia/src/pjmedia/sdp_neg.c @@ -771,7 +771,6 @@ static pj_status_t process_m_answer( pj_pool_t *pool, /* Check if remote has rejected our offer */ - if (answer->desc.port == 0) { /* Remote has rejected our offer. @@ -783,6 +782,13 @@ static pj_status_t process_m_answer( pj_pool_t *pool, return PJ_SUCCESS; } + /* Ticket #1148: check if remote answer does not set port to zero when + * offered with port zero. Let's just tolerate it. + */ + if (offer->desc.port == 0) { + /* Don't need to proceed */ + return PJ_SUCCESS; + } /* Process direction attributes */ update_media_direction(pool, answer, offer); |