New issue
Advanced search Search tips

Issue 686231 link

Starred by 4 users

Issue metadata

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



Sign in to add a comment

Chromad: Fetch GPO with sAMAccountName

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

Issue description

Net ads gpo list requires the sAMAccountName. If the sAMAccountName is different from the principal, and the user logs in with the principal, net ads gpo list will probably return an empty list (verify that!).
 
Status: Started (was: Unconfirmed)
Project Member

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

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

commit 991ade4ea4bb3318d1148de835d1528481bfcfb5
Author: Lutz Justen <ljusten@chromium.org>
Date: Tue Jan 31 15:12:03 2017

authpolicy: Fetch GPO with sAMAccountName

Fixes policy fetch in case the user principal name (UPN) and sAMAccountName
don't match. Searching the objectGUID already handled this correctly, but
when the user logged in with the UPN, policy fetch would fail since it
requires the sAMAccountName. This CL properly retrieves the
sAMAccountName and uses it for policy fetch.

BUG= chromium:686231 
TEST=Successfully fetched policy for an account with UPN != sAMAccountName

Change-Id: Ib83f9688e339e2f1d87cb4ea2bb0556b337f86ca
Reviewed-on: https://chromium-review.googlesource.com/435358
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/991ade4ea4bb3318d1148de835d1528481bfcfb5/authpolicy/samba_interface.h
[modify] https://crrev.com/991ade4ea4bb3318d1148de835d1528481bfcfb5/authpolicy/constants.h
[modify] https://crrev.com/991ade4ea4bb3318d1148de835d1528481bfcfb5/authpolicy/samba_interface.cc
[modify] https://crrev.com/991ade4ea4bb3318d1148de835d1528481bfcfb5/authpolicy/proto/authpolicy_containers.proto
[modify] https://crrev.com/991ade4ea4bb3318d1148de835d1528481bfcfb5/authpolicy/authpolicy_parser_main.cc

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

Sign in to add a comment