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

Commit a7378285 authored by Treehugger Robot's avatar Treehugger Robot Committed by Automerger Merge Worker
Browse files

Merge "Ensure valid parameters for CHOOSER_ACTION events." into udc-dev am: aefac4e0

parents d6f466e3 aefac4e0
Loading
Loading
Loading
Loading
+6 −1
Original line number Original line Diff line number Diff line
@@ -2540,10 +2540,15 @@ public class UsageStatsService extends SystemService implements


        @Override
        @Override
        public void reportChooserSelection(@NonNull String packageName, int userId,
        public void reportChooserSelection(@NonNull String packageName, int userId,
                String contentType, String[] annotations, String action) {
                @NonNull String contentType, String[] annotations, @NonNull String action) {
            if (packageName == null) {
            if (packageName == null) {
                throw new IllegalArgumentException("Package selection must not be null.");
                throw new IllegalArgumentException("Package selection must not be null.");
            }
            }
            // A valid contentType and action must be provided for chooser selection events.
            if (contentType == null || contentType.isBlank()
                    || action == null || action.isBlank()) {
                return;
            }
            // Verify if this package exists before reporting an event for it.
            // Verify if this package exists before reporting an event for it.
            if (mPackageManagerInternal.getPackageUid(packageName, 0, userId) < 0) {
            if (mPackageManagerInternal.getPackageUid(packageName, 0, userId) < 0) {
                Slog.w(TAG, "Event report user selecting an invalid package");
                Slog.w(TAG, "Event report user selecting an invalid package");