New issue
Advanced search Search tips

Issue 685686 link

Starred by 4 users

Issue metadata

Status: Verified
Owner:
Closed: Feb 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

Chromad: Support GPFLAGS in GPO parsing

Project Member Reported by ljusten@chromium.org, Jan 26 2017

Issue description

4 states:
options:0 GPFLAGS_ALL_ENABLED
options:1 GPFLAGS_USER_SETTINGS_DISABLED
options:2 GPFLAGS_MACHINE_SETTINGS_DISABLED
options:3 GPFLAGS_ALL_DISABLED

 
Description: Show this description

Comment 2 by tnagel@chromium.org, Jan 31 2017

Labels: -Pri-2 Pri-1
Status: Fixed (was: Assigned)
Marking fixed, CL is in the CQ.
Feeling lucky?
Project Member

Comment 5 by bugdroid1@chromium.org, Feb 2 2017

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

commit efc172fd0ac733473df5470229e9f523bf4938eb
Author: Lutz Justen <ljusten@chromium.org>
Date: Thu Feb 02 20:29:40 2017

authpolicy: Add support for GPFLAGS

GPOs have flags that cause them to have no effect on user or machine
policy. This CL handles those flags properly and selectively filters
out GPOs depending on the flags.

BUG= chromium:685686 
TEST=Compiled, ran tests, tested with GPOs having various flags

Change-Id: I739c40bc30ae2addda6e7d2833325737a10fcdcf
Reviewed-on: https://chromium-review.googlesource.com/435300
Commit-Ready: Lutz Justen <ljusten@chromium.org>
Tested-by: Lutz Justen <ljusten@chromium.org>
Reviewed-by: Roman Sorokin <rsorokin@chromium.org>

[modify] https://crrev.com/efc172fd0ac733473df5470229e9f523bf4938eb/authpolicy/samba_interface_internal_unittest.cc
[modify] https://crrev.com/efc172fd0ac733473df5470229e9f523bf4938eb/authpolicy/samba_interface_internal.cc
[modify] https://crrev.com/efc172fd0ac733473df5470229e9f523bf4938eb/authpolicy/samba_interface_internal.h
[modify] https://crrev.com/efc172fd0ac733473df5470229e9f523bf4938eb/authpolicy/authpolicy_parser_main.cc

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

Sign in to add a comment