TetherOffload: Address VTS failure in AdditionalStopsWithInitReturnFalseandroid-9.0.0_r47android-9.0.0_r46android-9.0.0_r45android-9.0.0_r44android-9.0.0_r43android-9.0.0_r42android-9.0.0_r41android-9.0.0_r40android-9.0.0_r39android-9.0.0_r38android-9.0.0_r37android-9.0.0_r36android-9.0.0_r35android-9.0.0_r34android-9.0.0_r33android-9.0.0_r32android-9.0.0_r31android-9.0.0_r30android-9.0.0_r22android-9.0.0_r21android-9.0.0_r20android-9.0.0_r19android-9.0.0_r16android-9.0.0_r12android-9.0.0_r11pie-qpr3-s1-releasepie-qpr3-releasepie-qpr3-b-releasepie-qpr2-releasepie-qpr1-s3-releasepie-qpr1-s2-releasepie-qpr1-s1-releasepie-qpr1-releasepie-dr1-releasepie-dr1-devpie-devpie-b4s4-releasepie-b4s4-dev

- Currently if NAT iface is not added, IPACM cache setupstream
  events but default gw index will be set only after adding
  NAT iface. If IPACM receives stopOffload in between, then
  it is returning input check error to framework. Add check and
  clear event cache in case of stopOffload.

- Fixed variable shadowing issue in HAL.

Test: - build and boot - pass
      - VtsHalTetheroffloadControlV1_0TargetTest    - pass
      - VTS Test AdditionalStopsWithInitReturnFalse - pass

Change-Id: I335b7834c9a7567bc5bc05f683ec789b39c1efac
Crs-fixed: 2228293
bug: 77996655
(cherry picked from commit b8c3d7ae84541f40c7b82e6c32ea1915ad4b1766)

1 files changed, 1 insertions, 1 deletions

diff --git a/msm8998/hal/src/HAL.cpp b/msm8998/hal/src/HAL.cpp
index 11dd26d..42fe868 100644
--- a/msm8998/hal/src/HAL.cpp
+++ b/msm8998/hal/src/HAL.cpp
@@ -408,7 +408,7 @@ Return<void> HAL::setLocalPrefixes
     fl.addArg("prefixes", prefixesStr);
 
     if (!isInitialized()) {
-        BoolResult res = makeInputCheckFailure("Not initialized");
+        res = makeInputCheckFailure("Not initialized");
     } else if(prefixesStr.size() < 1) {
         res = ipaResultToBoolResult(RET::FAIL_INPUT_CHECK);
     } else if (!parser.add(prefixesStr)) {