New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 740753 link

Starred by 2 users

Issue metadata

Status: Verified
Owner:
Last visit > 30 days ago
Closed: Nov 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

cryptohomed: add support for different profiles in TpmAttestationCreateCertRequest under ServiceDistributed

Project Member Reported by mvanotti@google.com, Jul 10 2017

Issue description

When cryptohomed runs in ServiceDistributed mode, the TpmAttestationCreateCertRequest method doesn't use the parameters of the DBus call to create the cert request. Instead, an empty request is created.
 
Project Member

Comment 1 by sheriffbot@chromium.org, Jul 11 2017

Labels: Hotlist-Google
Project Member

Comment 2 by bugdroid1@chromium.org, Jul 12 2017

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

commit 3f9ad82e9dcd920cec9faa8d7dc71a54d3ba0983
Author: Marco Vanotti <mvanotti@google.com>
Date: Wed Jul 12 00:48:04 2017

cryptohome: fix CreateCertRequest

When cryptohome is working on distributed mode, the CreateCertRequest
method ignores the request information (username, profile, origin).

This change sets those fields in the request.

BUG= chromium:740753 
TEST=tested on device.
Change-Id: I0e7287704c45aa826162f72e7cf66c04d590e25c
Reviewed-on: https://chromium-review.googlesource.com/566038
Commit-Ready: Marco Vanotti <mvanotti@google.com>
Tested-by: Marco Vanotti <mvanotti@google.com>
Reviewed-by: Andrey Pronin <apronin@chromium.org>

[modify] https://crrev.com/3f9ad82e9dcd920cec9faa8d7dc71a54d3ba0983/cryptohome/service_distributed.cc

Comment 3 by mvanotti@google.com, Nov 27 2017

Status: Verified (was: Untriaged)

Sign in to add a comment