generic-library/webrtc
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

If one of the bundled content is missing in SDP, return false to MaybeEnalbeMuxingSupport().

Browse Source 
Verified in chromium. Now the existing content still could work.

BUG=4096
TEST=Manual Test
R=juberti@webrtc.org

Review URL: https://webrtc-codereview.appspot.com/36529004

git-svn-id: http://webrtc.googlecode.com/svn/trunk@7926 4adac7df-926f-26a2-2b94-8c16560cd09d
This commit is contained in:
braveyao@webrtc.org 2014-12-17 05:59:41 +00:00
parent 950c518251
commit 38881be912
1 changed files with 6 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
webrtc/p2p/base/session.cc
View File

@ -730,7 +730,12 @@ bool BaseSession::MaybeEnableMuxingSupport() {
const std::string* content_name = local_bundle_group->FirstContentName();
const ContentInfo* content =
local_description_->GetContentByName(*content_name);
ASSERT(content != NULL);
if (!content) {
LOG(LS_WARNING) << "Content \"" << *content_name
<< "\" referenced in BUNDLE group is not present";
return false;
}
if (!SetSelectedProxy(content->name, local_bundle_group)) {
LOG(LS_WARNING) << "Failed to set up BUNDLE";
return false;