diff options
author | Fan Zhang <zhfan@google.com> | 2019-04-08 13:55:58 -0700 |
---|---|---|
committer | Fan Zhang <zhfan@google.com> | 2019-04-08 13:55:58 -0700 |
commit | f2390512c166866f694066a8274b8ea0d686fd06 (patch) | |
tree | 75a0b7c956a665a428421db26f2eba94f0512cbc /src/com/android/settings/users | |
parent | 44427259e8206d4e66ac2f268a755450a7613c94 (diff) | |
download | packages_apps_Settings-f2390512c166866f694066a8274b8ea0d686fd06.tar.gz packages_apps_Settings-f2390512c166866f694066a8274b8ea0d686fd06.tar.bz2 packages_apps_Settings-f2390512c166866f694066a8274b8ea0d686fd06.zip |
Add null check in UserPreference
The comparator can get null objects according to bug report.
Fixes: 130144755
Test: manual
Change-Id: Ic8a332d390f411d2eb69c944fb7fd166d9961535
Diffstat (limited to 'src/com/android/settings/users')
-rw-r--r-- | src/com/android/settings/users/UserPreference.java | 26 |
1 files changed, 15 insertions, 11 deletions
diff --git a/src/com/android/settings/users/UserPreference.java b/src/com/android/settings/users/UserPreference.java index d87d78ecba..3603d44ea0 100644 --- a/src/com/android/settings/users/UserPreference.java +++ b/src/com/android/settings/users/UserPreference.java @@ -40,18 +40,22 @@ public class UserPreference extends RestrictedPreference { public static final int USERID_UNKNOWN = -10; public static final int USERID_GUEST_DEFAULTS = -11; public static final Comparator<UserPreference> SERIAL_NUMBER_COMPARATOR = - new Comparator<UserPreference>() { - @Override - public int compare(UserPreference p1, UserPreference p2) { - int sn1 = p1.getSerialNumber(); - int sn2 = p2.getSerialNumber(); - if (sn1 < sn2) { - return -1; - } else if (sn1 > sn2) { - return 1; - } - return 0; + (p1, p2) -> { + + if (p1 == null) { + return -1; + } + else if (p2 == null) { + return 1; + } + int sn1 = p1.getSerialNumber(); + int sn2 = p2.getSerialNumber(); + if (sn1 < sn2) { + return -1; + } else if (sn1 > sn2) { + return 1; } + return 0; }; private OnClickListener mDeleteClickListener; |