diff options
| author | Lutz Justen <ljusten@chromium.org> | 2018-01-21 15:14:28 +0100 |
|---|---|---|
| committer | chrome-bot <chrome-bot@chromium.org> | 2018-01-24 20:24:21 -0800 |
| commit | 9ea924a2ea937f9cd5fd3cdef952017ff4dbb794 (patch) | |
| tree | 645cd386969fbd7d9d18d86d2593c1dd762934ec | |
| parent | 55af1ecf95886bd60661dd7c62a9e32ec364afc1 (diff) | |
| download | system_api-9ea924a2ea937f9cd5fd3cdef952017ff4dbb794.tar.gz | |
authpolicy: Add ERROR_NO_WINDOWS_POLICY
CL:876124 adds user policy loopback processing, which now requires the
availability of Windows policy during user policy fetch. If this is
missing (user policy fetched before device policy, disk file
corruption), user policy fetch errors out for security reasons and
returns ERROR_NO_WINDOWS_POLICY.
CQ-DEPEND=CL:876124
BUG=chromium:796186
TEST=cros_run_unit_tests --board=amd64-generic --packages authpolicy
Tested by setting loopback_processing on AD server and verified
policies get merged as expected.
Change-Id: I3ea484127560784b8639eac9f8cf7f33e1a6b4c2
Reviewed-on: https://chromium-review.googlesource.com/877939
Commit-Ready: Lutz Justen <ljusten@chromium.org>
Tested-by: Lutz Justen <ljusten@chromium.org>
Reviewed-by: Lutz Justen <ljusten@chromium.org>
Reviewed-by: Roman Sorokin <rsorokin@chromium.org>
| -rw-r--r-- | dbus/authpolicy/active_directory_info.proto | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/dbus/authpolicy/active_directory_info.proto b/dbus/authpolicy/active_directory_info.proto index d65fb91..6996654 100644 --- a/dbus/authpolicy/active_directory_info.proto +++ b/dbus/authpolicy/active_directory_info.proto @@ -82,8 +82,10 @@ enum ErrorType { ERROR_OU_ACCESS_DENIED = 32; // Setting computer organizational unit failed with unspecified error. ERROR_SETTING_OU_FAILED = 33; + // Fetching user policy failed because Windows policy was unavailable. + ERROR_NO_WINDOWS_POLICY = 34; // Should be the last. - ERROR_COUNT = 34; + ERROR_COUNT = 35; } // Message sent to Chrome by authpolicyd as a response of a successful |