Merge changes Ib0690243,I0cabce70,I8f13a0d8,I81efeaa4,If2abfec1 into sc-dev (183576c8) · Commits · e / os / android_frameworks_base

services/core/java/com/android/server/VcnManagementService.java

+36 −0

Original line number	Diff line number	Diff line
		@@ -935,13 +935,31 @@ public class VcnManagementService extends IVcnManagementService.Stub {
		pw.println("VcnManagementService dump:");
		pw.increaseIndent();

		pw.println("mNetworkProvider:");
		pw.increaseIndent();
		mNetworkProvider.dump(pw);
		pw.decreaseIndent();
		pw.println();

		pw.println("mTrackingNetworkCallback:");
		pw.increaseIndent();
		mTrackingNetworkCallback.dump(pw);
		pw.decreaseIndent();
		pw.println();

		synchronized (mLock) {
		pw.println("mLastSnapshot:");
		pw.increaseIndent();
		mLastSnapshot.dump(pw);
		pw.decreaseIndent();
		pw.println();

		pw.println("mVcns:");
		pw.increaseIndent();
		for (Vcn vcn : mVcns.values()) {
		vcn.dump(pw);
		}
		pw.decreaseIndent();
		pw.println();
		}

		@@ -1003,6 +1021,24 @@ public class VcnManagementService extends IVcnManagementService.Stub {

		return false;
		}

		/** Dumps the state of this snapshot for logging and debugging purposes. */
		public void dump(IndentingPrintWriter pw) {
		pw.println("TrackingNetworkCallback:");
		pw.increaseIndent();

		pw.println("mCaps:");
		pw.increaseIndent();
		synchronized (mCaps) {
		for (Entry<Network, NetworkCapabilities> entry : mCaps.entrySet()) {
		pw.println(entry.getKey() + ": " + entry.getValue());
		}
		}
		pw.decreaseIndent();
		pw.println();

		pw.decreaseIndent();
		}
		}

		/** VcnCallbackImpl for Vcn signals sent up to VcnManagementService. */

services/core/java/com/android/server/vcn/TelephonySubscriptionTracker.java

+12 −0

Original line number	Diff line number	Diff line
		@@ -43,6 +43,7 @@ import android.util.Slog;

		import com.android.internal.annotations.VisibleForTesting;
		import com.android.internal.annotations.VisibleForTesting.Visibility;
		import com.android.internal.util.IndentingPrintWriter;

		import java.util.Collections;
		import java.util.HashMap;
		@@ -320,6 +321,17 @@ public class TelephonySubscriptionTracker extends BroadcastReceiver {
		&& mPrivilegedPackages.equals(other.mPrivilegedPackages);
		}

		/** Dumps the state of this snapshot for logging and debugging purposes. */
		public void dump(IndentingPrintWriter pw) {
		pw.println("TelephonySubscriptionSnapshot:");
		pw.increaseIndent();

		pw.println("mSubIdToGroupMap: " + mSubIdToGroupMap);
		pw.println("mPrivilegedPackages: " + mPrivilegedPackages);

		pw.decreaseIndent();
		}

		@Override
		public String toString() {
		return "TelephonySubscriptionSnapshot{ "

services/core/java/com/android/server/vcn/UnderlyingNetworkTracker.java

+13 −0

Original line number	Diff line number	Diff line
		@@ -31,6 +31,7 @@ import android.util.Slog;

		import com.android.internal.annotations.VisibleForTesting;
		import com.android.internal.annotations.VisibleForTesting.Visibility;
		import com.android.internal.util.IndentingPrintWriter;
		import com.android.server.vcn.TelephonySubscriptionTracker.TelephonySubscriptionSnapshot;

		import java.util.ArrayList;
		@@ -396,6 +397,18 @@ public class UnderlyingNetworkTracker {
		return Objects.hash(network, networkCapabilities, linkProperties, isBlocked);
		}

		/** Dumps the state of this record for logging and debugging purposes. */
		public void dump(IndentingPrintWriter pw) {
		pw.println("UnderlyingNetworkRecord:");
		pw.increaseIndent();

		pw.println("mNetwork: " + network);
		pw.println("mNetworkCapabilities: " + networkCapabilities);
		pw.println("mLinkProperties: " + linkProperties);

		pw.decreaseIndent();
		}

		/** Builder to incrementally construct an UnderlyingNetworkRecord. */
		private static class Builder {
		@NonNull private final Network mNetwork;

services/core/java/com/android/server/vcn/Vcn.java

+158 −29

Original line number	Diff line number	Diff line
		@@ -16,6 +16,8 @@

		package com.android.server.vcn;

		import static android.net.NetworkCapabilities.NET_CAPABILITY_DUN;
		import static android.net.NetworkCapabilities.NET_CAPABILITY_INTERNET;
		import static android.net.NetworkCapabilities.NET_CAPABILITY_NOT_VCN_MANAGED;
		import static android.net.NetworkCapabilities.TRANSPORT_CELLULAR;
		import static android.net.vcn.VcnManager.VCN_STATUS_CODE_ACTIVE;
		@@ -26,14 +28,21 @@ import static com.android.server.VcnManagementService.VDBG;

		import android.annotation.NonNull;
		import android.annotation.Nullable;
		import android.content.ContentResolver;
		import android.database.ContentObserver;
		import android.net.NetworkCapabilities;
		import android.net.NetworkRequest;
		import android.net.NetworkScore;
		import android.net.Uri;
		import android.net.vcn.VcnConfig;
		import android.net.vcn.VcnGatewayConnectionConfig;
		import android.net.vcn.VcnManager.VcnErrorCode;
		import android.os.Handler;
		import android.os.Message;
		import android.os.ParcelUuid;
		import android.provider.Settings;
		import android.telephony.TelephonyManager;
		import android.util.ArraySet;
		import android.util.Slog;

		import com.android.internal.annotations.VisibleForTesting;
		@@ -42,9 +51,11 @@ import com.android.internal.util.IndentingPrintWriter;
		import com.android.server.VcnManagementService.VcnCallback;
		import com.android.server.vcn.TelephonySubscriptionTracker.TelephonySubscriptionSnapshot;

		import java.util.Arrays;
		import java.util.Collections;
		import java.util.HashMap;
		import java.util.HashSet;
		import java.util.List;
		import java.util.Map;
		import java.util.Map.Entry;
		import java.util.Objects;
		@@ -61,6 +72,11 @@ import java.util.Set;
		public class Vcn extends Handler {
		private static final String TAG = Vcn.class.getSimpleName();

		private static final int VCN_LEGACY_SCORE_INT = 52;

		private static final List<Integer> CAPS_REQUIRING_MOBILE_DATA =
		Arrays.asList(NET_CAPABILITY_INTERNET, NET_CAPABILITY_DUN);

		private static final int MSG_EVENT_BASE = 0;
		private static final int MSG_CMD_BASE = 100;

		@@ -110,6 +126,15 @@ public class Vcn extends Handler {
		*/
		private static final int MSG_EVENT_SAFE_MODE_STATE_CHANGED = MSG_EVENT_BASE + 4;

		/**
		* Triggers reevaluation of mobile data enabled conditions.
		*
		* <p>Upon this notification, the VCN will check if any of the underlying subIds have mobile
		* data enabled. If not, the VCN will restart any GatewayConnections providing INTERNET or DUN
		* with the current mobile data toggle status.
		*/
		private static final int MSG_EVENT_MOBILE_DATA_TOGGLED = MSG_EVENT_BASE + 5;

		/** Triggers an immediate teardown of the entire Vcn, including GatewayConnections. */
		private static final int MSG_CMD_TEARDOWN = MSG_CMD_BASE;

		@@ -118,6 +143,8 @@ public class Vcn extends Handler {
		@NonNull private final Dependencies mDeps;
		@NonNull private final VcnNetworkRequestListener mRequestListener;
		@NonNull private final VcnCallback mVcnCallback;
		@NonNull private final VcnContentResolver mContentResolver;
		@NonNull private final ContentObserver mMobileDataSettingsObserver;

		/**
		* Map containing all VcnGatewayConnections and their VcnGatewayConnectionConfigs.
		@@ -154,6 +181,8 @@ public class Vcn extends Handler {
		// Accessed from different threads, but always under lock in VcnManagementService
		private volatile int mCurrentStatus = VCN_STATUS_CODE_ACTIVE;

		private boolean mIsMobileDataEnabled = false;

		public Vcn(
		@NonNull VcnContext vcnContext,
		@NonNull ParcelUuid subscriptionGroup,
		@@ -177,10 +206,19 @@ public class Vcn extends Handler {
		mVcnCallback = Objects.requireNonNull(vcnCallback, "Missing vcnCallback");
		mDeps = Objects.requireNonNull(deps, "Missing deps");
		mRequestListener = new VcnNetworkRequestListener();
		mContentResolver = mDeps.newVcnContentResolver(mVcnContext);
		mMobileDataSettingsObserver = new VcnMobileDataContentObserver(this /* handler */);

		final Uri uri = Settings.Global.getUriFor(Settings.Global.MOBILE_DATA);
		mContentResolver.registerContentObserver(
		uri, true /* notifyForDescendants */, mMobileDataSettingsObserver);

		mConfig = Objects.requireNonNull(config, "Missing config");
		mLastSnapshot = Objects.requireNonNull(snapshot, "Missing snapshot");

		// Update mIsMobileDataEnabled before starting handling of NetworkRequests.
		mIsMobileDataEnabled = getMobileDataStatus();

		// Register to receive cached and future NetworkRequests
		mVcnContext.getVcnNetworkProvider().registerListener(mRequestListener);
		}
		@@ -230,10 +268,10 @@ public class Vcn extends Handler {

		private class VcnNetworkRequestListener implements VcnNetworkProvider.NetworkRequestListener {
		@Override
		public void onNetworkRequested(@NonNull NetworkRequest request, int score, int providerId) {
		public void onNetworkRequested(@NonNull NetworkRequest request) {
		Objects.requireNonNull(request, "Missing request");

		sendMessage(obtainMessage(MSG_EVENT_NETWORK_REQUESTED, score, providerId, request));
		sendMessage(obtainMessage(MSG_EVENT_NETWORK_REQUESTED, request));
		}
		}

		@@ -249,7 +287,7 @@ public class Vcn extends Handler {
		handleConfigUpdated((VcnConfig) msg.obj);
		break;
		case MSG_EVENT_NETWORK_REQUESTED:
		handleNetworkRequested((NetworkRequest) msg.obj, msg.arg1, msg.arg2);
		handleNetworkRequested((NetworkRequest) msg.obj);
		break;
		case MSG_EVENT_SUBSCRIPTIONS_CHANGED:
		handleSubscriptionsChanged((TelephonySubscriptionSnapshot) msg.obj);
		@@ -260,6 +298,9 @@ public class Vcn extends Handler {
		case MSG_EVENT_SAFE_MODE_STATE_CHANGED:
		handleSafeModeStatusChanged();
		break;
		case MSG_EVENT_MOBILE_DATA_TOGGLED:
		handleMobileDataToggled();
		break;
		case MSG_CMD_TEARDOWN:
		handleTeardown();
		break;
		@@ -327,25 +368,9 @@ public class Vcn extends Handler {
		}
		}

		private void handleNetworkRequested(
		@NonNull NetworkRequest request, int score, int providerId) {
		private void handleNetworkRequested(@NonNull NetworkRequest request) {
		Slog.v(getLogTag(), "Received request " + request);

		if (score > getNetworkScore()) {
		if (VDBG) {
		Slog.v(
		getLogTag(),
		"Request already satisfied by higher-scoring ("
		+ score
		+ ") network from "
		+ "provider "
		+ providerId
		+ ": "
		+ request);
		}
		return;
		}

		// If preexisting VcnGatewayConnection(s) satisfy request, return
		for (VcnGatewayConnectionConfig gatewayConnectionConfig : mVcnGatewayConnections.keySet()) {
		if (isRequestSatisfiedByGatewayConnectionConfig(request, gatewayConnectionConfig)) {
		@@ -366,18 +391,37 @@ public class Vcn extends Handler {
		if (isRequestSatisfiedByGatewayConnectionConfig(request, gatewayConnectionConfig)) {
		Slog.v(getLogTag(), "Bringing up new VcnGatewayConnection for request " + request);

		if (getExposedCapabilitiesForMobileDataState(gatewayConnectionConfig).isEmpty()) {
		// Skip; this network does not provide any services if mobile data is disabled.
		continue;
		}

		final VcnGatewayConnection vcnGatewayConnection =
		mDeps.newVcnGatewayConnection(
		mVcnContext,
		mSubscriptionGroup,
		mLastSnapshot,
		gatewayConnectionConfig,
		new VcnGatewayStatusCallbackImpl(gatewayConnectionConfig));
		new VcnGatewayStatusCallbackImpl(gatewayConnectionConfig),
		mIsMobileDataEnabled);
		mVcnGatewayConnections.put(gatewayConnectionConfig, vcnGatewayConnection);
		}
		}
		}

		private Set<Integer> getExposedCapabilitiesForMobileDataState(
		VcnGatewayConnectionConfig gatewayConnectionConfig) {
		if (mIsMobileDataEnabled) {
		return gatewayConnectionConfig.getAllExposedCapabilities();
		}

		final Set<Integer> exposedCapsWithoutMobileData =
		new ArraySet<>(gatewayConnectionConfig.getAllExposedCapabilities());
		exposedCapsWithoutMobileData.removeAll(CAPS_REQUIRING_MOBILE_DATA);

		return exposedCapsWithoutMobileData;
		}

		private void handleGatewayConnectionQuit(VcnGatewayConnectionConfig config) {
		Slog.v(getLogTag(), "VcnGatewayConnection quit: " + config);
		mVcnGatewayConnections.remove(config);
		@@ -396,12 +440,55 @@ public class Vcn extends Handler {
		}
		}

		private void handleMobileDataToggled() {
		final boolean oldMobileDataEnabledStatus = mIsMobileDataEnabled;
		mIsMobileDataEnabled = getMobileDataStatus();

		if (oldMobileDataEnabledStatus != mIsMobileDataEnabled) {
		// Teardown any GatewayConnections that advertise INTERNET or DUN. If they provide other
		// services, the VcnGatewayConnections will be restarted without advertising INTERNET or
		// DUN.
		for (Entry<VcnGatewayConnectionConfig, VcnGatewayConnection> entry :
		mVcnGatewayConnections.entrySet()) {
		final VcnGatewayConnectionConfig gatewayConnectionConfig = entry.getKey();
		final VcnGatewayConnection gatewayConnection = entry.getValue();

		final Set<Integer> exposedCaps =
		gatewayConnectionConfig.getAllExposedCapabilities();
		if (exposedCaps.contains(NET_CAPABILITY_INTERNET)
		\|\| exposedCaps.contains(NET_CAPABILITY_DUN)) {
		if (gatewayConnection == null) {
		Slog.wtf(
		getLogTag(),
		"Found gatewayConnectionConfig without GatewayConnection");
		} else {
		// TODO(b/184868850): Optimize by restarting NetworkAgents without teardown.
		gatewayConnection.teardownAsynchronously();
		}
		}
		}
		}
		}

		private boolean getMobileDataStatus() {
		final TelephonyManager genericTelMan =
		mVcnContext.getContext().getSystemService(TelephonyManager.class);

		for (int subId : mLastSnapshot.getAllSubIdsInGroup(mSubscriptionGroup)) {
		if (genericTelMan.createForSubscriptionId(subId).isDataEnabled()) {
		return true;
		}
		}

		return false;
		}

		private boolean isRequestSatisfiedByGatewayConnectionConfig(
		@NonNull NetworkRequest request, @NonNull VcnGatewayConnectionConfig config) {
		final NetworkCapabilities.Builder builder = new NetworkCapabilities.Builder();
		builder.addTransportType(TRANSPORT_CELLULAR);
		builder.addCapability(NET_CAPABILITY_NOT_VCN_MANAGED);
		for (int cap : config.getAllExposedCapabilities()) {
		for (int cap : getExposedCapabilitiesForMobileDataState(config)) {
		builder.addCapability(cap);
		}

		@@ -432,12 +519,20 @@ public class Vcn extends Handler {
		pw.decreaseIndent();
		}

		@VisibleForTesting(visibility = Visibility.PRIVATE)
		public boolean isMobileDataEnabled() {
		return mIsMobileDataEnabled;
		}

		@VisibleForTesting(visibility = Visibility.PRIVATE)
		public void setMobileDataEnabled(boolean isMobileDataEnabled) {
		mIsMobileDataEnabled = isMobileDataEnabled;
		}

		/** Retrieves the network score for a VCN Network */
		// Package visibility for use in VcnGatewayConnection
		static int getNetworkScore() {
		// TODO: STOPSHIP (b/173549607): Make this use new NetworkSelection, or some magic "max in
		// subGrp" value
		return 52;
		// Package visibility for use in VcnGatewayConnection and VcnNetworkProvider
		static NetworkScore getNetworkScore() {
		return new NetworkScore.Builder().setLegacyInt(VCN_LEGACY_SCORE_INT).build();
		}

		/** Callback used for passing status signals from a VcnGatewayConnection to its managing Vcn. */
		@@ -485,6 +580,17 @@ public class Vcn extends Handler {
		}
		}

		private class VcnMobileDataContentObserver extends ContentObserver {
		private VcnMobileDataContentObserver(Handler handler) {
		super(handler);
		}

		@Override
		public void onChange(boolean selfChange) {
		sendMessage(obtainMessage(MSG_EVENT_MOBILE_DATA_TOGGLED));
		}
		}

		/** External dependencies used by Vcn, for injection in tests */
		@VisibleForTesting(visibility = Visibility.PRIVATE)
		public static class Dependencies {
		@@ -494,13 +600,36 @@ public class Vcn extends Handler {
		ParcelUuid subscriptionGroup,
		TelephonySubscriptionSnapshot snapshot,
		VcnGatewayConnectionConfig connectionConfig,
		VcnGatewayStatusCallback gatewayStatusCallback) {
		VcnGatewayStatusCallback gatewayStatusCallback,
		boolean isMobileDataEnabled) {
		return new VcnGatewayConnection(
		vcnContext,
		subscriptionGroup,
		snapshot,
		connectionConfig,
		gatewayStatusCallback);
		gatewayStatusCallback,
		isMobileDataEnabled);
		}

		/** Builds a new VcnContentResolver instance */
		public VcnContentResolver newVcnContentResolver(VcnContext vcnContext) {
		return new VcnContentResolver(vcnContext);
		}
		}

		/** Proxy Implementation of NetworkAgent, used for testing. */
		@VisibleForTesting(visibility = Visibility.PRIVATE)
		public static class VcnContentResolver {
		private final ContentResolver mImpl;

		public VcnContentResolver(VcnContext vcnContext) {
		mImpl = vcnContext.getContext().getContentResolver();
		}

		/** Registers the content observer */
		public void registerContentObserver(
		@NonNull Uri uri, boolean notifyForDescendants, @NonNull ContentObserver observer) {
		mImpl.registerContentObserver(uri, notifyForDescendants, observer);
		}
		}
		}

services/core/java/com/android/server/vcn/VcnGatewayConnection.java

+27 −6

Original line number	Diff line number	Diff line
		@@ -16,6 +16,8 @@

		package com.android.server.vcn;

		import static android.net.NetworkCapabilities.NET_CAPABILITY_DUN;
		import static android.net.NetworkCapabilities.NET_CAPABILITY_INTERNET;
		import static android.net.NetworkCapabilities.NET_CAPABILITY_NOT_CONGESTED;
		import static android.net.NetworkCapabilities.NET_CAPABILITY_NOT_METERED;
		import static android.net.NetworkCapabilities.NET_CAPABILITY_NOT_ROAMING;
		@@ -47,6 +49,7 @@ import android.net.NetworkAgent;
		import android.net.NetworkAgentConfig;
		import android.net.NetworkCapabilities;
		import android.net.NetworkProvider;
		import android.net.NetworkScore;
		import android.net.RouteInfo;
		import android.net.TelephonyNetworkSpecifier;
		import android.net.Uri;
		@@ -517,6 +520,7 @@ public class VcnGatewayConnection extends StateMachine {
		@NonNull private final VcnGatewayStatusCallback mGatewayStatusCallback;
		@NonNull private final Dependencies mDeps;
		@NonNull private final VcnUnderlyingNetworkTrackerCallback mUnderlyingNetworkTrackerCallback;
		private final boolean mIsMobileDataEnabled;

		@NonNull private final IpSecManager mIpSecManager;

		@@ -626,13 +630,15 @@ public class VcnGatewayConnection extends StateMachine {
		@NonNull ParcelUuid subscriptionGroup,
		@NonNull TelephonySubscriptionSnapshot snapshot,
		@NonNull VcnGatewayConnectionConfig connectionConfig,
		@NonNull VcnGatewayStatusCallback gatewayStatusCallback) {
		@NonNull VcnGatewayStatusCallback gatewayStatusCallback,
		boolean isMobileDataEnabled) {
		this(
		vcnContext,
		subscriptionGroup,
		snapshot,
		connectionConfig,
		gatewayStatusCallback,
		isMobileDataEnabled,
		new Dependencies());
		}

		@@ -643,6 +649,7 @@ public class VcnGatewayConnection extends StateMachine {
		@NonNull TelephonySubscriptionSnapshot snapshot,
		@NonNull VcnGatewayConnectionConfig connectionConfig,
		@NonNull VcnGatewayStatusCallback gatewayStatusCallback,
		boolean isMobileDataEnabled,
		@NonNull Dependencies deps) {
		super(TAG, Objects.requireNonNull(vcnContext, "Missing vcnContext").getLooper());
		mVcnContext = vcnContext;
		@@ -650,6 +657,7 @@ public class VcnGatewayConnection extends StateMachine {
		mConnectionConfig = Objects.requireNonNull(connectionConfig, "Missing connectionConfig");
		mGatewayStatusCallback =
		Objects.requireNonNull(gatewayStatusCallback, "Missing gatewayStatusCallback");
		mIsMobileDataEnabled = isMobileDataEnabled;
		mDeps = Objects.requireNonNull(deps, "Missing deps");

		mLastSnapshot = Objects.requireNonNull(snapshot, "Missing snapshot");
		@@ -1502,7 +1510,7 @@ public class VcnGatewayConnection extends StateMachine {
		@NonNull VcnNetworkAgent agent,
		@NonNull VcnChildSessionConfiguration childConfig) {
		final NetworkCapabilities caps =
		buildNetworkCapabilities(mConnectionConfig, mUnderlying);
		buildNetworkCapabilities(mConnectionConfig, mUnderlying, mIsMobileDataEnabled);
		final LinkProperties lp =
		buildConnectedLinkProperties(
		mConnectionConfig, tunnelIface, childConfig, mUnderlying);
		@@ -1515,7 +1523,7 @@ public class VcnGatewayConnection extends StateMachine {
		@NonNull IpSecTunnelInterface tunnelIface,
		@NonNull VcnChildSessionConfiguration childConfig) {
		final NetworkCapabilities caps =
		buildNetworkCapabilities(mConnectionConfig, mUnderlying);
		buildNetworkCapabilities(mConnectionConfig, mUnderlying, mIsMobileDataEnabled);
		final LinkProperties lp =
		buildConnectedLinkProperties(
		mConnectionConfig, tunnelIface, childConfig, mUnderlying);
		@@ -1843,7 +1851,8 @@ public class VcnGatewayConnection extends StateMachine {
		@VisibleForTesting(visibility = Visibility.PRIVATE)
		static NetworkCapabilities buildNetworkCapabilities(
		@NonNull VcnGatewayConnectionConfig gatewayConnectionConfig,
		@Nullable UnderlyingNetworkRecord underlying) {
		@Nullable UnderlyingNetworkRecord underlying,
		boolean isMobileDataEnabled) {
		final NetworkCapabilities.Builder builder = new NetworkCapabilities.Builder();

		builder.addTransportType(TRANSPORT_CELLULAR);
		@@ -1853,6 +1862,12 @@ public class VcnGatewayConnection extends StateMachine {

		// Add exposed capabilities
		for (int cap : gatewayConnectionConfig.getAllExposedCapabilities()) {
		// Skip adding INTERNET or DUN if mobile data is disabled.
		if (!isMobileDataEnabled
		&& (cap == NET_CAPABILITY_INTERNET \|\| cap == NET_CAPABILITY_DUN)) {
		continue;
		}

		builder.addCapability(cap);
		}

		@@ -2040,6 +2055,12 @@ public class VcnGatewayConnection extends StateMachine {
		"mNetworkAgent.getNetwork(): "
		+ (mNetworkAgent == null ? null : mNetworkAgent.getNetwork()));

		pw.println("mUnderlying:");
		pw.increaseIndent();
		mUnderlying.dump(pw);
		pw.decreaseIndent();
		pw.println();

		pw.decreaseIndent();
		}

		@@ -2183,7 +2204,7 @@ public class VcnGatewayConnection extends StateMachine {
		@NonNull String tag,
		@NonNull NetworkCapabilities caps,
		@NonNull LinkProperties lp,
		@NonNull int score,
		@NonNull NetworkScore score,
		@NonNull NetworkAgentConfig nac,
		@NonNull NetworkProvider provider,
		@NonNull Consumer<VcnNetworkAgent> networkUnwantedCallback,
		@@ -2324,7 +2345,7 @@ public class VcnGatewayConnection extends StateMachine {
		@NonNull String tag,
		@NonNull NetworkCapabilities caps,
		@NonNull LinkProperties lp,
		@NonNull int score,
		@NonNull NetworkScore score,
		@NonNull NetworkAgentConfig nac,
		@NonNull NetworkProvider provider,
		@NonNull Consumer<VcnNetworkAgent> networkUnwantedCallback,