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

Stop using NetworkStack#checkNetworkStackPermission* am: d6789606d5

Browse Source 
Original change: https://android-review.googlesource.com/c/platform/frameworks/opt/net/ethernet/+/1827093

Change-Id: I304701a1ad5be47bec534d679eca259b06cb084d
This commit is contained in:
paulhu
2021-09-16 02:03:58 +00:00
committed by Automerger Merge Worker
parent 07060e39fd d6789606d5
commit 1932a7e7ec
2 changed files with 7 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
service-t/src/com/android/server/ethernet/EthernetServiceImpl.java
View File

@@ -22,16 +22,15 @@ import android.net.IEthernetManager;
import android.net.IEthernetServiceListener; import android.net.IEthernetServiceListener;
import android.net.ITetheredInterfaceCallback; import android.net.ITetheredInterfaceCallback;
import android.net.IpConfiguration; import android.net.IpConfiguration;
import android.net.NetworkStack;
import android.os.Binder; import android.os.Binder;
import android.os.Handler; import android.os.Handler;
import android.os.HandlerThread; import android.os.HandlerThread;
import android.os.RemoteException; import android.os.RemoteException;
import android.provider.Settings;
import android.util.Log; import android.util.Log;
import android.util.PrintWriterPrinter; import android.util.PrintWriterPrinter;
import com.android.internal.util.IndentingPrintWriter; import com.android.internal.util.IndentingPrintWriter;
import com.android.net.module.util.PermissionUtils;
import java.io.FileDescriptor; import java.io.FileDescriptor;
import java.io.PrintWriter; import java.io.PrintWriter;
@@ -117,7 +116,7 @@ public class EthernetServiceImpl extends IEthernetManager.Stub {
Log.w(TAG, "System isn't ready enough to change ethernet configuration"); Log.w(TAG, "System isn't ready enough to change ethernet configuration");
} }
NetworkStack.checkNetworkStackPermission(mContext); PermissionUtils.enforceNetworkStackPermission(mContext);
if (mTracker.isRestrictedInterface(iface)) { if (mTracker.isRestrictedInterface(iface)) {
enforceUseRestrictedNetworksPermission(); enforceUseRestrictedNetworksPermission();
@@ -168,7 +167,7 @@ public class EthernetServiceImpl extends IEthernetManager.Stub {
@Override @Override
public void setIncludeTestInterfaces(boolean include) { public void setIncludeTestInterfaces(boolean include) {
NetworkStack.checkNetworkStackPermissionOr(mContext, PermissionUtils.enforceNetworkStackPermissionOr(mContext,
android.Manifest.permission.NETWORK_SETTINGS); android.Manifest.permission.NETWORK_SETTINGS);
mTracker.setIncludeTestInterfaces(include); mTracker.setIncludeTestInterfaces(include);
} }
@@ -176,7 +175,7 @@ public class EthernetServiceImpl extends IEthernetManager.Stub {
@Override @Override
public void requestTetheredInterface(ITetheredInterfaceCallback callback) { public void requestTetheredInterface(ITetheredInterfaceCallback callback) {
Objects.requireNonNull(callback, "callback must not be null"); Objects.requireNonNull(callback, "callback must not be null");
NetworkStack.checkNetworkStackPermissionOr(mContext, PermissionUtils.enforceNetworkStackPermissionOr(mContext,
android.Manifest.permission.NETWORK_SETTINGS); android.Manifest.permission.NETWORK_SETTINGS);
mTracker.requestTetheredInterface(callback); mTracker.requestTetheredInterface(callback);
} }
@@ -184,7 +183,7 @@ public class EthernetServiceImpl extends IEthernetManager.Stub {
@Override @Override
public void releaseTetheredInterface(ITetheredInterfaceCallback callback) { public void releaseTetheredInterface(ITetheredInterfaceCallback callback) {
Objects.requireNonNull(callback, "callback must not be null"); Objects.requireNonNull(callback, "callback must not be null");
NetworkStack.checkNetworkStackPermissionOr(mContext, PermissionUtils.enforceNetworkStackPermissionOr(mContext,
android.Manifest.permission.NETWORK_SETTINGS); android.Manifest.permission.NETWORK_SETTINGS);
mTracker.releaseTetheredInterface(callback); mTracker.releaseTetheredInterface(callback);
} }

4
service-t/src/com/android/server/ethernet/EthernetTracker.java
View File

@@ -29,7 +29,6 @@ import android.net.IpConfiguration.IpAssignment;
import android.net.IpConfiguration.ProxySettings; import android.net.IpConfiguration.ProxySettings;
import android.net.LinkAddress; import android.net.LinkAddress;
import android.net.NetworkCapabilities; import android.net.NetworkCapabilities;
import android.net.NetworkStack;
import android.net.StaticIpConfiguration; import android.net.StaticIpConfiguration;
import android.os.Handler; import android.os.Handler;
import android.os.IBinder; import android.os.IBinder;
@@ -45,6 +44,7 @@ import android.net.util.NetdService;
import com.android.internal.annotations.VisibleForTesting; import com.android.internal.annotations.VisibleForTesting;
import com.android.internal.util.IndentingPrintWriter; import com.android.internal.util.IndentingPrintWriter;
import com.android.net.module.util.NetdUtils; import com.android.net.module.util.NetdUtils;
import com.android.net.module.util.PermissionUtils;
import com.android.server.net.BaseNetworkObserver; import com.android.server.net.BaseNetworkObserver;
import java.io.FileDescriptor; import java.io.FileDescriptor;
@@ -287,7 +287,7 @@ final class EthernetTracker {
InterfaceConfiguration config = null; InterfaceConfiguration config = null;
// Bring up the interface so we get link status indications. // Bring up the interface so we get link status indications.
try { try {
NetworkStack.checkNetworkStackPermission(mContext); PermissionUtils.enforceNetworkStackPermission(mContext);
NetdUtils.setInterfaceUp(mNetd, iface); NetdUtils.setInterfaceUp(mNetd, iface);
config = mNMService.getInterfaceConfig(iface); config = mNMService.getInterfaceConfig(iface);
} catch (RemoteException | IllegalStateException e) { } catch (RemoteException | IllegalStateException e) {