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

Commit 20226581 authored by Jackal Guo's avatar Jackal Guo Committed by Automerger Merge Worker
Browse files

[RESTRICT AUTOMERGE] Limit length of the name in <uses-permission> am:... 

[RESTRICT AUTOMERGE] Limit length of the name in <uses-permission> am: de8ef32d am: 3ce1ae63 am: f9030b26

Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/20953243



Change-Id: I55fab1dff887f5df49a111bdffaaf04d5daf4082
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 77ff257a f9030b26

services/core/java/com/android/server/pm/pkg/parsing/ParsingPackageUtils.java

+8 −0
Original line number Diff line number Diff line
@@ -247,6 +247,9 @@ public class ParsingPackageUtils { 
    private static final String MAX_NUM_COMPONENTS_ERR_MSG =

            "Total number of components has exceeded the maximum number: " + MAX_NUM_COMPONENTS;



    /** The maximum permission name length. */

    private static final int MAX_PERMISSION_NAME_LENGTH = 512;



    @IntDef(flag = true, prefix = { "PARSE_" }, value = {

            PARSE_CHATTY,

            PARSE_COLLECT_CERTIFICATES,
@@ -1275,6 +1278,11 @@ public class ParsingPackageUtils { 
            // that may change.

            String name = sa.getNonResourceString(

                    R.styleable.AndroidManifestUsesPermission_name);

            if (TextUtils.length(name) > MAX_PERMISSION_NAME_LENGTH) {

                return input.error(INSTALL_PARSE_FAILED_MANIFEST_MALFORMED,

                        "The name in the <uses-permission> is greater than "

                                + MAX_PERMISSION_NAME_LENGTH);

            }



            int maxSdkVersion = 0;

            TypedValue val = sa.peekValue(