commit 0182a2cba07494de4d530ef1c346cf572c83ecc1
author Ryszard Grzesica <ryszard.grzesica@sonymobile.com> Tue Dec 29 06:28:44 2015 +0100
committer android-build-team Robot <android-build-team-robot@google.com> Fri May 25 18:38:53 2018 +0000
tree 389c83311b814cccf18a55fcbc64cc781a8594b6
parent ed964f613f051afcbda50648c27865d6defa021b

Add check preventing div0 issue

There might be a scenario while period is zero or after including
precision would be zero, prevent from division in that case and
return false (to use previously used period).

Bug: 73898703
bug: 74067957
Test: run playback as stability test

Change-Id: I3fad1060b095b7b5ea4c1f9cb3f9d42a4c503560
(cherry picked from commit 27e47ce3c3bbc0b4dc629163de7ebbba7e80b149)

media/libstagefright/VideoFrameScheduler.cpp

diff --git a/media/libstagefright/VideoFrameScheduler.cpp b/media/libstagefright/VideoFrameScheduler.cpp
index 03226c7..6819bba 100644
--- a/media/libstagefright/VideoFrameScheduler.cpp
+++ b/media/libstagefright/VideoFrameScheduler.cpp
@@ -129,6 +129,11 @@
         numSamplesToUse = mNumSamples;
     }
 
+    if ((period >> kPrecision) == 0 ) {
+        ALOGW("Period is 0, or after including precision is 0 - would cause div0, returning");
+        return false;
+    }
+
     int64_t sumX = 0;
     int64_t sumXX = 0;
     int64_t sumXY = 0;