commit 095bc25818904fa31ae30c454d011f33cc936530
author Wei Jia <wjia@google.com> Fri Jan 27 10:16:16 2017 -0800
committer Wei Jia <wjia@google.com> Fri Jan 27 13:58:01 2017 -0800
tree 3e6518d87adba2efdc0dd0fc0b5049b6e640a21e
parent d7ae0b8529d0b08496392c85ae4d00753e28882e

avc_utils: fix sample aspect ratio.

Test: ensure no 0:* and *:0 pattern of sample aspect ratio show up in
NuPlayer.
Bug: 34770857

Change-Id: I40bb2a9dc0ea7f50e3936d19dadb103d8214edf7

media/libmediaplayerservice/nuplayer/NuPlayer.cpp

diff --git a/media/libmediaplayerservice/nuplayer/NuPlayer.cpp b/media/libmediaplayerservice/nuplayer/NuPlayer.cpp
index 4c576a5..6593fcd 100644
--- a/media/libmediaplayerservice/nuplayer/NuPlayer.cpp
+++ b/media/libmediaplayerservice/nuplayer/NuPlayer.cpp
@@ -1822,7 +1822,8 @@
     // Take into account sample aspect ratio if necessary:
     int32_t sarWidth, sarHeight;
     if (inputFormat->findInt32("sar-width", &sarWidth)
-            && inputFormat->findInt32("sar-height", &sarHeight)) {
+            && inputFormat->findInt32("sar-height", &sarHeight)
+            && sarWidth > 0 && sarHeight > 0) {
         ALOGV("Sample aspect ratio %d : %d", sarWidth, sarHeight);
 
         displayWidth = (displayWidth * sarWidth) / sarHeight;