MT#55283 don't copy media_id from src to dst

Change-Id: Ia7c1256dff3025162431e8568cdfcfdfef475070
(cherry picked from commit 0b82e17f01)
mr13.5
Richard Fuchs 2 months ago
parent 5773627871
commit 25da9b1b97

@ -2286,7 +2286,6 @@ static void __update_media_id(struct call_media *media, struct call_media *other
if (flags->opmode == OP_OFFER || if (flags->opmode == OP_OFFER ||
flags->opmode == OP_PUBLISH || flags->opmode == OP_PUBLISH ||
flags->opmode == OP_SUBSCRIBE_REQ ||
IS_OP_OTHER(flags->opmode)) IS_OP_OTHER(flags->opmode))
{ {
if (!other_media->media_id.s) { if (!other_media->media_id.s) {
@ -2348,6 +2347,10 @@ static void __update_media_id(struct call_media *media, struct call_media *other
STR_FMT(&sp->media_id), STR_FMT(&other_media->media_id)); STR_FMT(&sp->media_id), STR_FMT(&other_media->media_id));
} }
} }
else if (flags->generate_mid && !media->media_id.len) {
generate_mid(media, media->index);
t_hash_table_insert(ml->media_ids, &media->media_id, media);
}
} }
static void __t38_reset(struct call_media *media, struct call_media *other_media) { static void __t38_reset(struct call_media *media, struct call_media *other_media) {

@ -1791,7 +1791,7 @@ class TestVideoroom(unittest.TestCase):
"a=msid-semantic: WMS hJifdaJwqEqHxSG0pVbs1DrLAwiHqz7fKlqC\r\n" "a=msid-semantic: WMS hJifdaJwqEqHxSG0pVbs1DrLAwiHqz7fKlqC\r\n"
"m=audio \\d+ UDP/TLS/RTP/SAVPF 111\r\n" "m=audio \\d+ UDP/TLS/RTP/SAVPF 111\r\n"
"c=IN IP4 203.0.113.1\r\n" "c=IN IP4 203.0.113.1\r\n"
"a=mid:0\r\n" "a=mid:1\r\n"
"a=rtpmap:111 opus/48000/2\r\n" "a=rtpmap:111 opus/48000/2\r\n"
"a=fmtp:111 useinbandfec=1; minptime=10\r\n" "a=fmtp:111 useinbandfec=1; minptime=10\r\n"
"a=rtcp-fb:111 transport-cc\r\n" "a=rtcp-fb:111 transport-cc\r\n"
@ -1818,7 +1818,7 @@ class TestVideoroom(unittest.TestCase):
"a=end-of-candidates\r\n" "a=end-of-candidates\r\n"
"m=video \\d+ UDP/TLS/RTP/SAVPF 96\r\n" "m=video \\d+ UDP/TLS/RTP/SAVPF 96\r\n"
"c=IN IP4 203.0.113.1\r\n" "c=IN IP4 203.0.113.1\r\n"
"a=mid:1\r\n" "a=mid:2\r\n"
"a=rtpmap:96 VP8/90000\r\n" "a=rtpmap:96 VP8/90000\r\n"
"a=rtcp-fb:96 goog-remb\r\n" "a=rtcp-fb:96 goog-remb\r\n"
"a=rtcp-fb:96 transport-cc\r\n" "a=rtcp-fb:96 transport-cc\r\n"
@ -2056,7 +2056,7 @@ class TestVideoroom(unittest.TestCase):
"t=0 0\r\n" "t=0 0\r\n"
"m=audio \\d+ UDP/TLS/RTP/SAVPF 8\r\n" "m=audio \\d+ UDP/TLS/RTP/SAVPF 8\r\n"
"c=IN IP4 203.0.113.1\r\n" "c=IN IP4 203.0.113.1\r\n"
"a=mid:audio\r\n" "a=mid:1\r\n"
"a=rtpmap:8 PCMA/8000\r\n" "a=rtpmap:8 PCMA/8000\r\n"
"a=sendonly\r\n" "a=sendonly\r\n"
"a=rtcp-mux\r\n" "a=rtcp-mux\r\n"
@ -2105,7 +2105,7 @@ class TestVideoroom(unittest.TestCase):
"s=foobar\r\n" "s=foobar\r\n"
"t=0 0\r\n" "t=0 0\r\n"
"m=audio 9 RTP/AVP 8\r\n" "m=audio 9 RTP/AVP 8\r\n"
"a=mid:audio\r\n" "a=mid:1\r\n"
"a=ice-ufrag:abcd\r\n" "a=ice-ufrag:abcd\r\n"
"a=ice-pwd:WD1pLsdgsdfsdWuEBb0vjyZr\r\n" "a=ice-pwd:WD1pLsdgsdfsdWuEBb0vjyZr\r\n"
"a=ice-options:trickle\r\n" "a=ice-options:trickle\r\n"
@ -2152,7 +2152,7 @@ class TestVideoroom(unittest.TestCase):
"janus": "trickle", "janus": "trickle",
"candidate": { "candidate": {
"candidate": "candidate:3fgsdfs273 1 udp 2113937151 203.0.113.2 30002 typ host generation 0", "candidate": "candidate:3fgsdfs273 1 udp 2113937151 203.0.113.2 30002 typ host generation 0",
"sdpMid": "audio", "sdpMid": "1",
"usernameFragment": "abcd", "usernameFragment": "abcd",
}, },
"handle_id": sub_handle, "handle_id": sub_handle,
@ -3188,13 +3188,13 @@ class TestVideoroom(unittest.TestCase):
"t=0 0\r\n" "t=0 0\r\n"
"m=audio (\\d+) RTP/AVP 96\r\n" "m=audio (\\d+) RTP/AVP 96\r\n"
"c=IN IP4 203.0.113.1\r\n" "c=IN IP4 203.0.113.1\r\n"
"a=mid:a\r\n" "a=mid:1\r\n"
"a=rtpmap:96 opus/48000/2\r\n" "a=rtpmap:96 opus/48000/2\r\n"
"a=sendonly\r\n" "a=sendonly\r\n"
"a=rtcp:\\d+\r\n" "a=rtcp:\\d+\r\n"
"m=video (\\d+) RTP/AVP 97\r\n" "m=video (\\d+) RTP/AVP 97\r\n"
"c=IN IP4 203.0.113.1\r\n" "c=IN IP4 203.0.113.1\r\n"
"a=mid:v\r\n" "a=mid:2\r\n"
"a=rtpmap:97 VP9/90000\r\n" "a=rtpmap:97 VP9/90000\r\n"
"a=sendonly\r\n" "a=sendonly\r\n"
"a=rtcp:\\d+\r\n$", "a=rtcp:\\d+\r\n$",
@ -3240,11 +3240,11 @@ class TestVideoroom(unittest.TestCase):
"t=0 0\r\n" "t=0 0\r\n"
"m=audio 31010 RTP/AVP 96\r\n" "m=audio 31010 RTP/AVP 96\r\n"
"a=rtpmap:96 opus/48000/2\r\n" "a=rtpmap:96 opus/48000/2\r\n"
"a=mid:a\r\n" "a=mid:1\r\n"
"a=recvonly\r\n" "a=recvonly\r\n"
"m=video 31110 RTP/AVP 97\r\n" "m=video 31110 RTP/AVP 97\r\n"
"a=rtpmap:97 VP9/90000\r\n" "a=rtpmap:97 VP9/90000\r\n"
"a=mid:v\r\n" "a=mid:2\r\n"
"a=recvonly\r\n" "a=recvonly\r\n"
), ),
}, },

Loading…
Cancel
Save