Skip to content

Commit eec689f

Browse files
Ioana AlexandruAndroid Build Coastguard Worker
authored andcommitted
Lower per-app NotificationChannelGroup limit.
A higher limit can be abused to cause PDoS. For the rationale behind choosing 6000 as the max value, see http://b/261723753#comment20. Test: atest PreferencesHelperTest Bug: 261723753 Change-Id: I3f3a99765c161369e1b026686a0e5f0c83ed839e (cherry picked from commit 86441bd) Merged-In: I3f3a99765c161369e1b026686a0e5f0c83ed839e
1 parent 37e9ac2 commit eec689f

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

services/core/java/com/android/server/notification/PreferencesHelper.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -108,7 +108,7 @@ public class PreferencesHelper implements RankingConfig {
108108
@VisibleForTesting
109109
static final int NOTIFICATION_CHANNEL_COUNT_LIMIT = 5000;
110110
@VisibleForTesting
111-
static final int NOTIFICATION_CHANNEL_GROUP_COUNT_LIMIT = 50000;
111+
static final int NOTIFICATION_CHANNEL_GROUP_COUNT_LIMIT = 6000;
112112

113113
private static final int NOTIFICATION_PREFERENCES_PULL_LIMIT = 1000;
114114
private static final int NOTIFICATION_CHANNEL_PULL_LIMIT = 2000;

0 commit comments

Comments
 (0)