New issue
Advanced search Search tips

Issue 713037 link

Starred by 2 users

Issue metadata

Status: Verified
Owner:
Closed: Apr 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

Authpolicy should return correct error in case machine account was deleted from KDC

Project Member Reported by rsorokin@chromium.org, Apr 19 2017

Issue description

Steps:
1. Enroll machine, join domain
2. Remove machine account from KDc
3. Try to authenticate

Bad user name is shown.
 
Status: Started (was: Assigned)
It's going to return ERROR_BAD_MACHINE_NAME instead.
Project Member

Comment 3 by bugdroid1@chromium.org, Apr 25 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform/system_api/+/e0703ca1368dc66e533b3da359bce6edfe49fea2

commit e0703ca1368dc66e533b3da359bce6edfe49fea2
Author: Lutz Justen <ljusten@chromium.org>
Date: Tue Apr 25 21:03:43 2017

authpolicy: Add error code for missing machine account

Renamed ERROR_BAD_MACHINE_NAME to ERROR_INVALID_MACHINE_NAME since
it does not have the same meaning as ERROR_BAD_USER_NAME. The former
means that the machine name has invalid characters, the latter usually
means that the account does not exist in Active Directory, e.g.
mistyped user name. Did not change ID to not break UMA.

Added a new ERROR_BAD_MACHINE_NAME error in the end that has the
corresponding meaning as ERROR_BAD_USER_NAME but for machine accounts.

CQ-DEPEND=CL:482976

BUG= chromium:713037 
TEST=Compiled, ran tests.

Change-Id: I2363d2735a38c0ee2e7172ec5cb07e7b85d8fa91
Reviewed-on: https://chromium-review.googlesource.com/483099
Commit-Ready: Lutz Justen <ljusten@chromium.org>
Tested-by: Lutz Justen <ljusten@chromium.org>
Reviewed-by: Dan Erat <derat@chromium.org>
Reviewed-by: Roman Sorokin <rsorokin@chromium.org>

[modify] https://crrev.com/e0703ca1368dc66e533b3da359bce6edfe49fea2/dbus/authpolicy/dbus-constants.h

Status: Fixed (was: Started)
Also this: https://chromium-review.googlesource.com/c/482976/
bugdroid1 didn't post...
Status: Verified (was: Fixed)
bulk Verify of older or not-user-facing Chromad bugs

Sign in to add a comment