<div dir="ltr">I add a stream instead of replace stream, I think the problem is caused by the a=sendrecv created by WebRTC client, I will get deeper in to this.<div>Can you point out where's the actual SDP negotiation? I will check it too. Thanks for your help.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Joshua C. Colp <<a href="mailto:jcolp@digium.com">jcolp@digium.com</a>> 于2019年5月9日周四 上午12:40写道:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Wed, May 8, 2019, at 1:26 PM, Xiemin Chen wrote:<br>
> Please see the comments in the following codes, at the moment, A & B <br>
> both have one audio & video stream separately and B is starting to add <br>
> an extra sharing video:<br>
<br>
<snip><br>
<br>
> <br>
> /* The stream count of B's active_media_state->topology is 3, one B's <br>
> audio, one B's video, one A's video with name softbridge_dest_PJSIP... <br>
> The sdp->media_count is 3, one for audio, one for camera video, one <br>
> for sharing video<br>
> Here B's sharing video is map to A's video with name <br>
> softbridge_dest_PJSIP... */<br>
<br>
So are you adding a stream or replacing a stream? If adding I would expect 4 streams in the SDP media count: Audio, camera video, A's video, and the new sharing video. Are you actually instead replacing/reusing an existing stream?<br>
<br>
<snip><br>
<br>
> <br>
> /* If the code of B's new sharing video stream executes here, does it <br>
> need to call set_mid_and_bundle_group() & <br>
> set_remote_mslabel_and_stream_group() <br>
> & handler->negotiate_incoming_sdp_stream for the new stream setup <br>
> instead of the handler->defer_incoming_sdp_stream()?<br>
> */<br>
<br>
The purpose of this function is for determining if the SDP handling should be deferred until a later time, it's not for actually performing the SDP negotiation. That is done elsewhere. I don't believe this function should have any of the above done in it.<br>
<br>
-- <br>
Joshua C. Colp<br>
Digium - A Sangoma Company | Senior Software Developer<br>
445 Jan Davis Drive NW - Huntsville, AL 35806 - US<br>
Check us out at: <a href="http://www.digium.com" rel="noreferrer" target="_blank">www.digium.com</a> & <a href="http://www.asterisk.org" rel="noreferrer" target="_blank">www.asterisk.org</a><br>
<br>
-- <br>
_____________________________________________________________________<br>
-- Bandwidth and Colocation Provided by <a href="http://www.api-digital.com" rel="noreferrer" target="_blank">http://www.api-digital.com</a> --<br>
<br>
asterisk-dev mailing list<br>
To UNSUBSCRIBE or update options visit:<br>
<a href="http://lists.digium.com/mailman/listinfo/asterisk-dev" rel="noreferrer" target="_blank">http://lists.digium.com/mailman/listinfo/asterisk-dev</a></blockquote></div>