commit 01b20a6cebcd7d03f610528e332ab6579cc4b4a2
author Tyler Wear <twear@codeaurora.org> Wed Aug 09 15:31:18 2017 -0700
committer Gerrit - the friendly Code Review server <code-review@localhost> Wed Oct 18 06:57:32 2017 -0700
tree 0a93b48cdf6611491381f750af5196bd5843ed92
parent cebbb78d45d2a38d49c0701c997606a7dc6b088d

SetDataLimit: Init Check

Check that offload has been initialized before setting the data
limit.

Bug: 38220415
Change-Id: Ide3924e6a69e239344751eff67de45e7cf2ac890
CRs-fixed: 2083182

hal/src/HAL.cpp

diff --git a/hal/src/HAL.cpp b/hal/src/HAL.cpp
index 9a8f211..d812f9b 100644
--- a/hal/src/HAL.cpp
+++ b/hal/src/HAL.cpp
@@ -463,11 +463,17 @@
     fl.addArg("upstream", upstream);
     fl.addArg("limit", limit);
 
-    RET ipaReturn = mIPA->setQuota(upstream.c_str(), limit);
-    BoolResult res = ipaResultToBoolResult(ipaReturn);
-    hidl_cb(res.success, res.errMsg);
+    if (!isInitialized()) {
+        BoolResult res = makeInputCheckFailure("Not initialized (setDataLimit)");
+        hidl_cb(res.success, res.errMsg);
+        fl.setResult(res.success, res.errMsg);
+    } else {
+        RET ipaReturn = mIPA->setQuota(upstream.c_str(), limit);
+        BoolResult res = ipaResultToBoolResult(ipaReturn);
+        hidl_cb(res.success, res.errMsg);
+        fl.setResult(res.success, res.errMsg);
+    }
 
-    fl.setResult(res.success, res.errMsg);
     mLogs.addLog(fl);
     return Void();
 } /* setDataLimit */