commit 9597360e2019ff0dfa0ad501d55ba488a2b38332
author Tim Lin <timtmlin@google.com> Thu Apr 07 09:33:45 2022 +0800
committer Tim Lin <timtmlin@google.com> Mon Apr 11 10:03:01 2022 +0800
tree 0d9fdb55674b44025f9cea2dd61020fb4b368ef0
parent b5e9cdaf7e4a88daeaba9e37cd8c4bbef0a11749

fix infinite loop in enableModem TC

Bug: 227585563
Test: run vts -m VtsHalRadioTargetTest
Change-Id: I879faa5ecef026865d17ad1a23e52cb9f8f70454
Merged-In: I879faa5ecef026865d17ad1a23e52cb9f8f70454

radio/aidl/vts/radio_modem_response.cpp

diff --git a/radio/aidl/vts/radio_modem_response.cpp b/radio/aidl/vts/radio_modem_response.cpp
index d2715a8..20b44c5 100644
--- a/radio/aidl/vts/radio_modem_response.cpp
+++ b/radio/aidl/vts/radio_modem_response.cpp
@@ -24,6 +24,7 @@
 
 ndk::ScopedAStatus RadioModemResponse::enableModemResponse(const RadioResponseInfo& info) {
     rspInfo = info;
+    enableModemResponseToggle = !enableModemResponseToggle;
     parent_modem.notify(info.serial);
     return ndk::ScopedAStatus::ok();
 }

radio/aidl/vts/radio_modem_utils.h

diff --git a/radio/aidl/vts/radio_modem_utils.h b/radio/aidl/vts/radio_modem_utils.h
index 8779e0c..49e1891 100644
--- a/radio/aidl/vts/radio_modem_utils.h
+++ b/radio/aidl/vts/radio_modem_utils.h
@@ -37,7 +37,7 @@
 
     RadioResponseInfo rspInfo;
     bool isModemEnabled;
-    bool enableModemResponseToggle;
+    bool enableModemResponseToggle = false;
 
     virtual ndk::ScopedAStatus acknowledgeRequest(int32_t serial) override;