commit fe82e3b241c867143a28652e4df2fd9fddee4980
author Dante Russo <drusso@codeaurora.org> Thu Feb 22 12:35:48 2018 -0800
committer Siddharth Ray <siddharthr@google.com> Thu Feb 22 15:14:35 2018 -0800
tree 010265099769612df9e177dbd527ab7aa2c91d37
parent 87b632f0955ccb7e2ad59ed34d02544b29e65e16

Fix incorrect cast to avoid integer overflow

Fix incorrect cast in converting time uncertainty
from integer based source to float.

Change-Id: Ibb75eb1a2d925a7e7ce13275f09a618f241c31bf
CRs-Fixed: 2157401
Bug: 70215155

msm8998/gnss/GnssAdapter.cpp

diff --git a/msm8998/gnss/GnssAdapter.cpp b/msm8998/gnss/GnssAdapter.cpp
index 6202a0e..e14cf28 100644
--- a/msm8998/gnss/GnssAdapter.cpp
+++ b/msm8998/gnss/GnssAdapter.cpp
@@ -2829,8 +2829,8 @@
               (int64_t)(reports.mTimeAndClock.back().mGpsTowMs);
 
         r.mTime.timeUncertaintyNs =
-            (float)((reports.mTimeAndClock.back().mTimeUnc +
-                     reports.mTimeAndClock.back().mLeapSecUnc)*1000);
+                ((float)(reports.mTimeAndClock.back().mTimeUnc) +
+                 (float)(reports.mTimeAndClock.back().mLeapSecUnc))*1000.0f;
         r.mTime.frequencyUncertaintyNsPerSec =
             (float)(reports.mTimeAndClock.back().mClockFreqBiasUnc);
         LOC_LOGV("getDebugReport - timeestimate=%ld unc=%f frequnc=%f",
@@ -2850,4 +2850,4 @@
     LOC_LOGV("getDebugReport - satellite=%lu", r.mSatelliteInfo.size());
 
     return true;
-}
\ No newline at end of file
+}