From 2bc620b60737231a9a443191a985ba260f25154f Mon Sep 17 00:00:00 2001 From: Fabian Kozynski Date: Wed, 10 Jun 2020 16:14:35 -0400 Subject: [PATCH] Prevent error due to new API UserManager#createGuest will throw instead of returning null. Check for it and react in the same way. Test: no test Bug: 158659165 Change-Id: If2d873e0e45cc20a2b6a5fc6588d9a4af29efb8b --- .../statusbar/policy/UserSwitcherController.java | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/packages/SystemUI/src/com/android/systemui/statusbar/policy/UserSwitcherController.java b/packages/SystemUI/src/com/android/systemui/statusbar/policy/UserSwitcherController.java index 270f248e1a341..ce5bb0508c0a3 100644 --- a/packages/SystemUI/src/com/android/systemui/statusbar/policy/UserSwitcherController.java +++ b/packages/SystemUI/src/com/android/systemui/statusbar/policy/UserSwitcherController.java @@ -367,8 +367,14 @@ public class UserSwitcherController implements Dumpable { int id; if (record.isGuest && record.info == null) { // No guest user. Create one. - UserInfo guest = mUserManager.createGuest( - mContext, mContext.getString(com.android.settingslib.R.string.guest_nickname)); + UserInfo guest; + try { + guest = mUserManager.createGuest(mContext, + mContext.getString(com.android.settingslib.R.string.guest_nickname)); + } catch (UserManager.UserOperationException e) { + Log.e(TAG, "Couldn't create guest user", e); + return; + } if (guest == null) { // Couldn't create guest, most likely because there already exists one, we just // haven't reloaded the user list yet.