diff options
| author | Farid Zare Seisan <faridzs@google.com> | 2018-03-30 11:19:10 -0700 |
|---|---|---|
| committer | Farid Zare Seisan <faridzs@google.com> | 2018-04-30 14:36:47 -0700 |
| commit | 7f6b8120da3de99bace28eb1c1602016d0e5fbb0 (patch) | |
| tree | f9cadf45179ef11d5f75a138fb41e11ecbe3b3aa /tools/aapt2/java/JavaClassGenerator_test.cpp | |
| parent | 1072d0d18a8701e5fac0d7e545aed7a620f706b3 (diff) | |
UserDataPrepare: CE storage property set too early.
Set sys.user.[userId].ce_available only after CE
storage has been prepared. This property was being set
too early. This will cause a race for first boot between
preparing these folders and writting to them. If CE
storage is not empty before setting encryption policy
then setting encryption policy would fail.
Also, if setting CE storage fails, we should not destroy
DE storage since recovery is only making call to retry
setting up CE.
Only when userId == UserHandle.USER_SYSTEM then mkdri
should fail when CE property isn't set.
Bug: 76222913
Bug: 76102397
Bug: 78514593
Bug: 78515110
Test: Waiting for property and no longer seeing the race.
Verified below cts failures are now passing.
android.appsecurity.cts.ExternalStorageHostTest#testMultiViewMoveConsistenc
android.appsecurity.cts.ExternalStorageHostTest#testExternalStorageWrite
android.appsecurity.cts.ExternalStorageHostTest#testExternalStorageNone
Change-Id: Ied0308ef099090e8f3954e905f78320214e14335
(cherry picked from commit 420e825111ca5c95c6200c1fb4681c315906e71e)
Diffstat (limited to 'tools/aapt2/java/JavaClassGenerator_test.cpp')
0 files changed, 0 insertions, 0 deletions