Merge "Fix permission issues for network update service" am: 1aed0289c9
Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/2253795 Change-Id: I13ee7ef214ee5606d4cd32aa14f952e5bcd7f31f Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
@@ -165,9 +165,14 @@ public class NetworkTimeUpdateService extends Binder {
|
||||
mContext.enforceCallingPermission(
|
||||
android.Manifest.permission.SET_TIME, "clear latest network time");
|
||||
|
||||
mTime.clearCachedTimeResult();
|
||||
final long token = Binder.clearCallingIdentity();
|
||||
try {
|
||||
mTime.clearCachedTimeResult();
|
||||
|
||||
mLocalLog.log("clearTimeForTests");
|
||||
mLocalLog.log("clearTimeForTests");
|
||||
} finally {
|
||||
Binder.restoreCallingIdentity(token);
|
||||
}
|
||||
}
|
||||
|
||||
/**
|
||||
@@ -181,15 +186,19 @@ public class NetworkTimeUpdateService extends Binder {
|
||||
mContext.enforceCallingPermission(
|
||||
android.Manifest.permission.SET_TIME, "force network time refresh");
|
||||
|
||||
boolean success = mTime.forceRefresh();
|
||||
mLocalLog.log("forceRefreshForTests: success=" + success);
|
||||
final long token = Binder.clearCallingIdentity();
|
||||
try {
|
||||
boolean success = mTime.forceRefresh();
|
||||
mLocalLog.log("forceRefreshForTests: success=" + success);
|
||||
|
||||
if (success) {
|
||||
makeNetworkTimeSuggestion(mTime.getCachedTimeResult(),
|
||||
"Origin: NetworkTimeUpdateService: forceRefreshForTests");
|
||||
if (success) {
|
||||
makeNetworkTimeSuggestion(mTime.getCachedTimeResult(),
|
||||
"Origin: NetworkTimeUpdateService: forceRefreshForTests");
|
||||
}
|
||||
return success;
|
||||
} finally {
|
||||
Binder.restoreCallingIdentity(token);
|
||||
}
|
||||
|
||||
return success;
|
||||
}
|
||||
|
||||
/**
|
||||
@@ -201,10 +210,15 @@ public class NetworkTimeUpdateService extends Binder {
|
||||
mContext.enforceCallingPermission(
|
||||
android.Manifest.permission.SET_TIME, "set NTP server config for tests");
|
||||
|
||||
mLocalLog.log("Setting server config for tests: hostname=" + hostname
|
||||
+ ", port=" + port
|
||||
+ ", timeout=" + timeout);
|
||||
mTime.setServerConfigForTests(hostname, port, timeout);
|
||||
final long token = Binder.clearCallingIdentity();
|
||||
try {
|
||||
mLocalLog.log("Setting server config for tests: hostname=" + hostname
|
||||
+ ", port=" + port
|
||||
+ ", timeout=" + timeout);
|
||||
mTime.setServerConfigForTests(hostname, port, timeout);
|
||||
} finally {
|
||||
Binder.restoreCallingIdentity(token);
|
||||
}
|
||||
}
|
||||
|
||||
private void onPollNetworkTime(int event) {
|
||||
|
||||
Reference in New Issue
Block a user