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

Merge "Read-only properties can have values of arbitrary length"

Browse Source
This commit is contained in:
Jerome Gaillard
2022-03-23 10:21:15 +00:00
committed by Gerrit Code Review
parent b344dfaee9 069f22cf0c
commit 329f25d91a
1 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
core/java/android/os/SystemProperties.java
View File

@@ -218,14 +218,15 @@ public class SystemProperties {
/** /**
* Set the value for the given {@code key} to {@code val}. * Set the value for the given {@code key} to {@code val}.
* *
* @throws IllegalArgumentException if the {@code val} exceeds 91 characters * @throws IllegalArgumentException for non read-only properties if the {@code val} exceeds
* 91 characters
* @throws RuntimeException if the property cannot be set, for example, if it was blocked by * @throws RuntimeException if the property cannot be set, for example, if it was blocked by
* SELinux. libc will log the underlying reason. * SELinux. libc will log the underlying reason.
* @hide * @hide
*/ */
@UnsupportedAppUsage @UnsupportedAppUsage
public static void set(@NonNull String key, @Nullable String val) { public static void set(@NonNull String key, @Nullable String val) {
if (val != null && !val.startsWith("ro.") && val.length() > PROP_VALUE_MAX) { if (val != null && !key.startsWith("ro.") && val.length() > PROP_VALUE_MAX) {
throw new IllegalArgumentException("value of system property '" + key throw new IllegalArgumentException("value of system property '" + key
+ "' is longer than " + PROP_VALUE_MAX + " characters: " + val); + "' is longer than " + PROP_VALUE_MAX + " characters: " + val);
} }