commit 27ed3cc28cf950456a0c66d7a10656a96832fedd
author lally <lally@chromium.org> Mon Jan 11 10:24:33 2016 -0800
committer Commit bot <commit-bot@chromium.org> Mon Jan 11 18:24:35 2016 +0000
tree 4122c2bbd319e6ad0836c6ad2f397ac2965c4298
parent a9a1d2acaf71281a6f09cd85acb6bc21b02e5ce4

SCTP: Stopped accepting SSRCs higher than max.
Seems to fix asan-related crash.

BUG=https://code.google.com/p/chromium/issues/detail?id=570261

Review URL: https://codereview.webrtc.org/1571853002

Cr-Commit-Position: refs/heads/master@{#11205}

talk/media/sctp/sctpdataengine.cc

diff --git a/talk/media/sctp/sctpdataengine.cc b/talk/media/sctp/sctpdataengine.cc
index 94f242a..3753cd2 100644
--- a/talk/media/sctp/sctpdataengine.cc
+++ b/talk/media/sctp/sctpdataengine.cc
@@ -729,7 +729,13 @@
   }
 
   const uint32_t ssrc = stream.first_ssrc();
-  if (open_streams_.find(ssrc) != open_streams_.end()) {
+  if (ssrc >= cricket::kMaxSctpSid) {
+    LOG(LS_WARNING) << debug_name_ << "->Add(Send|Recv)Stream(...): "
+                    << "Not adding data stream '" << stream.id
+                    << "' with ssrc=" << ssrc
+                    << " because stream ssrc is too high.";
+    return false;
+  } else if (open_streams_.find(ssrc) != open_streams_.end()) {
     LOG(LS_WARNING) << debug_name_ << "->Add(Send|Recv)Stream(...): "
                     << "Not adding data stream '" << stream.id
                     << "' with ssrc=" << ssrc