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

Merge "Be more defensive around invalid tzids"

Browse Source
This commit is contained in:
Neil Fuller
2020-05-26 10:20:09 +00:00
committed by Gerrit Code Review
parent 808cb66c50 26fe60a188
commit f60c302dad
3 changed files with 19 additions and 22 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
core/java/android/app/AlarmManager.java
View File

@@ -37,7 +37,6 @@ import android.util.proto.ProtoOutputStream;
import libcore.timezone.ZoneInfoDb; import libcore.timezone.ZoneInfoDb;
import java.io.IOException;
import java.lang.annotation.Retention; import java.lang.annotation.Retention;
import java.lang.annotation.RetentionPolicy; import java.lang.annotation.RetentionPolicy;
@@ -998,12 +997,7 @@ public class AlarmManager {
// Reject this timezone if it isn't an Olson zone we recognize. // Reject this timezone if it isn't an Olson zone we recognize.
if (mTargetSdkVersion >= Build.VERSION_CODES.M) { if (mTargetSdkVersion >= Build.VERSION_CODES.M) {
boolean hasTimeZone = false; boolean hasTimeZone = ZoneInfoDb.getInstance().hasTimeZone(timeZone);
try {
hasTimeZone = ZoneInfoDb.getInstance().hasTimeZone(timeZone);
} catch (IOException ignored) {
}
if (!hasTimeZone) { if (!hasTimeZone) {
throw new IllegalArgumentException("Timezone: " + timeZone + " is not an Olson ID"); throw new IllegalArgumentException("Timezone: " + timeZone + " is not an Olson ID");
} }

6
core/java/android/text/format/Time.java
View File

@@ -21,7 +21,6 @@ import android.util.TimeFormatException;
import libcore.timezone.ZoneInfoDb; import libcore.timezone.ZoneInfoDb;
import libcore.util.ZoneInfo; import libcore.util.ZoneInfo;
import java.io.IOException;
import java.util.Locale; import java.util.Locale;
import java.util.TimeZone; import java.util.TimeZone;
@@ -1106,7 +1105,6 @@ public class Time {
} }
private static ZoneInfo lookupZoneInfo(String timezoneId) { private static ZoneInfo lookupZoneInfo(String timezoneId) {
try {
ZoneInfo zoneInfo = ZoneInfoDb.getInstance().makeTimeZone(timezoneId); ZoneInfo zoneInfo = ZoneInfoDb.getInstance().makeTimeZone(timezoneId);
if (zoneInfo == null) { if (zoneInfo == null) {
zoneInfo = ZoneInfoDb.getInstance().makeTimeZone("GMT"); zoneInfo = ZoneInfoDb.getInstance().makeTimeZone("GMT");
@@ -1115,10 +1113,6 @@ public class Time {
throw new AssertionError("GMT not found: \"" + timezoneId + "\""); throw new AssertionError("GMT not found: \"" + timezoneId + "\"");
} }
return zoneInfo; return zoneInfo;
} catch (IOException e) {
// This should not ever be thrown.
throw new AssertionError("Error loading timezone: \"" + timezoneId + "\"", e);
}
} }
public void switchTimeZone(String timezone) { public void switchTimeZone(String timezone) {

13
services/java/com/android/server/SystemServer.java
View File

@@ -165,6 +165,8 @@ import com.android.server.wm.ActivityTaskManagerService;
import com.android.server.wm.WindowManagerGlobalLock; import com.android.server.wm.WindowManagerGlobalLock;
import com.android.server.wm.WindowManagerService; import com.android.server.wm.WindowManagerService;
import libcore.timezone.ZoneInfoDb;
import dalvik.system.VMRuntime; import dalvik.system.VMRuntime;
import java.io.File; import java.io.File;
@@ -395,8 +397,9 @@ public final class SystemServer {
// Default the timezone property to GMT if not set. // Default the timezone property to GMT if not set.
// //
String timezoneProperty = SystemProperties.get("persist.sys.timezone"); String timezoneProperty = SystemProperties.get("persist.sys.timezone");
if (timezoneProperty == null || timezoneProperty.isEmpty()) { if (!isValidTimeZoneId(timezoneProperty)) {
Slog.w(TAG, "Timezone not set; setting to GMT."); Slog.w(TAG, "persist.sys.timezone is not valid (" + timezoneProperty
+ "); setting to GMT.");
SystemProperties.set("persist.sys.timezone", "GMT"); SystemProperties.set("persist.sys.timezone", "GMT");
} }
@@ -564,6 +567,12 @@ public final class SystemServer {
throw new RuntimeException("Main thread loop unexpectedly exited"); throw new RuntimeException("Main thread loop unexpectedly exited");
} }
private static boolean isValidTimeZoneId(String timezoneProperty) {
return timezoneProperty != null
&& !timezoneProperty.isEmpty()
&& ZoneInfoDb.getInstance().hasTimeZone(timezoneProperty);
}
private boolean isFirstBootOrUpgrade() { private boolean isFirstBootOrUpgrade() {
return mPackageManagerService.isFirstBoot() || mPackageManagerService.isDeviceUpgrading(); return mPackageManagerService.isFirstBoot() || mPackageManagerService.isDeviceUpgrading();
} }