LineageOS/android_packages_modules_Connectivity
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge "Never satisfy requests with networks that aren't connected."

Browse Source
This commit is contained in:
Treehugger Robot
2022-08-29 08:30:31 +00:00
committed by Gerrit Code Review
parent d407328dd4 275ee60963
commit 05021aa886
2 changed files with 6 additions and 13 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
service/src/com/android/server/ConnectivityService.java
View File

@@ -7470,9 +7470,7 @@ public class ConnectivityService extends IConnectivityManager.Stub
notifyIfacesChangedForNetworkStats(); notifyIfacesChangedForNetworkStats();
networkAgent.networkMonitor().notifyLinkPropertiesChanged( networkAgent.networkMonitor().notifyLinkPropertiesChanged(
new LinkProperties(newLp, true /* parcelSensitiveFields */)); new LinkProperties(newLp, true /* parcelSensitiveFields */));
if (networkAgent.everConnected) { notifyNetworkCallbacks(networkAgent, ConnectivityManager.CALLBACK_IP_CHANGED);
notifyNetworkCallbacks(networkAgent, ConnectivityManager.CALLBACK_IP_CHANGED);
}
} }
mKeepaliveTracker.handleCheckKeepalivesStillValid(networkAgent); mKeepaliveTracker.handleCheckKeepalivesStillValid(networkAgent);
@@ -8732,9 +8730,6 @@ public class ConnectivityService extends IConnectivityManager.Stub
// Gather the list of all relevant agents. // Gather the list of all relevant agents.
final ArrayList<NetworkAgentInfo> nais = new ArrayList<>(); final ArrayList<NetworkAgentInfo> nais = new ArrayList<>();
for (final NetworkAgentInfo nai : mNetworkAgentInfos) { for (final NetworkAgentInfo nai : mNetworkAgentInfos) {
if (!nai.everConnected) {
continue;
}
nais.add(nai); nais.add(nai);
} }
@@ -8858,7 +8853,6 @@ public class ConnectivityService extends IConnectivityManager.Stub
} }
for (final NetworkAgentInfo nai : nais) { for (final NetworkAgentInfo nai : nais) {
if (!nai.everConnected) continue;
final boolean oldBackground = oldBgNetworks.contains(nai); final boolean oldBackground = oldBgNetworks.contains(nai);
// Process listen requests and update capabilities if the background state has // Process listen requests and update capabilities if the background state has
// changed for this network. For consistency with previous behavior, send onLost // changed for this network. For consistency with previous behavior, send onLost
@@ -9451,7 +9445,7 @@ public class ConnectivityService extends IConnectivityManager.Stub
} }
} }
for (NetworkAgentInfo nai : mNetworkAgentInfos) { for (NetworkAgentInfo nai : mNetworkAgentInfos) {
if (nai.everConnected && (activeNetIds.contains(nai.network().netId) || nai.isVPN())) { if (activeNetIds.contains(nai.network().netId) || nai.isVPN()) {
defaultNetworks.add(nai.network); defaultNetworks.add(nai.network);
} }
} }

9
service/src/com/android/server/connectivity/NetworkAgentInfo.java
View File

@@ -983,14 +983,13 @@ public class NetworkAgentInfo implements NetworkRanker.Scoreable {
// Does this network satisfy request? // Does this network satisfy request?
public boolean satisfies(NetworkRequest request) { public boolean satisfies(NetworkRequest request) {
return created && return everConnected
request.networkCapabilities.satisfiedByNetworkCapabilities(networkCapabilities); && request.networkCapabilities.satisfiedByNetworkCapabilities(networkCapabilities);
} }
public boolean satisfiesImmutableCapabilitiesOf(NetworkRequest request) { public boolean satisfiesImmutableCapabilitiesOf(NetworkRequest request) {
return created && return everConnected && request.networkCapabilities.satisfiedByImmutableNetworkCapabilities(
request.networkCapabilities.satisfiedByImmutableNetworkCapabilities( networkCapabilities);
networkCapabilities);
} }
/** Whether this network is a VPN. */ /** Whether this network is a VPN. */