summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author TreeHugger Robot <treehugger-gerrit@google.com> 2022-10-03 22:42:28 +0000
committer Android (Google) Code Review <android-gerrit@google.com> 2022-10-03 22:42:28 +0000
commit202617a47c4444fe1ff001881522ae3119ec033c (patch)
tree2a7fbf583022a76207dbadbc4135fb7a393ec685
parent94d00b167c1ac4245aa38b6e4220946503aab5a6 (diff)
parent89d3afd5764f0e2bb792539601275ece6c8dcb5d (diff)
Merge "Clarify network validation code."
-rw-r--r--apex/jobscheduler/service/java/com/android/server/job/JobStore.java3
1 files changed, 3 insertions, 0 deletions
diff --git a/apex/jobscheduler/service/java/com/android/server/job/JobStore.java b/apex/jobscheduler/service/java/com/android/server/job/JobStore.java
index 78ab06c9e332..ff4d26dc807d 100644
--- a/apex/jobscheduler/service/java/com/android/server/job/JobStore.java
+++ b/apex/jobscheduler/service/java/com/android/server/job/JobStore.java
@@ -1095,6 +1095,8 @@ public final class JobStore {
}
if ((netCapabilitiesIntArray != null) && (netTransportTypesIntArray != null)) {
+ // S+ format. No capability or transport validation since the values should be in
+ // line with what's defined in the Connectivity mainline module.
final NetworkRequest.Builder builder = new NetworkRequest.Builder()
.clearCapabilities();
@@ -1111,6 +1113,7 @@ public final class JobStore {
}
jobBuilder.setRequiredNetwork(builder.build());
} else if (netCapabilitiesLong != null && netTransportTypesLong != null) {
+ // Format used on R- builds. Drop any unexpected capabilities and transports.
final NetworkRequest.Builder builder = new NetworkRequest.Builder()
.clearCapabilities();
final int maxNetCapabilityInR = NET_CAPABILITY_TEMPORARILY_NOT_METERED;