commit b2d842021b90562b439e547d91bcf6d1c0fd78ba
author Jayant Chowdhary <jchowdhary@google.com> Mon Jul 11 21:46:53 2022 +0000
committer Jayant Chowdhary <jchowdhary@google.com> Wed Jul 13 14:47:13 2022 -0700
tree 0bfebbe6a450822ceb9832c8b500971d6a884dbe
parent c728edd9e2dd94bb78679aad0d0778489d25e24e

cameraserver: Fix floating point truncation for getUHRMaxJpegBufferSize

Bug: 238103879

Test: Vendor testing
Test: DngCreatorTest.java on cuttlefish

Change-Id: I6553ed5f4b617e70516cfaac3c248da3176912da
Signed-off-by: Jayant Chowdhary <jchowdhary@google.com>

services/camera/libcameraservice/utils/SessionConfigurationUtils.cpp

diff --git a/services/camera/libcameraservice/utils/SessionConfigurationUtils.cpp b/services/camera/libcameraservice/utils/SessionConfigurationUtils.cpp
index 7dde268..ed490a8 100644
--- a/services/camera/libcameraservice/utils/SessionConfigurationUtils.cpp
+++ b/services/camera/libcameraservice/utils/SessionConfigurationUtils.cpp
@@ -127,7 +127,7 @@
 
 size_t getUHRMaxJpegBufferSize(camera3::Size uhrMaxJpegSize,
         camera3::Size defaultMaxJpegSize, size_t defaultMaxJpegBufferSize) {
-    return (uhrMaxJpegSize.width * uhrMaxJpegSize.height) /
+    return ((float)(uhrMaxJpegSize.width * uhrMaxJpegSize.height)) /
             (defaultMaxJpegSize.width * defaultMaxJpegSize.height) * defaultMaxJpegBufferSize;
 }