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

Don't log meaningless caps change am: f59aeb7cd1

Browse Source 
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/23422194

Change-Id: Ia348ea0e64c75245ba76d8f200c2e89136c1b7b8
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
Chalard Jean
2023-05-25 06:01:33 +00:00
committed by Automerger Merge Worker
parent c2c44857c5 f59aeb7cd1
commit 62bca082a8
1 changed files with 19 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

17
services/core/java/com/android/server/connectivity/Vpn.java
View File

@@ -3828,10 +3828,27 @@ public class Vpn {
}, retryDelayMs, TimeUnit.MILLISECONDS); }, retryDelayMs, TimeUnit.MILLISECONDS);
} }
private boolean significantCapsChange(@Nullable final NetworkCapabilities left,
@Nullable final NetworkCapabilities right) {
if (left == right) return false;
return null == left
|| null == right
|| !Arrays.equals(left.getTransportTypes(), right.getTransportTypes())
|| !Arrays.equals(left.getCapabilities(), right.getCapabilities())
|| !Arrays.equals(left.getEnterpriseIds(), right.getEnterpriseIds())
|| !Objects.equals(left.getTransportInfo(), right.getTransportInfo())
|| !Objects.equals(left.getAllowedUids(), right.getAllowedUids())
|| !Objects.equals(left.getUnderlyingNetworks(), right.getUnderlyingNetworks())
|| !Objects.equals(left.getNetworkSpecifier(), right.getNetworkSpecifier());
}
/** Called when the NetworkCapabilities of underlying network is changed */ /** Called when the NetworkCapabilities of underlying network is changed */
public void onDefaultNetworkCapabilitiesChanged(@NonNull NetworkCapabilities nc) { public void onDefaultNetworkCapabilitiesChanged(@NonNull NetworkCapabilities nc) {
if (significantCapsChange(mUnderlyingNetworkCapabilities, nc)) {
// TODO : make this log terser
mEventChanges.log("[UnderlyingNW] Cap changed from " mEventChanges.log("[UnderlyingNW] Cap changed from "
+ mUnderlyingNetworkCapabilities + " to " + nc); + mUnderlyingNetworkCapabilities + " to " + nc);
}
final NetworkCapabilities oldNc = mUnderlyingNetworkCapabilities; final NetworkCapabilities oldNc = mUnderlyingNetworkCapabilities;
mUnderlyingNetworkCapabilities = nc; mUnderlyingNetworkCapabilities = nc;
if (oldNc == null || !nc.getSubscriptionIds().equals(oldNc.getSubscriptionIds())) { if (oldNc == null || !nc.getSubscriptionIds().equals(oldNc.getSubscriptionIds())) {