diff options
| author | Lorenzo Colitti <lorenzo@google.com> | 2022-03-24 20:53:24 +0900 |
|---|---|---|
| committer | Lorenzo Colitti <lorenzo@google.com> | 2022-03-29 08:38:06 +0000 |
| commit | 1727d9f1f25ffad24912d02fbddcacd1f7bac16a (patch) | |
| tree | 6325cebc1528cd7dc287abb0a5487e654357a51b /tests/common/java/android/net/NetworkCapabilitiesTest.java | |
| parent | a63e2341d6f1a5ba0b21425578e6ba99c06776f5 (diff) | |
Don't throw in FullScore#policyNameOf.
This code is correct on userdebug builds, but it is dangerous on
user builds because proguard might strip out the POLICY_*
constants and that would lead to crashes.
For now just log a wtf if an invalid policy name is found. A
better solution would be to make MessageUtils robust to this
problem, e.g., by having it store the SparseArray internally and
providing getters that do not throw, instead of the current
behaviour that returns the SparseArray. That is left to a future
CL.
Fix: 227161413
Test: atest FullScoreTest
Change-Id: I68b69ee9dd84773018e62c9a8f43e754ae04c486
Diffstat (limited to 'tests/common/java/android/net/NetworkCapabilitiesTest.java')
0 files changed, 0 insertions, 0 deletions