libstagefright: Check for duration > 0 to avoid DivideByZero crash

- duration = 0 can cause divide by zero and for this clip duration is indiacted as 0. - check for duration > 0 rather than duration >= 0 Change-Id: I58ccacbf7ede892dff9626715162ea7b1f2ddbc6 CRs-Fixed: 451855
jied83 · Feb 19, 2013 · 1bf291a · 1bf291a
1 parent 390e2b1
commit 1bf291a
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/media/libstagefright/AwesomePlayer.cpp b/media/libstagefright/AwesomePlayer.cpp
@@ -638,7 +638,7 @@ void AwesomePlayer::notifyListener_l(int msg, int ext1, int ext2) {
 
 bool AwesomePlayer::getBitrate(int64_t *bitrate) {
     off64_t size;
-    if (mDurationUs >= 0 && mCachedSource != NULL
+    if (mDurationUs > 0 && mCachedSource != NULL
             && mCachedSource->getSize(&size) == OK) {
         *bitrate = size * 8000000ll / mDurationUs;  // in bits/sec
         return true;