Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 0c581e16 authored by Akhil Gangu's avatar Akhil Gangu Committed by Android (Google) Code Review
Browse files

Revert "Add parsing support for new purpose declaration manifest elements." 

This reverts commit 089a8a61.

Reason for revert: b/420985780. To confirm if this CL is the cause.

Change-Id: I91aa330494fc17b6aa8241cf42d7f2f028776edf
parent 089a8a61

core/java/android/content/pm/PermissionInfo.java

+0 −21
Original line number Diff line number Diff line
@@ -508,21 +508,6 @@ public class PermissionInfo extends PackageItemInfo implements Parcelable { 
    @SystemApi

    public @NonNull Set<String> knownCerts = Collections.emptySet();



    /**

     * A boolean to signify if purposes are required to be declared in order to use the permission.

     *

     * @hide

     */

    public boolean requiresPurpose;



    /**

     * A {@link Set} of valid purposes defined for using this permission. Apps that request this

     * permission will be required to declare at least one permission from this set.

     *

     * @hide

     */

    public @NonNull Set<String> validPurposes = Collections.emptySet();



    /** @hide */

    public static int fixProtectionLevel(int level) {

        if (level == PROTECTION_SIGNATURE_OR_SYSTEM) {
@@ -694,8 +679,6 @@ public class PermissionInfo extends PackageItemInfo implements Parcelable { 
        nonLocalizedDescription = orig.nonLocalizedDescription;

        // Note that knownCerts wasn't properly copied before Android U.

        knownCerts = orig.knownCerts;

        requiresPurpose = orig.requiresPurpose;

        validPurposes = orig.validPurposes;

    }



    /**
@@ -761,8 +744,6 @@ public class PermissionInfo extends PackageItemInfo implements Parcelable { 
        dest.writeInt(requestRes);

        TextUtils.writeToParcel(nonLocalizedDescription, dest, parcelableFlags);

        sForStringSet.parcel(knownCerts, dest, parcelableFlags);

        dest.writeBoolean(requiresPurpose);

        sForStringSet.parcel(validPurposes, dest, parcelableFlags);

    }



    /** @hide */
@@ -824,7 +805,5 @@ public class PermissionInfo extends PackageItemInfo implements Parcelable { 
        requestRes = source.readInt();

        nonLocalizedDescription = TextUtils.CHAR_SEQUENCE_CREATOR.createFromParcel(source);

        knownCerts = sForStringSet.unparcel(source);

        requiresPurpose = source.readBoolean();

        validPurposes = sForStringSet.unparcel(source);

    }

}

core/java/com/android/internal/pm/parsing/PackageInfoCommonUtils.java

+0 −2
Original line number Diff line number Diff line
@@ -509,8 +509,6 @@ public class PackageInfoCommonUtils { 
        pi.descriptionRes = p.getDescriptionRes();

        pi.flags = p.getFlags();

        pi.knownCerts = p.getKnownCerts();

        pi.requiresPurpose = p.isPurposeRequired();

        pi.validPurposes = p.getValidPurposes();



        if ((flags & PackageManager.GET_META_DATA) == 0) {

            pi.metaData = null;

core/java/com/android/internal/pm/parsing/pkg/PackageImpl.java

+2 −43
Original line number Diff line number Diff line
@@ -283,8 +283,6 @@ public class PackageImpl implements ParsedPackage, AndroidPackageInternal, 
    @NonNull

    private List<ParsedUsesPermission> usesPermissions = emptyList();

    @NonNull

    private Map<String, ParsedUsesPermission> usesPermissionMapping = emptyMap();

    @NonNull

    @DataClass.ParcelWith(Parcelling.BuiltIn.ForInternedStringSet.class)

    private Set<String> implicitPermissions = emptySet();

    @NonNull
@@ -547,9 +545,7 @@ public class PackageImpl implements ParsedPackage, AndroidPackageInternal, 


    @Override

    public PackageImpl addImplicitPermission(String permission) {

        addUsesPermission(

                new ParsedUsesPermissionImpl(

                        permission, /* usesPermissionFlags= */ 0, /* purposes= */ emptySet()));

        addUsesPermission(new ParsedUsesPermissionImpl(permission, 0 /*usesPermissionFlags*/));

        this.implicitPermissions = CollectionUtils.add(this.implicitPermissions,

                TextUtils.safeIntern(permission));

        return this;
@@ -726,7 +722,6 @@ public class PackageImpl implements ParsedPackage, AndroidPackageInternal, 
        return this;

    }



    // TODO(419394776) - Use single source of truth for storing uses permission metadata.

    @Override

    public PackageImpl addUsesPermission(ParsedUsesPermission permission) {

        this.usesPermissions = CollectionUtils.add(this.usesPermissions, permission);
@@ -736,11 +731,6 @@ public class PackageImpl implements ParsedPackage, AndroidPackageInternal, 
        this.requestedPermissions = CollectionUtils.add(this.requestedPermissions,

                permission.getName());



        // During manifest parsing, we ignore duplicate permission requests. Therefore, it's safe

        // to directly add to the mapping.

        this.usesPermissionMapping =

                CollectionUtils.add(this.usesPermissionMapping, permission.getName(), permission);



        return this;

    }
@@ -1503,15 +1493,7 @@ public class PackageImpl implements ParsedPackage, AndroidPackageInternal, 


    @NonNull

    @Override

    public Map<String, ParsedUsesPermission> getUsesPermissionMapping() {

        return usesPermissionMapping;

    }



    @NonNull

    @Override

    public List<String> getUsesSdkLibraries() {

        return usesSdkLibraries;

    }

    public List<String> getUsesSdkLibraries() { return usesSdkLibraries; }



    @NonNull

    @Override
@@ -2875,7 +2857,6 @@ public class PackageImpl implements ParsedPackage, AndroidPackageInternal, 
        reqFeatures = Collections.unmodifiableList(reqFeatures);

        featureGroups = Collections.unmodifiableList(featureGroups);

        usesPermissions = Collections.unmodifiableList(usesPermissions);

        usesPermissionMapping = Collections.unmodifiableMap(usesPermissionMapping);

        usesSdkLibraries = Collections.unmodifiableList(usesSdkLibraries);

        implicitPermissions = Collections.unmodifiableSet(implicitPermissions);

        upgradeKeySets = Collections.unmodifiableSet(upgradeKeySets);
@@ -3256,7 +3237,6 @@ public class PackageImpl implements ParsedPackage, AndroidPackageInternal, 
        sForInternedStringList.parcel(this.adoptPermissions, dest, flags);

        sForInternedStringSet.parcel(this.requestedPermissions, dest, flags);

        ParsingUtils.writeParcelableList(dest, this.usesPermissions);

        writeUsesPermissionMapping(dest);

        sForInternedStringSet.parcel(this.implicitPermissions, dest, flags);

        sForStringSet.parcel(this.upgradeKeySets, dest, flags);

        ParsingPackageUtils.writeKeySetMapping(dest, this.keySetMapping);
@@ -3345,15 +3325,6 @@ public class PackageImpl implements ParsedPackage, AndroidPackageInternal, 
        dest.writeInt(this.mPageSizeAppCompatFlags);

    }



    private void writeUsesPermissionMapping(@NonNull Parcel dest) {

        // No need to deal with null case because the mapping is already instantiated to be empty.

        final Bundle bundle = new Bundle();

        for (Map.Entry<String, ParsedUsesPermission> entry : usesPermissionMapping.entrySet()) {

            bundle.putParcelable(entry.getKey(), (ParsedUsesPermissionImpl) entry.getValue());

        }

        dest.writeBundle(bundle);

    }



    private void writeFeatureFlagState(@NonNull Parcel dest) {

        // Use a string array to encode flag state. One string per flag in the form `<flag>=<value>`

        // where value is 0 (disabled), 1 (enabled) or ? (unknown flag or value).
@@ -3455,7 +3426,6 @@ public class PackageImpl implements ParsedPackage, AndroidPackageInternal, 
        this.requestedPermissions = sForInternedStringSet.unparcel(in);

        this.usesPermissions = ParsingUtils.createTypedInterfaceList(in,

                ParsedUsesPermissionImpl.CREATOR);

        readUsesPermissionMapping(in);

        this.implicitPermissions = sForInternedStringSet.unparcel(in);

        this.upgradeKeySets = sForStringSet.unparcel(in);

        this.keySetMapping = ParsingPackageUtils.readKeySetMapping(in);
@@ -3556,17 +3526,6 @@ public class PackageImpl implements ParsedPackage, AndroidPackageInternal, 
        // to mutate this instance before it's finalized.

    }



    private void readUsesPermissionMapping(@NonNull Parcel in) {

        final Bundle bundle = in.readBundle(ParsedUsesPermissionImpl.class.getClassLoader());

        final Map<String, ParsedUsesPermission> mapping = new ArrayMap<>();

        if (bundle != null) {

            for (String key : bundle.keySet()) {

                mapping.put(key, bundle.getParcelable(key, ParsedUsesPermissionImpl.class));

            }

        }

        usesPermissionMapping = mapping;

    }



    private void readFeatureFlagState(@NonNull Parcel in) {

        // See comment in writeFeatureFlagState() for encoding of flag state.

        String[] featureFlagStateAsArray = in.createStringArray();

core/java/com/android/internal/pm/pkg/component/ComponentParseUtils.java

+1 −37
Original line number Diff line number Diff line
@@ -27,11 +27,8 @@ import android.content.pm.parsing.result.ParseResult; 
import android.content.res.Resources;

import android.content.res.TypedArray;

import android.content.res.XmlResourceParser;

import android.permission.flags.Flags;

import android.text.TextUtils;

import android.util.ArraySet;



import com.android.internal.R;

import com.android.internal.pm.pkg.parsing.ParsingPackage;

import com.android.internal.pm.pkg.parsing.ParsingPackageUtils;

import com.android.internal.pm.pkg.parsing.ParsingUtils;
@@ -41,7 +38,6 @@ import org.xmlpull.v1.XmlPullParser; 
import org.xmlpull.v1.XmlPullParserException;



import java.io.IOException;

import java.util.Set;



/**

 * @hide
@@ -59,14 +55,6 @@ public class ComponentParseUtils { 
    static <Component extends ParsedComponentImpl> ParseResult<Component> parseAllMetaData(

            ParsingPackage pkg, Resources res, XmlResourceParser parser, String tag,

            Component component, ParseInput input) throws XmlPullParserException, IOException {

        // Beginning in Android 17, permissions may specify valid usage purposes. Currently, valid

        // purposes are only processed for enforcement if the permission is defined within the

        // Android platform manifest. This limitation might be lifted in future versions.

        final boolean shouldParseValidPurposes =

                Flags.purposeDeclarationEnabled()

                        && component instanceof ParsedPermissionImpl

                        && "android".equals(pkg.getPackageName());

        final Set<String> validPurposes = new ArraySet<>();

        final int depth = parser.getDepth();

        int type;

        while ((type = parser.next()) != XmlPullParser.END_DOCUMENT
@@ -79,16 +67,9 @@ public class ComponentParseUtils { 
                continue;

            }



            final ParseResult<?> result;

            final ParseResult result;

            if ("meta-data".equals(parser.getName())) {

                result = ParsedComponentUtils.addMetaData(component, pkg, res, parser, input);

            } else if (shouldParseValidPurposes && "valid-purpose".equals(parser.getName())) {

                final ParseResult<String> validPurposeResult =

                        parseValidPurpose(res, parser, input);

                result = validPurposeResult;

                if (validPurposeResult.isSuccess() && validPurposeResult.getResult() != null) {

                    validPurposes.add(validPurposeResult.getResult());

                }

            } else {

                result = ParsingUtils.unknownTag(tag, pkg, parser, input);

            }
@@ -98,26 +79,9 @@ public class ComponentParseUtils { 
            }

        }



        if (!validPurposes.isEmpty()) {

            // There can only be valid purposes if component is an instance of ParsedPermissionImpl.

            final ParsedPermissionImpl permission = (ParsedPermissionImpl) component;

            permission.setValidPurposes(validPurposes);

        }



        return input.success(component);

    }



    private static ParseResult<String> parseValidPurpose(

            Resources res, XmlResourceParser parser, ParseInput input) {

        TypedArray sa = res.obtainAttributes(parser, R.styleable.AndroidManifestValidPurpose);

        try {

            final String validPurpose = sa.getString(R.styleable.AndroidManifestValidPurpose_name);

            return input.success(TextUtils.isEmpty(validPurpose) ? null : validPurpose);

        } finally {

            sa.recycle();

        }

    }



    @NonNull

    public static ParseResult<String> buildProcessName(@NonNull String pkg, String defProc,

            CharSequence procSeq, int flags, String[] separateProcesses, ParseInput input) {

core/java/com/android/internal/pm/pkg/component/ParsedPermission.java

+0 −5
Original line number Diff line number Diff line
@@ -41,10 +41,5 @@ public interface ParsedPermission extends ParsedComponent { 


    int getRequestRes();



    boolean isPurposeRequired();



    @NonNull

    Set<String> getValidPurposes();



    boolean isTree();

}