Fix PseudoTcp to handle incoming packets with invalid SEQ field

webrtc/p2p/base/pseudotcp.cc

Index: webrtc/p2p/base/pseudotcp.cc
diff --git a/webrtc/p2p/base/pseudotcp.cc b/webrtc/p2p/base/pseudotcp.cc
index ec302d3520423fa7ad1f95971feb950f6e37765d..db16c549da68699a3b1f5c96fbc696afc1e1aa1e 100644
--- a/webrtc/p2p/base/pseudotcp.cc
+++ b/webrtc/p2p/base/pseudotcp.cc
@@ -910,8 +910,13 @@ bool PseudoTcp::process(Segment& seg) {
     } else {
       uint32_t nOffset = seg.seq - m_rcv_nxt;
 
-      rtc::StreamResult result = m_rbuf.WriteOffset(seg.data, seg.len,
-                                                          nOffset, NULL);
+      rtc::StreamResult result =
+          m_rbuf.WriteOffset(seg.data, seg.len, nOffset, NULL);
+      if (result == rtc::SR_BLOCK) {
+        // Ignore incoming packets outside of the receive window.
+        return false;
+      }
+
       RTC_DCHECK(result == rtc::SR_SUCCESS);
       RTC_UNUSED(result);

[nisse-webrtc, 2017/01/19 07:44:16]

You can delete this RTC_UNUSED, it's a left over from my conversion from ASSERT
to RTC_CHECK.

[Sergey Ulanov, 2017/01/19 18:06:32]

Done.