From 3ea3e300dcc3424b49a01a61acaaabbb17f5e884 Mon Sep 17 00:00:00 2001 From: Natalie Silvanovich Date: Wed, 16 May 2018 11:03:12 -0700 Subject: [PATCH] Fixing some SIGFPEs that are making my tests crash Bug: none Change-Id: Ib538e4f131a2c05b9b832bc8235f4f0bb35d04c0 Reviewed-on: https://webrtc-review.googlesource.com/74622 Commit-Queue: Philip Eliasson Reviewed-by: Stefan Holmer Reviewed-by: Philip Eliasson Cr-Commit-Position: refs/heads/master@{#23515} --- modules/video_coding/rtp_frame_reference_finder.cc | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/modules/video_coding/rtp_frame_reference_finder.cc b/modules/video_coding/rtp_frame_reference_finder.cc index f05f8d3e48..296ff04888 100644 --- a/modules/video_coding/rtp_frame_reference_finder.cc +++ b/modules/video_coding/rtp_frame_reference_finder.cc @@ -440,6 +440,11 @@ RtpFrameReferenceFinder::FrameDecision RtpFrameReferenceFinder::ManageFrameVp9( "layer frame. Scalability structure ignored."; } else { current_ss_idx_ = Add(current_ss_idx_, 1); + if (codec_header.gof.num_frames_in_gof == 0 || + codec_header.gof.num_frames_in_gof > kMaxVp9FramesInGof) { + return kDrop; + } + scalability_structures_[current_ss_idx_] = codec_header.gof; scalability_structures_[current_ss_idx_].pid_start = frame->id.picture_id; gof_info_.emplace(codec_header.tl0_pic_idx,