Skip to content
Snippets Groups Projects
Commit 90ef6a14 authored by Torrey Searle's avatar Torrey Searle Committed by George Joseph
Browse files

res/res_rtp_asterisk: generate new SSRC on native bridge end

For RTCP to work, we update the ssrc to be the one corresponding to
the native bridge while active.  However when the bridge ends we
should generate a new SSRC as the sequence numbers will not continue
from the native bridge left off.

ASTERISK-29300 #close

Change-Id: I23334b6934d2bf6490bda4bbf6414d96b8d17d10
parent a9acbd19
No related branches found
No related tags found
No related merge requests found
......@@ -362,6 +362,8 @@ struct ast_rtp {
struct ast_frame f;
unsigned char rawdata[8192 + AST_FRIENDLY_OFFSET];
unsigned int ssrc; /*!< Synchronization source, RFC 3550, page 10. */
unsigned int ssrc_orig; /*!< SSRC used before native bridge activated */
unsigned char ssrc_saved; /*!< indicates if ssrc_orig has a value */
char cname[AST_UUID_STR_LEN]; /*!< Our local CNAME */
unsigned int themssrc; /*!< Their SSRC */
unsigned int themssrc_valid; /*!< True if their SSRC is available. */
......@@ -8530,6 +8532,18 @@ static int ast_rtp_local_bridge(struct ast_rtp_instance *instance0, struct ast_r
ast_smoother_free(rtp->smoother);
rtp->smoother = NULL;
}
/* We must use a new SSRC when local bridge ends */
if (!instance1) {
rtp->ssrc = rtp->ssrc_orig;
rtp->ssrc_orig = 0;
rtp->ssrc_saved = 0;
} else if (!rtp->ssrc_saved) {
/* In case ast_rtp_local_bridge is called multiple times, only save the ssrc from before local bridge began */
rtp->ssrc_orig = rtp->ssrc;
rtp->ssrc_saved = 1;
}
ao2_unlock(instance0);
return 0;
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment