New issue
Advanced search Search tips

Issue 755153 link

Starred by 2 users

Issue metadata

Status: Verified
Owner:
Closed: Oct 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug



Sign in to add a comment

Remove support for GSSAPILibraryName on Chrome OS

Project Member Reported by ljusten@chromium.org, Aug 14 2017

Issue description

Check if GSSAPILibraryName affects Chrome OS and ifdef it out if it does.
 
Status: Started (was: Assigned)
Project Member

Comment 2 by bugdroid1@chromium.org, Sep 29 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/9edd41e5c77d248e17b77f16af6d5aa19992093a

commit 9edd41e5c77d248e17b77f16af6d5aa19992093a
Author: Lutz Justen <ljusten@chromium.org>
Date: Fri Sep 29 04:53:08 2017

Ifdef GSSAPILibraryName out on Chrome OS

Don't allow changing GSSAPI library name on Chrome OS (there's no
reason to). Note that the GSSAPILibraryName policy is not supported
on Chrome OS, which means that it already cannot be set through
policy. This CL is an additional security measure.

BUG= chromium:755153 
TEST=Compiled, trybots

Change-Id: Ibc2eb7e0ce59ccc5e7f7c8fd66078bc0efb2e3a5
Reviewed-on: https://chromium-review.googlesource.com/685836
Reviewed-by: Nicolas Zea <zea@chromium.org>
Reviewed-by: Matt Menke <mmenke@chromium.org>
Reviewed-by: Asanka Herath <asanka@chromium.org>
Commit-Queue: Lutz Justen <ljusten@chromium.org>
Cr-Commit-Position: refs/heads/master@{#505297}
[modify] https://crrev.com/9edd41e5c77d248e17b77f16af6d5aa19992093a/chrome/browser/io_thread.cc
[modify] https://crrev.com/9edd41e5c77d248e17b77f16af6d5aa19992093a/chrome/browser/io_thread.h
[modify] https://crrev.com/9edd41e5c77d248e17b77f16af6d5aa19992093a/google_apis/gcm/tools/mcs_probe.cc
[modify] https://crrev.com/9edd41e5c77d248e17b77f16af6d5aa19992093a/net/http/http_auth_handler_factory.cc
[modify] https://crrev.com/9edd41e5c77d248e17b77f16af6d5aa19992093a/net/http/http_auth_preferences.cc
[modify] https://crrev.com/9edd41e5c77d248e17b77f16af6d5aa19992093a/net/http/http_auth_preferences.h
[modify] https://crrev.com/9edd41e5c77d248e17b77f16af6d5aa19992093a/net/http/http_auth_preferences_unittest.cc
[modify] https://crrev.com/9edd41e5c77d248e17b77f16af6d5aa19992093a/net/http/mock_allow_http_auth_preferences.cc

Status: Fixed (was: Started)

Comment 4 by dchan@chromium.org, Jan 22 2018

Status: Archived (was: Fixed)

Comment 5 by dchan@chromium.org, Jan 23 2018

Status: Fixed (was: Archived)
Status: Verified (was: Fixed)
Marked as verified based on the code changes and code review already completed.

Sign in to add a comment