New issue
Advanced search Search tips

Issue 866535 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Aug 7
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug

Blocking:
issue 784495


Participants' hotlists:
LoginRefresh


Sign in to add a comment

Views login, managed device: Remove and re-add user results in auth error

Project Member Reported by bartfab@chromium.org, Jul 23

Issue description

Repro steps:
1) Enroll a device into enterprise management (Ctrl-Alt-E during OOBE)
2) Go through GAIA flow to add a new managed user to the device
3) Log out
4) Remove the user
5) Go through GAIA flow to add the same user to the device

Expected:
* After going through the GAIA flow, I am logged in as the new user.

Observed instead:
* I see a black screen for a few seconds and then the login screen again.
* A red exclamation mark and the user's user name are shown underneath the newly added pod.
* Clicking on the pod takes me to GAIA, with the user name pre-filled. If I authenticate once more, I can log in as the new user.

Note that both the device (1) and the user (2) are managed. In all my tests, both were part of the same domain. If the device is unmanaged or the user is a consumer user (@gmail.com), the problem does not occur.

The problem only happens with the Views login flow.

Observed on kevin M70 ToT today.
 
Status: Assigned (was: Untriaged)
This bug has an owner, thus, it's been triaged. Changing status to "assigned".
Labels: ReleaseBlock-Beta
Labels: M-70 OS-Chrome
Status: Started (was: Assigned)
Project Member

Comment 6 by bugdroid1@chromium.org, Aug 7

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/fb01a3b79d9e1ee1c26da5281eb7c710205a5be4

commit fb01a3b79d9e1ee1c26da5281eb7c710205a5be4
Author: Jacob Dufault <jdufault@google.com>
Date: Tue Aug 07 20:13:54 2018

cros: Fix crash when adding a user after removing one.

Caused by a DCHECK crash from a pointer pointing to deallocated memory.

Bug:  866535 
Change-Id: I96d76aeaeb8c89316cd6ffb237928cd2bfcfb744
Reviewed-on: https://chromium-review.googlesource.com/1165850
Reviewed-by: Xiaoyin Hu <xiaoyinh@chromium.org>
Commit-Queue: Jacob Dufault <jdufault@chromium.org>
Cr-Commit-Position: refs/heads/master@{#581326}
[modify] https://crrev.com/fb01a3b79d9e1ee1c26da5281eb7c710205a5be4/ash/login/ui/lock_contents_view.cc

Status: Fixed (was: Started)

Sign in to add a comment