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/branches/1.4@114103 65c4cc65-6c06-0410-ace0-fbb531ad65f3

diff --git a/channels/chan_sip.c b/channels/chan_sip.c
index c21815e231a1ae2a5cdeceb5359053d7edc4d4fa..03f5ae091dbfcb3fe76d28a8bc183598cd7f86b3 100644 (file)
--- a/channels/chan_sip.c
+++ b/channels/chan_sip.c
@@ -5489,7 +5489,7 @@ static int process_sdp(struct sip_pvt *p, struct sip_request *req)
        }
        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;