Merged revisions 114103 via svnmerge from
authorJoshua Colp <jcolp@digium.com>
Mon, 14 Apr 2008 14:53:33 +0000 (14:53 +0000)
committerJoshua Colp <jcolp@digium.com>
Mon, 14 Apr 2008 14:53:33 +0000 (14:53 +0000)
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r114103 | file | 2008-04-14 11:52:46 -0300 (Mon, 14 Apr 2008) | 4 lines

It is possible for the remote side to say they want T38 but not give any capabilities.
(closes issue #12414)
Reported by: MVF

........

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

channels/chan_sip.c

index cf758c4..52bca55 100644 (file)
@@ -6896,7 +6896,7 @@ static int process_sdp(struct sip_pvt *p, struct sip_request *req, int t38action
        }
        if (!newjointcapability) {
                /* If T.38 was not negotiated either, totally bail out... */
-               if (!p->t38.jointcapability || !p->t38.peercapability) {
+               if (!p->t38.jointcapability || !udptlportno) {
                        ast_log(LOG_NOTICE, "No compatible codecs, not accepting this offer!\n");
                        /* Do NOT Change current setting */
                        return -1;