chan_sip: Fix negotiation of iLBC 30.

iLBC 20 was advertised in a SIP/SDP negotiation. However, only iLBC 30 is
supported. Removes "a=fmtp:x mode=y" from SDP. Because of RFC 3952 section 5,
only iLBC 30 is negotiated now.

ASTERISK-25309 #close

Change-Id: I92d724600a183eec3114da0ac607b994b1a793da

diff --git a/channels/chan_sip.c b/channels/chan_sip.c
index 9ba0e192b942edf496046793ae29ca8cf2b4fe99..1f45339e4b24b5390c10f8713e3870fa2194b617 100644 (file)
--- a/channels/chan_sip.c
+++ b/channels/chan_sip.c
@@ -12811,9 +12811,6 @@ static void add_codec_to_sdp(const struct sip_pvt *p,
        } else if (ast_format_cmp(format, ast_format_g723) == AST_FORMAT_CMP_EQUAL) {
                /* Indicate that we don't support VAD (G.723.1 annex A) */
                ast_str_append(a_buf, 0, "a=fmtp:%d annexa=no\r\n", rtp_code);
-       } else if (ast_format_cmp(format, ast_format_ilbc) == AST_FORMAT_CMP_EQUAL) {
-               /* Add information about us using only 20/30 ms packetization */
-               ast_str_append(a_buf, 0, "a=fmtp:%d mode=%u\r\n", rtp_code, framing);
        } else if (ast_format_cmp(format, ast_format_siren7) == AST_FORMAT_CMP_EQUAL) {
                /* Indicate that we only expect 32Kbps */
                ast_str_append(a_buf, 0, "a=fmtp:%d bitrate=32000\r\n", rtp_code);