New issue
Advanced search Search tips

Issue 713754 link

Starred by 2 users

Issue metadata

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



Sign in to add a comment

Clean state on start

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

Issue description

clean /var/lib/authpolicyd on start in case install attributes are not set. That could be the case when user cancelled domain joining in the middle.
 
Project Member

Comment 1 by bugdroid1@chromium.org, May 2 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform2/+/c03c9de10c11795a7ea5197956ffbf71646a0702

commit c03c9de10c11795a7ea5197956ffbf71646a0702
Author: Lutz Justen <ljusten@chromium.org>
Date: Tue May 02 14:31:31 2017

authpolicy: Clean state on startup

Cleans state on startup when install attributes are not set to Active
Directory enrollment. This might occur if something unexpected happens
during enrollment, e.g. the user cancels domain join.

Also cleans up AuthPolicy construction and simplifies ownership of
metrics and paths, which makes some *ForTesting accessors obsolete.

BUG= chromium:713754 
TEST=Compiled, ran tests, tested on device.

Change-Id: If1cf6c406c75423d1900d460e9200737a800730e
Reviewed-on: https://chromium-review.googlesource.com/485601
Commit-Ready: Lutz Justen <ljusten@chromium.org>
Tested-by: Lutz Justen <ljusten@chromium.org>
Reviewed-by: Thiemo Nagel <tnagel@chromium.org>

[modify] https://crrev.com/c03c9de10c11795a7ea5197956ffbf71646a0702/authpolicy/authpolicy_unittest.cc
[modify] https://crrev.com/c03c9de10c11795a7ea5197956ffbf71646a0702/authpolicy/authpolicy.h
[modify] https://crrev.com/c03c9de10c11795a7ea5197956ffbf71646a0702/authpolicy/samba_interface.cc
[modify] https://crrev.com/c03c9de10c11795a7ea5197956ffbf71646a0702/authpolicy/samba_interface.h
[modify] https://crrev.com/c03c9de10c11795a7ea5197956ffbf71646a0702/authpolicy/authpolicy.cc
[modify] https://crrev.com/c03c9de10c11795a7ea5197956ffbf71646a0702/authpolicy/authpolicy_main.cc

Status: Fixed (was: Assigned)
Status: Verified (was: Fixed)
bulk Verify of older or not-user-facing Chromad bugs

Sign in to add a comment