summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorv-krishna.rao <v-krishna.rao@oneplus.com>2020-07-30 09:22:05 -0400
committerDavid Su <dysu@google.com>2020-08-04 21:09:32 +0000
commit81437590af33ddfc00f92de88878399f93590125 (patch)
treeaceb8cf61fe76a3fe6e6a73265dc0052662e11bb
parentece9f2221fcb68c1f18ab78966e7eb8f5049dfdd (diff)
Handle IllegalArgumentException while doing addInterface.
In aging test, IllegalArgumentException was observed sometimes when set command to HwRemoteBinder. add exception handling for that. Bug: 162705230 Test: atest FrameworksWifiTests Change-Id: Ie0ac58e2b122fc003bdd088cb48757cf8fa89f44
-rw-r--r--service/java/com/android/server/wifi/SupplicantStaIfaceHal.java11
1 files changed, 11 insertions, 0 deletions
diff --git a/service/java/com/android/server/wifi/SupplicantStaIfaceHal.java b/service/java/com/android/server/wifi/SupplicantStaIfaceHal.java
index 657b081dd..667abfe35 100644
--- a/service/java/com/android/server/wifi/SupplicantStaIfaceHal.java
+++ b/service/java/com/android/server/wifi/SupplicantStaIfaceHal.java
@@ -527,6 +527,10 @@ public class SupplicantStaIfaceHal {
Log.e(TAG, "ISupplicant.addInterface exception: " + e);
handleNoSuchElementException(e, "addInterface");
return null;
+ } catch (IllegalArgumentException e) {
+ handleIllegalArgumentException(e, "addInterface");
+ Log.e(TAG, "ISupplicant.addInterface exception: " + e);
+ return null;
}
return supplicantIface.value;
}
@@ -2593,6 +2597,13 @@ public class SupplicantStaIfaceHal {
}
}
+ private void handleIllegalArgumentException(IllegalArgumentException e, String methodStr) {
+ synchronized (mLock) {
+ clearState();
+ Log.e(TAG, "ISupplicantStaIface." + methodStr + " failed with exception", e);
+ }
+ }
+
/**
* Converts the Wps config method string to the equivalent enum value.
*/