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

Issue 637868 link

Starred by 2 users

Issue metadata

Status: Verified
Owner:
Last visit > 30 days ago
Closed: Aug 2016
Cc:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

security_StatefulPermissions Failure due to unexpected owners

Project Member Reported by benzh@chromium.org, Aug 15 2016

Issue description

The test has been failing intermittently on the release builders. e.g.
https://uberchromegw.corp.google.com/i/chromeos/builders/samus-release/builds/3473

auto-filed bug is at https://bugs.chromium.org/p/chromium/issues/detail?id=604606

The log shows many files with unexpected owners, printed by self.log_owned_files(). e.g.
/mnt/stateful_partition/home/.shadow/b822d9f466a13b401258726d72896bdf7405a119/vault/root/ECRYPTFS_FNEK_ENCRYPTED.FWZ5IFbq7p4z--RDYbZineRMfy00cKh0rz-68FYz5h7QrUM93hgi8shtwU--/ECRYPTFS_FNEK_ENCRYPTED.FWZ5IFbq7p4z--RDYbZineRMfy00cKh0rz-6RFOTCnQhhlFTz5zBvO4SCk--/ECRYPTFS_FNEK_ENCRYPTED.FWZ5IFbq7p4z--RDYbZineRMfy00cKh0rz-6RFOTCnQhhlFTz5zBvO4SCk--/ECRYPTFS_FNEK_ENCRYPTED.FXZ5IFbq7p4z--RDYbZineRMfy00cKh0rz-6JUxke221fV5Usrt0LeMt2cPAO0kx7Dj3PZIcYseuWgM-

These files are expected to be there after android runs. We should fix the test by ignoring them.

 

Comment 1 by benzh@chromium.org, Aug 16 2016

Owner: benzh@chromium.org
Status: Started (was: Available)
Fix is at https://chromium-review.googlesource.com/#/c/371338/

Comment 2 by nya@chromium.org, Aug 22 2016

Cc: nya@chromium.org
Project Member

Comment 3 by bugdroid1@chromium.org, Aug 25 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/c4f8865f3788dab086cd75760a1596194448dd7a

commit c4f8865f3788dab086cd75760a1596194448dd7a
Author: Ben Zhang <benzh@chromium.org>
Date: Tue Aug 16 18:01:51 2016

security_StatefulPermissions: Allow android users

Android creates files in the stateful partition with uids in
the range (android-root, android-root + 100k). These files
are expected to be created under
android-data/cache
android-data/data

This patch whitelists the cache and data directories.

BUG= chromium:637868 
TEST=On Samus, start Play store, run this test, see android users
are ignored in security_StatefulPermissions.DEBUG

Change-Id: I0de9aadcd827b46b049e83009388483f625a0a8d
Signed-off-by: Ben Zhang <benzh@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/371338
Reviewed-by: Ricky Zhou <rickyz@chromium.org>

[modify] https://crrev.com/c4f8865f3788dab086cd75760a1596194448dd7a/client/site_tests/security_StatefulPermissions/security_StatefulPermissions.py

Comment 4 by benzh@chromium.org, Aug 25 2016

Status: Fixed (was: Started)
Labels: VerifyIn-54
bulk verified
Status: Verified (was: Fixed)

Sign in to add a comment