diff options
author | v-krishna.rao <v-krishna.rao@oneplus.com> | 2020-07-30 09:22:05 -0400 |
---|---|---|
committer | David Su <dysu@google.com> | 2020-08-04 21:09:32 +0000 |
commit | 81437590af33ddfc00f92de88878399f93590125 (patch) | |
tree | aceb8cf61fe76a3fe6e6a73265dc0052662e11bb /service | |
parent | ece9f2221fcb68c1f18ab78966e7eb8f5049dfdd (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
Diffstat (limited to 'service')
-rw-r--r-- | service/java/com/android/server/wifi/SupplicantStaIfaceHal.java | 11 |
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. */ |