From 6777ec03406e8b10c5bb8bce5bbb47a7f7f1bdbb Mon Sep 17 00:00:00 2001 From: "Joshua C. Colp" Date: Wed, 17 Jun 2020 05:58:44 -0300 Subject: [PATCH] res_pjsip_session: Preserve label on incoming re-INVITE. When a re-INVITE is received we create a new set of streams that are then swapped in as the active streams. We did not preserve the SDP label from the previous streams, resulting in the label getting lost. This change ensures that if an SDP label is present on the previous stream then it is set on the new stream. ASTERISK-28953 Change-Id: I9dd63b88b562fe96ce5c791a3dae5bcaca258445 --- res/res_pjsip_session.c | 16 ++++++++++++++++ 1 file changed, 16 insertions(+) diff --git a/res/res_pjsip_session.c b/res/res_pjsip_session.c index 1bbc6fa0bc..d05f2c57f5 100644 --- a/res/res_pjsip_session.c +++ b/res/res_pjsip_session.c @@ -760,6 +760,14 @@ static int handle_incoming_sdp(struct ast_sip_session *session, const pjmedia_sd ast_stream_free(stream); return -1; } + if (existing_stream) { + const char *stream_label = ast_stream_get_metadata(existing_stream, "SDP:LABEL"); + + if (!ast_strlen_zero(stream_label)) { + ast_stream_set_metadata(stream, "SDP:LABEL", stream_label); + } + } + /* For backwards compatibility with the core the default audio stream is always sendrecv */ if (!ast_sip_session_is_pending_stream_default(session, stream) || strcmp(media, "audio")) { if (pjmedia_sdp_media_find_attr2(remote_stream, "sendonly", NULL)) { @@ -1964,6 +1972,14 @@ static int sdp_requires_deferral(struct ast_sip_session *session, const pjmedia_ return -1; } + if (existing_stream) { + const char *stream_label = ast_stream_get_metadata(existing_stream, "SDP:LABEL"); + + if (!ast_strlen_zero(stream_label)) { + ast_stream_set_metadata(stream, "SDP:LABEL", stream_label); + } + } + session_media = ast_sip_session_media_state_add(session, session->pending_media_state, ast_media_type_from_str(media), i); if (!session_media) { return -1; -- 2.47.2