diff options
| author | dvossel <dvossel@f38db490-d61c-443f-a65b-d21fe96a405b> | 2011-06-28 15:34:59 +0000 |
|---|---|---|
| committer | dvossel <dvossel@f38db490-d61c-443f-a65b-d21fe96a405b> | 2011-06-28 15:34:59 +0000 |
| commit | c2c84f23b0746d1c133faf6de647a979754b687e (patch) | |
| tree | 0155023e756331e8fae7adca264a791db996e566 /channels/chan_sip.c | |
| parent | 7a3481e1608ff4b1e7ea01538e39c7b8d167df10 (diff) | |
Fixes issue with video and text not being reinvited correctly with directmedia
If a SDP does not modify the session, we ignore it. However, we were defaulting
no text and video support to true before checking to see if the sdp modified
anything or not. This would result in process_sdp ignoring an sdp but removing
video and text from the call during direct media reinvites.
git-svn-id: http://svn.digium.com/svn/asterisk/trunk@325151 f38db490-d61c-443f-a65b-d21fe96a405b
Diffstat (limited to 'channels/chan_sip.c')
| -rw-r--r-- | channels/chan_sip.c | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/channels/chan_sip.c b/channels/chan_sip.c index 1bfe85bd6..f3a969316 100644 --- a/channels/chan_sip.c +++ b/channels/chan_sip.c @@ -8719,11 +8719,6 @@ static int process_sdp(struct sip_pvt *p, struct sip_request *req, int t38action memset(p->offered_media, 0, sizeof(p->offered_media)); - - /* default: novideo and notext set */ - p->novideo = TRUE; - p->notext = TRUE; - if (p->vrtp) { ast_rtp_codecs_payloads_clear(&newvideortp, NULL); } @@ -8781,7 +8776,9 @@ static int process_sdp(struct sip_pvt *p, struct sip_request *req, int t38action ast_debug(3, "Processing session-level SDP %c=%s... %s\n", type, value, (processed == TRUE)? "OK." : "UNSUPPORTED."); } - + /* default: novideo and notext set */ + p->novideo = TRUE; + p->notext = TRUE; /* Scan media stream (m=) specific parameters loop */ while (!ast_strlen_zero(nextm)) { |