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

Use public API Network#getNetId() to get netid

Browse Source 
Network class will be a part of mainline module, external callers
cannot call its hidden constant. Use public API - getNetId() to
get netid instead.

Bug: 182963397
Test: m
Change-Id: I6d5dd0a8c3879df7bdc1d58e36022045c303bb40
This commit is contained in:
lucaslin
2021-03-19 16:06:33 +08:00
committed by Lucas Lin
parent cda16ed7ae
commit 83ca710aff
2 changed files with 8 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
services/core/java/com/android/server/IpSecService.java
View File

@@ -1651,7 +1651,7 @@ public class IpSecService extends IIpSecService.Stub {
c.getMode(),
c.getSourceAddress(),
c.getDestinationAddress(),
(c.getNetwork() != null) ? c.getNetwork().netId : 0,
(c.getNetwork() != null) ? c.getNetwork().getNetId() : 0,
spiRecord.getSpi(),
c.getMarkValue(),
c.getMarkMask(),

13
services/core/java/com/android/server/net/NetworkPolicyManagerService.java
View File

@@ -1216,10 +1216,11 @@ public class NetworkPolicyManagerService extends INetworkPolicyManager.Stub {
private static boolean updateCapabilityChange(SparseBooleanArray lastValues, boolean newValue,
Network network) {
final boolean lastValue = lastValues.get(network.netId, false);
final boolean changed = (lastValue != newValue) || lastValues.indexOfKey(network.netId) < 0;
final boolean lastValue = lastValues.get(network.getNetId(), false);
final boolean changed = (lastValue != newValue)
|| lastValues.indexOfKey(network.getNetId()) < 0;
if (changed) {
lastValues.put(network.netId, newValue);
lastValues.put(network.getNetId(), newValue);
}
return changed;
}
@@ -1242,7 +1243,7 @@ public class NetworkPolicyManagerService extends INetworkPolicyManager.Stub {
mNetworkRoaming, newRoaming, network);
if (meteredChanged || roamingChanged) {
mLogger.meterednessChanged(network.netId, newMetered);
mLogger.meterednessChanged(network.getNetId(), newMetered);
updateNetworkRulesNL();
}
}
@@ -2008,7 +2009,7 @@ public class NetworkPolicyManagerService extends INetworkPolicyManager.Stub {
mNetIdToSubId.clear();
final ArrayMap<NetworkStateSnapshot, NetworkIdentity> identified = new ArrayMap<>();
for (final NetworkStateSnapshot snapshot : snapshots) {
mNetIdToSubId.put(snapshot.network.netId, parseSubId(snapshot));
mNetIdToSubId.put(snapshot.network.getNetId(), parseSubId(snapshot));
// Policies matched by NPMS only match by subscriber ID or by ssid. Thus subtype
// in the object created here is never used and its value doesn't matter, so use
@@ -5777,7 +5778,7 @@ public class NetworkPolicyManagerService extends INetworkPolicyManager.Stub {
@GuardedBy("mNetworkPoliciesSecondLock")
private int getSubIdLocked(Network network) {
return mNetIdToSubId.get(network.netId, INVALID_SUBSCRIPTION_ID);
return mNetIdToSubId.get(network.getNetId(), INVALID_SUBSCRIPTION_ID);
}
@GuardedBy("mNetworkPoliciesSecondLock")