bridge: Don't try to match audio formats.

When bridging channels we were trying to match the audio
formats of both sides in combination with the configured
formats. While this is allowed in SDP in practice this
causes extra reinvites and problems. This change ensures
that audio streams use the formats of the first existing
active audio stream. It is only when other stream types
(like video) exist that this will result in re-negotiation
occurring for those streams only.

ASTERISK-28871

Change-Id: I22f5a3e7db29e00c165e74d05d10856f6086fe47

diff --git a/bridges/bridge_native_rtp.c b/bridges/bridge_native_rtp.c
index a6addf27110391fbfebb56ba8bc1eab5ec296486..2853b990cbba8d353f26528e18418dca98c93a1f 100644 (file)
--- a/bridges/bridge_native_rtp.c
+++ b/bridges/bridge_native_rtp.c
@@ -893,8 +893,8 @@ static struct ast_stream_topology *native_rtp_request_stream_topology_update(
                                continue;
                        }
 
-                       ast_format_cap_append_from_cap(ast_stream_get_formats(stream), audio_formats,
-                               AST_MEDIA_TYPE_AUDIO);
+                       /* We haven't actually modified audio_formats so this is safe */
+                       ast_stream_set_formats(stream, (struct ast_format_cap *)audio_formats);
                }
        }
 

diff --git a/bridges/bridge_simple.c b/bridges/bridge_simple.c
index 545b3ad1cbe04be718e22dfe757ac239184d1ff6..1750d2739028160b73733f2a9b55c61b314603b9 100644 (file)
--- a/bridges/bridge_simple.c
+++ b/bridges/bridge_simple.c
@@ -97,8 +97,8 @@ static struct ast_stream_topology *simple_bridge_request_stream_topology_update(
                                continue;
                        }
 
-                       ast_format_cap_append_from_cap(ast_stream_get_formats(stream), audio_formats,
-                               AST_MEDIA_TYPE_AUDIO);
+                       /* We haven't actually modified audio_formats so this is safe */
+                       ast_stream_set_formats(stream, (struct ast_format_cap *)audio_formats);
                }
        }