Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

am 1af5fe23: Merge "Require that verified intent filters only have http/https <data> decls" into mnc-dev

Browse Source 
* commit '1af5fe230835881fee04ce706ddc71181e014db6':
  Require that verified intent filters only have http/https <data> decls
This commit is contained in:
Christopher Tate
2015-06-24 20:41:07 +00:00
committed by Android Git Automerger
parent 61d728eca9 1af5fe2308
commit d1741ef961
2 changed files with 27 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

35
core/java/android/content/IntentFilter.java
View File

@@ -16,7 +16,6 @@
package android.content;
import android.content.pm.PackageParser;
import android.net.Uri;
import android.os.Parcel;
import android.os.Parcelable;
@@ -534,13 +533,15 @@ public class IntentFilter implements Parcelable {
*/
public final boolean handleAllWebDataURI() {
return hasCategory(Intent.CATEGORY_APP_BROWSER) ||
(hasWebDataURI() && countDataAuthorities() == 0);
(hasOnlyWebDataURI() && countDataAuthorities() == 0);
}
/**
* Return if this filter has any HTTP or HTTPS data URI or not.
* Return if this filter handles only HTTP or HTTPS data URIs.
*
* @return True if the filter has any HTTP or HTTPS data URI. False otherwise.
* @return True if the filter handles ACTION_VIEW/CATEGORY_BROWSABLE,
* has at least one HTTP or HTTPS data URI pattern defined, and does not
* define any non-http/https data URI patterns.
*
* This will check if if the Intent action is {@link android.content.Intent#ACTION_VIEW} and
* the Intent category is {@link android.content.Intent#CATEGORY_BROWSABLE} and the Intent
@@ -548,10 +549,26 @@ public class IntentFilter implements Parcelable {
*
* @hide
*/
public final boolean hasWebDataURI() {
return hasAction(Intent.ACTION_VIEW) &&
hasCategory(Intent.CATEGORY_BROWSABLE) &&
(hasDataScheme(SCHEME_HTTP) || hasDataScheme(SCHEME_HTTPS));
public final boolean hasOnlyWebDataURI() {
// Require ACTION_VIEW, CATEGORY_BROWSEABLE, and at least one scheme
if (!hasAction(Intent.ACTION_VIEW)
|| !hasCategory(Intent.CATEGORY_BROWSABLE)
|| mDataSchemes == null
|| mDataSchemes.size() == 0) {
return false;
}
// Now allow only the schemes "http" and "https"
final int N = mDataSchemes.size();
for (int i = 0; i < N; i++) {
final String scheme = mDataSchemes.get(i);
if (!SCHEME_HTTP.equals(scheme) && !SCHEME_HTTPS.equals(scheme)) {
return false;
}
}
// Everything passed, so it's an only-web-URIs filter
return true;
}
/**
@@ -568,7 +585,7 @@ public class IntentFilter implements Parcelable {
* @hide
*/
public final boolean needsVerification() {
return hasWebDataURI() && getAutoVerify();
return getAutoVerify() && hasOnlyWebDataURI();
}
/**

3
services/core/java/com/android/server/pm/PackageManagerService.java
View File

@@ -12020,8 +12020,7 @@ public class PackageManagerService extends IPackageManager.Stub {
final int verificationId = mIntentFilterVerificationToken++;
for (PackageParser.Activity a : pkg.activities) {
for (ActivityIntentInfo filter : a.intents) {
boolean needsFilterVerification = filter.hasWebDataURI();
if (needsFilterVerification && needsNetworkVerificationLPr(filter)) {
if (filter.hasOnlyWebDataURI() && needsNetworkVerificationLPr(filter)) {
if (DEBUG_DOMAIN_VERIFICATION) Slog.d(TAG,
"Verification needed for IntentFilter:" + filter.toString());
mIntentFilterVerifier.addOneIntentFilterVerification(