From 7a8e0643c3484a7991abf1a9d53cc12d0d3a7165 Mon Sep 17 00:00:00 2001
From: Saurabh Srivastava <ssrivast@codeaurora.org>
Date: Tue, 19 Jun 2018 18:43:51 +0530
Subject: [PATCH] Checking return for ODCPI CB

Adding return value check for ODCPI
HIDL callback API

CRs-Fixed: 2263480
Change-Id: I652a4717ab08f8a0c64b3bc0f78bae4e8f553513
---
 android/Gnss.cpp | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/android/Gnss.cpp b/android/Gnss.cpp
index 5cec786a..fca72e6e 100644
--- a/android/Gnss.cpp
+++ b/android/Gnss.cpp
@@ -434,10 +434,9 @@ void Gnss::odcpiRequestCb(const OdcpiRequestInfo& request) {
         // For emergency mode, request DBH (Device based hybrid) location
         // Mark Independent from GNSS flag to false.
         if (ODCPI_REQUEST_TYPE_START == request.type) {
-            if (request.isEmergencyMode) {
-                mGnssCbIface_1_1->gnssRequestLocationCb(false);
-            } else {
-                mGnssCbIface_1_1->gnssRequestLocationCb(true);
+            auto r = mGnssCbIface_1_1->gnssRequestLocationCb(!request.isEmergencyMode);
+            if (!r.isOk()) {
+                LOC_LOGe("Error invoking gnssRequestLocationCb %s", r.description().c_str());
             }
         } else {
             LOC_LOGv("Unsupported ODCPI request type: %d", request.type);