Commit Graph

3770 Commits

Author SHA1 Message Date
Jeff Sharkey
c3132513bf Parsing a static archive should be stateless.
When the caller hasn't specific encryption-related matching flags,
we should match both aware and unaware components.

Bug: 26508249
Change-Id: I2c35f6e00e451ba3f5fa0810223b7a3d80dee233
2016-01-12 14:07:00 -07:00
Jeff Sharkey
9229ee5959 Merge "Install non-EA providers once user is unlocked." 2016-01-12 03:16:22 +00:00
Todd Kennedy
12705131b9 Hide ephemerality
Change-Id: Ibf60366b3f00b1920e330cd804bcc1c550f8882a

Bug: 26404647
2016-01-11 16:21:03 -08:00
Jeff Sharkey
5eeefba37c Merge "Temporary patch to work around broken casting." 2016-01-11 03:41:22 +00:00
Jeff Sharkey
6c4b87be9e Temporary patch to work around broken casting.
Bug: 26481405
Change-Id: If91ecfbfda94a2cb694098d456126f87311e1d05
2016-01-10 20:40:08 -07:00
Jeff Sharkey
bc215b8426 Merge "Fix NPE when UID has no packages." 2016-01-10 20:18:20 +00:00
Jeff Sharkey
377ded0fce Fix NPE when UID has no packages.
Bug: 25224723
Change-Id: I637214666a1f02bd23474c27a285eea9152cfa9d
2016-01-10 13:15:43 -07:00
Jeff Sharkey
2bd31dbd02 Install non-EA providers once user is unlocked.
When starting encryption-aware apps while the device is locked, we
can only spin up ContentProviders that have been marked as
encryption-aware.  Once the user is unlocked, we need to go back and
install non-encryption-aware providers in already running apps.

Fix bugs in getPackageInfo() where only one of the various MATCH_
flags was being consulted (!).  Move matching logic to single unified
location in PackageUserState so we have consistent behavior.

Fix another class of bugs where Safe Mode wasn't correctly filtering
package details (!).  These bugs are fixed by splicing in the new
MATCH_SYSTEM_ONLY flag as part of state-based flag mutation that was
added for encryption.

Bug: 25944787
Change-Id: I39c8da74b1f9ba944cc817176983f50ba322329c
2016-01-09 19:28:09 -07:00
Jeff Sharkey
3ea44a52f0 Device encrypted storage is always available.
In order to support backup/restore on devices without native FBE
support, we always need to make the DE storage area available.  Add
docs clarifying the lifecycle.

Bug: 26279618
Change-Id: I789915f295b10aca6bf80ca58406aea212835ffd
2016-01-09 15:34:44 -07:00
Jeff Sharkey
5aa86938b8 Fix docs build by removing bad links.
Also define InstallFlags and DeleteFlags, and point installPackage()
users towards new PackageInstaller APIs.  Remove old movePackage()
API that is no longer used.

Change-Id: I661a65149e4778a7591dbd912b72b4dd81b266da
2016-01-08 19:13:43 -07:00
Amith Yamasani
2b60ca45a1 Fix typo in javadoc
Change-Id: I7a61f84f9d19d5ebc64c3a6e01eea42c3fac4fed
2016-01-08 17:53:18 -08:00
Jeff Sharkey
cd65448ccd Even more PackageManager caller triage.
Finish moving all UID/GID callers to single AIDL method that requires
callers to provide flags.

Triage AppWidgets and PrintServices, which currently can only live on
internal storage; we should revisit that later.

Fix two bugs where we'd drop pending install sessions and persisted
Uri grants for apps installed on external storage.

Bug: 26471205
Change-Id: I66fdfc737fda0042050d81ff8839de55c2b4effd
2016-01-08 18:35:54 -07:00
Jeff Sharkey
629f98466a Fix build.
Change-Id: If128c90225a2237ea015063cec526a64af5bf576
2016-01-08 16:36:54 -07:00
Jeff Sharkey
c4fd3c9871 Merge "Update javadoc for flags" 2016-01-08 23:18:13 +00:00
Todd Kennedy
6b9bfa1d16 Update javadoc for flags
Bug: 25997037
Change-Id: Ia44634b0271ad0195d857ef13afec2a1ef022fae
2016-01-08 16:16:25 -07:00
Jeff Sharkey
0ab0bdde2b Merge "More progress on triaging PackageManager callers." 2016-01-08 23:14:25 +00:00
Jeff Sharkey
c5967e9862 More progress on triaging PackageManager callers.
Catch a bunch of simple cases where the PackageManager flags are
obvious.  Add the ability to use the MATCH_SYSTEM_ONLY flag on
PackageInfo and ApplicationInfo queries.

Re-examine recent tasks after a user is unlocked, since some of the
activities may now be available and runnable.

Bug: 26471205, 26253870
Change-Id: I989d9f8409070e5cae13202b47e2c7de85bf4a5b
2016-01-08 16:10:33 -07:00
Jeff Sharkey
bd94022893 Oh look, GmsCore is using more hidden APIs.
Bug: 26429932
Change-Id: Ie949d4bbdead8864aaed16282a591258ef528ca8
2016-01-08 11:10:32 -07:00
Jeff Sharkey
703fe24e25 Null cursors don't need to be wrapped.
Change-Id: I59b604a60a066c2d808a796658757af952ef913c
2016-01-07 18:52:26 -07:00
Jeff Sharkey
ba75a9b6ba Instrumentation targets must have code.
If they don't have code, give the developer a somewhat helpful error
message instead of later falling into a weird classpath failure.

Change-Id: Iebda10173ff99943cbbd71127ae24aa455b709f4
2016-01-07 12:03:16 -07:00
Jeff Sharkey
8588bc1ef1 Add flags to requests for package UID/GIDs.
This gives callers the ability to request details for missing
packages.  Also add annotations for userId and appId variables and
start tagging their usage.

Change-Id: I63d5d7f870ac4b7ebae501e0ba4f40e08b14f3f6
2016-01-07 10:33:26 -07:00
Jeff Sharkey
8b0db0401e GmsCore is casting to a concrete subclass, sigh.
Since this technically wasn't an API change, there isn't a good
reason to protect with target API.  So change the behavior based on
the caller for now.

Bug: 26438049
Change-Id: I65aa1fc9af1f935544f8e191444440af4f98f26f
2016-01-07 09:47:23 -07:00
Jeff Sharkey
a73b8fd1f4 Add some TestApis to fix build.
Change-Id: Icf0d8d63f3a41d99e46ebe96003863bae637bd9d
2016-01-06 17:37:32 -07:00
Jeff Sharkey
51f3908c6a Merge "Consistent naming for PackageManager methods." 2016-01-06 22:26:16 +00:00
Jeff Sharkey
e06b4d1d9f Consistent naming for PackageManager methods.
When hidden PackageManager methods take a userId argument, they
should be named explicitly with the "AsUser" suffix.  This fixes
several lagging examples so that we can pave the way to safely
start passing flags to new methods without scary overloading.

Also fix spacing issues in various logging statements.

Change-Id: I1e42f7f66427410275df713bea04f6e0445fba28
2016-01-06 15:21:06 -07:00
Jeff Sharkey
81cf69a3b4 Merge "Update logic for resolving verifiers." 2016-01-06 21:50:06 +00:00
Jeff Sharkey
2a90f673f5 Update logic for resolving verifiers.
Verifiers or installers may not be encryption-aware, or the user may
have disabled them, so we probe pretty deeply during system boot to
resolve them.  Use the new MATCH_SYSTEM_ONLY flag to limit results to
packages on the system image.

When there are multiple matches, pick the one with highest priority
instead of crashing the system.

Switch to updated MATCH_ constants in more places.

Bug: 26250295
Change-Id: Ia7a3b1fb74da6c3b9d2c2edbf1deaa9fb52fc40a
2016-01-06 13:49:55 -07:00
Alan Viverette
946502f50a Merge "Cache XML blocks against file name rather than resource ID" 2016-01-06 20:36:05 +00:00
Alan Viverette
b7e3363359 Merge "Clean up asset manager package-private methods" 2016-01-06 20:29:49 +00:00
Alan Viverette
ec76b2a1d8 Merge "Skip the cache when loading drawables from a foreign density" 2016-01-06 20:29:19 +00:00
Alan Viverette
1097961389 Cache XML blocks against file name rather than resource ID
Caching against resource ID leads to incorrect cache hits, since multiple
files (ex. within drawable-mdpi, drawable-ldpi) may map to the same ID.

Also adds nullability annotations.

Bug: 26400880
Change-Id: I4d83caf3c44dc9b546511753e9e72171e8850eb2
2016-01-06 15:27:35 -05:00
Jeff Sharkey
11dce73277 Merge "Add Ethernet feature." 2016-01-06 18:04:57 +00:00
Jeff Sharkey
4efb8b4030 Add Ethernet feature.
Change-Id: I1f1badff5a8c9237d9489497e36e2578266e34b4
2016-01-06 11:02:52 -07:00
Jeff Sharkey
62728e3107 Merge "Discourage use of "_data" column." 2016-01-06 17:26:25 +00:00
Jeff Sharkey
60cfad80bd Discourage use of "_data" column.
Moving forward, all client file access really needs to be going
through explicit APIs like openFileDescriptor(), since that allows
the provider to better protect its underlying files.

This change also changes several classes to use the AutoClosable
pattern, which enables try-with-resources usage.  Older release()
methods are deprecated in favor of close().

Uniformly apply CloseGuard across several classes, using
AtomicBoolean to avoid double-freeing, and fix several resource
leaks and bugs related to MediaScanner allocation.  Switch
MediaScanner and friends to use public API instead of raw AIDL calls.

Bug: 22958127
Change-Id: Id722379f72c9e4b80d8b72550d7ce90e5e2bc786
2016-01-06 10:19:35 -07:00
Clara Bayarri
8819c61bcf Merge "Work Challenge: Handle Recents launches" 2016-01-06 16:57:55 +00:00
Clara Bayarri
ea9b10e644 Work Challenge: Handle Recents launches
Intercept calls to start activities from the recents
stack and show the Work Challenge if needed. This requires
passing the taskId to ConfirmDeviceCredential so it can
launch the recents task itself when the credentials are
confirmed.

Change-Id: I013b134f3f31a35b551ad683c68cc89b8af44499
2016-01-06 16:17:08 +00:00
Alan Viverette
7941a7fd8e Skip the cache when loading drawables from a foreign density
Bug: 26400880
Change-Id: I87c06b30afc61495fe62aa6a9dfe77990200cde7
2016-01-06 11:11:53 -05:00
Roozbeh Pournader
fb9236cb0c For locale negotiation, assume English is always supported
In practice, a lot of apps provide English resources as their default
resources, so assume that English is always supported.

With this change, users can safely set secondary locales after
English in their preferred locale lists. Such settings would help
the apps (and parts of the system that are aware of multiple locales)
consider the user's familiarity with the secondary locale or the
user's preference for glyphs forms in the secondary locale.  This
change makes sure that no app unintentionally switches to such
secondary locales.

Note that this doesn't break from the tradition of previous releases.
Traditionally, if the user had set the system locale to English, they
would get the default resources when the app didn't explicitly
provide English resources. This will continue to be the case where
the user's locale list has English in it before another supported
language. (English will be the first locale in most cases.)

Bug: 26192817
Change-Id: I2c1a003760299157786c1b3f9fb116ce18db3b8c
2016-01-05 14:52:38 -08:00
Alex Klyubin
4f6d400b0f Merge "No need to pass digest of AndroidManifest.xml around." 2016-01-05 21:19:41 +00:00
Roozbeh Pournader
4fac768030 Merge "Avoid matching system locales in locale negotiation" 2016-01-05 18:45:29 +00:00
Alan Viverette
e273b93651 Clean up asset manager package-private methods
Refactors duplicate code, flattens methods by inverting if blocks, updates
nullability annotations, fills out missing javadoc, uses consistent
variable names. No functional changes.

Change-Id: I4c3a139197697a9d6f6afc9c428a193c02fc777c
2016-01-05 10:25:54 -05:00
Wale Ogunwale
6cae765b67 Added support for android.R.attr#alwaysFocusable
Allows an activity to always be focusable regardless of if it is in a
stack whose activities are normally not focusable. For example, activities
in pinned stack aren't focusable. This flag allows them to be focusable.

Also, changed ActivityInfo.#{resizeable, supportsPip} to use flags.

Bug: 26273032
Bug: 26034613
Change-Id: I8c63e6d3256757e2e6931e08b8a65269f5169d35
2016-01-04 22:23:56 +00:00
Roozbeh Pournader
1c686f2ce6 Avoid matching system locales in locale negotiation
Also:
1. Add AssetManager method for finding non-system locales: This is
used in per-app locale negotiation. (Normally,
AssetManager#getLocales() returns both system and non-system
locales.)

2. Match pseudolocales correctly in locale negotiation.

Bug: 25800576
Bug: 26236938
Change-Id: I116caf3a91c290deb4ad68b291c65b7035b18dd4
2016-01-04 13:53:54 -08:00
Alex Klyubin
31ffb44241 No need to pass digest of AndroidManifest.xml around.
When an APK is being installed, the full path to the file is
occasionally passed between processes. To detect changes to the file
during these handovers, the digest of the AndroidManifest.xml entry
could be passed around as well.

This security feature is no longer used and is no longer needed. APKs
should be installed using the modern Package Installer API
(android.content.pm.PackageInstaller).

Bug: 24542768
Change-Id: I2762634c85448e7adcd47ffc9a5294c021e127bd
2016-01-04 12:31:29 -08:00
Jeff Sharkey
2f3e35376a More work towards triaging missing app behavior.
Many places across the platform query package details without
gracefully handling packages or components that go missing for
various reasons.  This can cause annoying user data loss, such as
resetting back to built-in apps or dropping of accounts, etc.

This change verifies that system callers have thought about these
edge cases by logging if they use default matching behaviors without
explicitly marking themselves as being "triaged."  (The logging is
currently disabled by default.)

Also creates explicit definitions of supported flags for various
incoming PackageManager calls, and defines a clear distinction
between flag types:

-- GET-style flags are used to request additional data that may have
been elided to save wire space.

-- MATCH-style flags are used to include components or packages that
would have otherwise been omitted from a result set by current system
state.

There are a handful of existing GET flags that better fit under the
MATCH definition, so this change clones them to new constants and
marks the old ones as deprecated.

Fixes bug in JobSchedulerService to consider jobs from apps on
external storage.  Revert some dialer behavior back to being
untriaged.

Change-Id: I9b6ab0968241e3479bddbd78de0c51e3b9917318
2016-01-04 13:19:02 -07:00
Jeff Sharkey
5217cacbd9 Make JobSchedulerService encryption aware.
When a user is started, but a persisted job component doesn't appear
in the normal resolution list, we avoid enqueuing the job.  Later
when the user is unlocked, we take another pass over the pending
jobs to see if they became available.

Load keyboard layouts from XML metadata regardless of crypto status,
since we don't need to spin up any remote code.

Add MATCH_SYSTEM_ONLY to make system logic easier to write when
looking for trusted components.

Sprinkle more annotations on ArrayUtils methods.

Bug: 26279465
Change-Id: Iec28e0bb46862b07d740b12a79f6360de68dab0f
2015-12-20 15:36:29 -07:00
Jeff Sharkey
2a9e3f8e68 Better named encryption flags, start triaging.
Create distinct flags for encryption aware, unaware, and both, and
name them like the other MATCH_ flags.

Start adding logic to help triage all system internal callers to
verify that they've done their homework and thought about how to
handle apps while locked.  Call sites in the system should either
ask for explicit matching behavior, or explicitly use the DEFAULT
match flag to indicate that they've been triaged to use the
default state-based matching.

Bug: 26250295
Change-Id: I86214e5c4f71a6dc72f06930800388713aecd107
2015-12-19 15:00:09 -07:00
Ruben Brunk
55bed95778 Merge "Add a framework service tracking VR mode state." 2015-12-18 19:50:24 +00:00
Todd Kennedy
36a832dd12 Merge "revert quick install" 2015-12-17 18:49:08 +00:00