Merge PacketBuffer H264 fix to M59 |
||||
Issue descriptionIn some cases receiving a H264 stream will cause an infinite loop in the PacketBuffer. Fix has been landed here: https://codereview.chromium.org/2868723003/
,
May 22 2017
The following revision refers to this bug: https://chromium.googlesource.com/external/webrtc.git/+/2996dc5a520cbe2051d401dfce5ee4800162007f commit 2996dc5a520cbe2051d401dfce5ee4800162007f Author: philipel <philipel@webrtc.org> Date: Mon May 22 12:13:22 2017 Break backwards traversal loop if we have looped around all packets in the PacketBuffer for H264 frames. BUG= webrtc:7532 , chromium:724031 R=holmer@google.com Review-Url: https://codereview.webrtc.org/2868723003 Cr-Original-Commit-Position: refs/heads/master@{#18191} Review-Url: https://codereview.webrtc.org/2899713002 . Cr-Commit-Position: refs/branch-heads/59@{#12} Cr-Branched-From: 10d095d4f743bc16f8e486e156c48a6d023b32c5-refs/heads/master@{#17657} [modify] https://crrev.com/2996dc5a520cbe2051d401dfce5ee4800162007f/webrtc/modules/video_coding/packet_buffer.cc [modify] https://crrev.com/2996dc5a520cbe2051d401dfce5ee4800162007f/webrtc/modules/video_coding/video_packet_buffer_unittest.cc
,
May 22 2017
This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible! If all merges have been completed, please remove any remaining Merge-Approved labels from this issue. Thanks for your time! To disable nags, add the Disable-Nags label. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
May 22 2017
,
May 22 2017
|
||||
►
Sign in to add a comment |
||||
Comment 1 by sheriffbot@chromium.org
, May 19 2017