Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit f2390512 authored by Fan Zhang's avatar Fan Zhang
Browse files

Add null check in UserPreference

The comparator can get null objects according to bug report.

Fixes: 130144755
Test: manual
Change-Id: Ic8a332d390f411d2eb69c944fb7fd166d9961535
parent 44427259
Loading
Loading
Loading
Loading
+15 −11
Original line number Diff line number Diff line
@@ -40,9 +40,14 @@ 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) {
            (p1, p2) -> {

                if (p1 == null) {
                    return -1;
                }
                else if (p2 == null) {
                    return 1;
                }
                int sn1 = p1.getSerialNumber();
                int sn2 = p2.getSerialNumber();
                if (sn1 < sn2) {
@@ -51,7 +56,6 @@ public class UserPreference extends RestrictedPreference {
                    return 1;
                }
                return 0;
                }
            };

    private OnClickListener mDeleteClickListener;