Commit Graph

7932 Commits

Author SHA1 Message Date
Hunter Knepshield
8a01758622 Merge changes from topic "bugreport-stub" into rvc-dev am: 9568960297 am: a7dad0b5cc
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/11856802

Change-Id: I38f827361f1f234e49c086953d0c0d7426e0d70f
2020-06-18 17:18:19 +00:00
Hunter Knepshield
9568960297 Merge changes from topic "bugreport-stub" into rvc-dev
* changes:
  Reevaluate some carrier-associated apps' status on SDK changes.
  Add new "addedInSdk" attribute to carrier-associated apps.
2020-06-18 17:01:01 +00:00
Luke Huang
dae5d3f223 Merge "Disable sockets and DNS if process lacks INTERNET permission." into rvc-dev am: 7144459282
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/11881939

Change-Id: I7c285a848ac947e0acf1acc8bc32f3ece3456412
2020-06-18 03:23:46 +00:00
Luke Huang
7144459282 Merge "Disable sockets and DNS if process lacks INTERNET permission." into rvc-dev 2020-06-18 03:03:06 +00:00
Luke Huang
6f214e8ebe Disable sockets and DNS if process lacks INTERNET permission.
This is a Client-only solution.
  - Add to NetdClient a per-process std::atomic_boolean
    similar to netIdForProcess and netIdForResolv.
  - The boolean says whether the process should be
    allowed Internet connectivity.
  - Add an @hide method to NetUtils.java to set the boolean;
    call it from the initialization code of the new
    process just after forking from zygote.
  - Make netdClientSocket and dnsOpenProxy check the
    boolean. If the boolean is false, return EPERM from
    socket calls.

Bug: 150028556
Test: atest NetworkUtilsTest
Test: atest CtsAppSecurityHostTestCases:UseProcessTest
Change-Id: If002280fbad493dfc2db3d9d505c0257d49a9056
Exempt-From-Owner-Approval: OWNERS already approved identical patchset 5
2020-06-18 03:02:06 +00:00
TreeHugger Robot
feeb4ab192 Merge "Clarify doc for users, clarifying profiles" into rvc-dev am: d41aca1009
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/11889023

Change-Id: I748a8b1af01b8e96c8a3e522655e77aa81683e4f
2020-06-18 02:28:08 +00:00
Adam Bookatz
36bacceee5 Clarify doc for users, clarifying profiles
Test: N/A
Bug: 159173436
Change-Id: I3c105559801a72ef1d91d9db2d42a816e27e24f6
Merged-In: I3c105559801a72ef1d91d9db2d42a816e27e24f6
2020-06-18 00:08:51 +00:00
Hunter Knepshield
1172ffa7c2 Add new "addedInSdk" attribute to carrier-associated apps.
Previously, the sysconfig wasn't capable of understanding
carrier-associated apps that were added after a device's initial launch
(i.e. via OTA) because the logic in CarrierAppUtils explicitly avoids
disabling such apps a second time.

Most of this change is just plumbing everything through. For now, it's
all @hide due to R API deadlines. It will be made public in S.

Bug: 154872019
Test: manual, QA, atest FrameworksTelephonyTests:CarrierAppUtilsTest
Change-Id: I530a4f73146b09879547ca2e0c26428957fef37a
2020-06-16 16:48:38 -07:00
Nikita Ioffe
4a911583b3 Merge "Respect MANAGE_EXTERNAL_STORAGE in checks for READ/WRITE_EXTERNAL_STORAGE" into rvc-dev am: 27d6605c52
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/11797136

Change-Id: I1446f282205ed5da4256ca9995e3d97020525f50
2020-06-16 10:20:37 +00:00
Nikita Ioffe
27d6605c52 Merge "Respect MANAGE_EXTERNAL_STORAGE in checks for READ/WRITE_EXTERNAL_STORAGE" into rvc-dev 2020-06-16 10:02:49 +00:00
TreeHugger Robot
84f65247cb Merge "Don't provide read logs for shell-initiated installations." into rvc-dev am: e0ba6d9dff
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/11778056

Change-Id: I866bc89927ae11a0da7069d33e3e06d5c1f1b51f
2020-06-15 16:42:24 +00:00
Nikita Ioffe
fc3a740282 Respect MANAGE_EXTERNAL_STORAGE in checks for READ/WRITE_EXTERNAL_STORAGE
Test: test app mentioned in the bug
Bug: 158316513
Change-Id: I254479d6616d2e09349fcd9c569c5092dbdefaca
2020-06-15 13:01:06 +01:00
Alex Buynytskyy
0403545508 Don't provide read logs for shell-initiated installations.
Only if the application is profileable.

Bug: 158238023
Fixes: 158238023
Test: atest PackageManagerShellCommandTest PackageManagerShellCommandIncrementalTest IncrementalServiceTest PackageParserTest
Change-Id: I8575830ec3f29850297fdbfbaa157072d6350a28
Merged-In: I8575830ec3f29850297fdbfbaa157072d6350a28
2020-06-12 13:30:45 -07:00
Adam Bookatz
dae6bc2c9f Merge "Default isUserSwitcherEnabled showEvenIfNotActionable" into rvc-dev am: a677e1c6f9
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/11829304

Change-Id: Ib25f90dbda73f5eeb05823d7672c33637a5e1e35
2020-06-12 01:00:47 +00:00
Adam Bookatz
a677e1c6f9 Merge "Default isUserSwitcherEnabled showEvenIfNotActionable" into rvc-dev 2020-06-12 00:47:47 +00:00
TreeHugger Robot
bdac8459fb Merge "Add details to VibrationEffect waveform documentation" into rvc-dev am: 19dd14d76a
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/11837599

Change-Id: Ic47e871ca849268ffaf8e8b759f2ceb9b93d6e48
2020-06-11 17:55:27 +00:00
Lais Andrade
ef5d8751e6 Add details to VibrationEffect waveform documentation
Explain the time unit of timings argument and copy explanation of
amplitude values to the summary segments of the javadocs.

Fix: 140846849
Test: N/A
Change-Id: Ia6fd6cb5d0e16076a54f24f7c3d2f3d4ca78d5bd
2020-06-11 16:39:36 +00:00
Adam Bookatz
600210d7cb Default isUserSwitcherEnabled showEvenIfNotActionable
In ag/11573771, the default value for isUserSwitcherEnabled() was
put as false, but it was supposed to be true (as it had been
in patch 1 of that cl).

Bug: 157881376
Test: manual
Change-Id: I456e7eb9527dad8da5d5f99ecbfc8bcf40b7bdcc
2020-06-11 00:22:37 +00:00
TreeHugger Robot
dc73a09889 Merge "Apply app data isolation on isolated or app zygote processes" into rvc-dev am: 8434b9fc88
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/11689593

Change-Id: Iee4baeba4580c74275ef54a93260050804a8c9cd
2020-06-08 11:07:57 +00:00
TreeHugger Robot
8434b9fc88 Merge "Apply app data isolation on isolated or app zygote processes" into rvc-dev 2020-06-08 11:03:56 +00:00
Ricky Wai
e986bc4cad Apply app data isolation on isolated or app zygote processes
Fix: 158088415
Test: atest AppDataIsolationTests
Change-Id: Ic55464397039645938822ac20eb12e375bbe4a72
2020-06-05 11:50:50 +01:00
TreeHugger Robot
224ce04e42 Merge "Get the instance count as the initial value" into rvc-dev am: d5fb11a761
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/11473225

Change-Id: I5277cd97e46659df8c0398371f86c9880efca9ef
2020-06-05 03:36:49 +00:00
TreeHugger Robot
d5fb11a761 Merge "Get the instance count as the initial value" into rvc-dev 2020-06-05 03:34:32 +00:00
Jeff Sharkey
5bb776d88b Merge "Logging to aid bug investigation." into rvc-dev am: bb9ea4e1fa
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/11745078

Change-Id: I72ef1b7bf14215c76ff67aea1941ab43a27f0e02
2020-06-04 23:51:12 +00:00
Jeff Sharkey
0a8e32c124 Logging to aid bug investigation.
We're trying to figure out where volumes are disappearing.

Bug: 158059178
Test: none
Change-Id: I68959f0a163e2fece13fcb08a6c9f55ec1beb0f6
2020-06-04 17:35:38 -06:00
Peiyong Lin
760338ed90 Merge "Allow to load implicit layer from /vendor/app." into rvc-dev am: 1371b76afd
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/11688191

Change-Id: I34257bdfd340334bcdfc19a3f6a9c5d9aeebe842
2020-06-04 00:53:47 +00:00
Peiyong Lin
1371b76afd Merge "Allow to load implicit layer from /vendor/app." into rvc-dev 2020-06-04 00:40:15 +00:00
Peiyong Lin
eb1c5a4c28 Allow to load implicit layer from /vendor/app.
Currently when loading implicit layers from apks, NativeLoaderNamespace
doesn't allow to dlopen the binaries if they come from apks from
/vendor/app. Implicit layers ship within /vendor/app should work like
other implicit layers. This patch extracts the construction of library
paths of the implicit layers and includes those paths when
NativeLoaderNamespace is created as the part of the permitted library
paths.

Bug: b/157832445
Test: atest android.gputools.cts.CtsRootlessGpuDebugHostTest
Test: setup debug layer and use adb logcat to check vulkan loader output
Change-Id: Ie2ca989bcab890578b5aa540d07f2aee2a0182bd
2020-06-03 21:30:08 +00:00
Zhen Zhang
08c915ea4c Merge "Allow low ram device to have multiple users and managed profies" into rvc-dev am: 2260f7102f
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/11667418

Change-Id: Id93d5b5bea8571ef5f8e177d5c17963d5ba8e746
2020-06-03 17:31:40 +00:00
Zhen Zhang
2260f7102f Merge "Allow low ram device to have multiple users and managed profies" into rvc-dev 2020-06-03 17:19:30 +00:00
Treehugger Robot
61538a7ff8 Merge "Use device SKU for Build.SKU instead of product/odm SKU" am: d1acd32722 am: 5dbd259527
Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/1318696

Change-Id: I60137d2247a7a7033d04f669f4f90c5e2644c218
2020-06-02 21:17:20 +00:00
Treehugger Robot
5dbd259527 Merge "Use device SKU for Build.SKU instead of product/odm SKU" am: d1acd32722
Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/1318696

Change-Id: Ida9865813088b5b1c6a0d0a728acb93dbe75cfda
2020-06-02 21:01:22 +00:00
Treehugger Robot
d1acd32722 Merge "Use device SKU for Build.SKU instead of product/odm SKU" 2020-06-02 20:48:33 +00:00
Martijn Coenen
5dab822067 Merge "Use killProcessGroup when killing app zygote." into rvc-dev am: 0a8ef2b45f
Original change: undetermined

Change-Id: I6033bc141efd30efb832c8cde26a2312d4f3c177
2020-06-02 07:08:35 +00:00
Martijn Coenen
0a8ef2b45f Merge "Use killProcessGroup when killing app zygote." into rvc-dev 2020-06-02 06:54:46 +00:00
Jeff Sharkey
459f2c806b Merge "Revert "Apps shouldn't be disabling StrictMode checks."" into rvc-dev am: c99a508218
Change-Id: I5d4a4161fa0c9e9ab8bef9a543c357bab4576408
2020-06-01 16:04:05 +00:00
Jeff Sharkey
c99a508218 Merge "Revert "Apps shouldn't be disabling StrictMode checks."" into rvc-dev 2020-06-01 15:55:14 +00:00
Jeff Sharkey
3080fb7c9e Revert "Apps shouldn't be disabling StrictMode checks."
This reverts commit 14e961870a.

Reason for revert: satayev@
Bug: 156336269

Change-Id: Ic0d3a650156c06f58fce4c28157191f4497469f0
2020-06-01 15:54:41 +00:00
Jeff Sharkey
e42cbb8704 Merge "Support getStorageVolume() for synthetic volumes." into rvc-dev am: fd71805e1c
Change-Id: I5f6675f1514ed308014e833d28e1b3a3cbe8c93a
2020-06-01 14:38:01 +00:00
Jeff Sharkey
f05d514436 Merge "Apps shouldn't be disabling StrictMode checks." into rvc-dev am: 4d96602de0
Change-Id: Iaf92a5949c9196d5d4194322e360d114325a4eea
2020-06-01 14:37:23 +00:00
Jeff Sharkey
fd71805e1c Merge "Support getStorageVolume() for synthetic volumes." into rvc-dev 2020-06-01 14:18:32 +00:00
Jeff Sharkey
4d96602de0 Merge "Apps shouldn't be disabling StrictMode checks." into rvc-dev 2020-06-01 14:16:54 +00:00
TreeHugger Robot
4af2991795 Merge "Fixed how ShellCommand show exceptions." into rvc-dev am: 725f9cea7f
Change-Id: I2b8d3c4941c79663d5f02e16b244801bbd7d97ea
2020-05-29 23:01:16 +00:00
Zhen Zhang
bd16014d0b Allow low ram device to have multiple users and managed profies
Remove these runtime restrictions of low ram device.
This change wouldn't bring the features onto low ram device, because
manufacturers need to change some configs to enable the features, but
would allow the usage.

Bug: 153016458
Test: build successful; multiple users and managed profiles can be
enabled with config changes

Change-Id: Ibe59219dab7e138d7dae19df77d36477e9e76b8e
2020-05-29 22:29:01 +00:00
Treehugger Robot
df04459b1d Merge "Add ParcelableHolder type" am: 7d2fc62868 am: 841a116b7e
Change-Id: Ibb9629999638c868e453889592b8290673be04c9
2020-05-29 22:08:18 +00:00
Treehugger Robot
841a116b7e Merge "Add ParcelableHolder type" am: 7d2fc62868
Change-Id: If7258d57ff14020c3127aa7eadd892ed51301cd4
2020-05-29 21:42:41 +00:00
felipeal
283e3c388d Fixed how ShellCommand show exceptions.
Prior to this change, it was duplicating the exception. Example:

$ adb shell cmd car_service create-user --flags 4444 --timeout 100 --hal-only Dude

Exception occurred while executing: new user flags mismatch on existing users on {.requestId = 2, .newUserInfo = {.userId = 11, .flags = 0x115c}, .newUserName = Dude, .usersInfo = {.currentUser = {.userId = 11, .flags = NONE}, .numberUsers = 3, .existingUsers = [{.userId = 0, .flags = 0x9}, {.userId = 10, .flags = ADMIN}, {.userId = 11, .flags = NONE}]}}
java.lang.IllegalArgumentException: new user flags mismatch on existing users on {.requestId = 2, .newUserInfo = {.userId = 11, .flags = 0x115c}, .newUserName = Dude, .usersInfo = {.currentUser = {.userId = 11, .flags = NONE}, .numberUsers = 3, .existingUsers = [{.userId = 0, .flags = 0x9}, {.userId = 10, .flags = ADMIN}, {.userId = 11, .flags = NONE}]}}
	at com.android.internal.util.Preconditions.checkArgument(Preconditions.java:69)
	at android.car.userlib.UserHalHelper.toVehiclePropValue(UserHalHelper.java:503)
	at com.android.car.hal.UserHalService.createUser(UserHalService.java:339)

Now it's showing the failed command instead:

Exception occurred while executing 'create-user --flags 4444 --timeout 100 --hal-only Dude':
java.lang.IllegalArgumentException: new user flags mismatch on existing users on {.requestId = 2, .newUserInfo = {.userId = 11, .flags = 0x115c}, .newUserName = Dude, .usersInfo = {.currentUser = {.userId = 11, .flags = NONE}, .numberUsers = 3, .existingUsers = [{.userId = 0, .flags = 0x9}, {.userId = 10, .flags = ADMIN}, {.userId = 11, .flags = NONE}]}}
	at com.android.internal.util.Preconditions.checkArgument(Preconditions.java:69)
	at android.car.userlib.UserHalHelper.toVehiclePropValue(UserHalHelper.java:503)
	at com.android.car.hal.UserHalService.createUser(UserHalService.java:339)

Test: manual verification
Bug: 150408921

Change-Id: I42221fbe2410770b0755c32d16620acc71d8d5d2
2020-05-29 21:21:56 +00:00
Martijn Coenen
6bec07ec39 Use killProcessGroup when killing app zygote.
To make sure we kill all untracked children, too.

Bug: 156741968
Bug: 157598956
Test: manual inspection, PoC no longer works.
Change-Id: I5d8efeb05ddec08a7fc7c00eabca6590c4cfdd8c
2020-05-29 14:49:25 +02:00
Treehugger Robot
7d2fc62868 Merge "Add ParcelableHolder type" 2020-05-29 04:05:46 +00:00
Jeff Sharkey
88818fe7b2 Support getStorageVolume() for synthetic volumes.
The MediaStore.VOLUME_EXTERNAL volume is a synthetic collection of
all currently mounted storage volumes, so if someone wants to find
the underlying StorageVolume for one of these Uris, we need to query
to resolve the real volume name.

Bug: 153664437
Test: atest android.provider.cts.MediaStoreTest
Change-Id: I7b28ba3bfd5b7a4cc577830940de4a2e1cbd1932
2020-05-28 19:18:40 -06:00