diff options
| author | paulhu <paulhu@google.com> | 2021-03-30 10:55:37 +0800 |
|---|---|---|
| committer | Paul Hu <paulhu@google.com> | 2021-04-26 06:05:39 +0000 |
| commit | 9c2ed6192d611399c374824d5cad5971cf756b51 (patch) | |
| tree | 2b91177ec4bedeb172e09d63b329b644587ff173 | |
| parent | e4d09da97d17d2375c542e374bdb4ecc2ac58d61 (diff) | |
| download | ethernet-9c2ed6192d611399c374824d5cad5971cf756b51.tar.gz | |
Enforce ACCESS_NETWORK_STATE to getAvailableInterfaces()
getAvailableInterfaces() will return available ethernet
interfaces which are the information about networks. So it should
enforce ACCESS_NETWORK_STATE permission check to ensure the
applications are allowed to access the information.
Bug: 174573778
Test: TetheringTests
Test: CtsTetheringTest
Change-Id: I7aaa5225d56f2feecc51ba263489ed0ce02fd651
| -rw-r--r-- | java/com/android/server/ethernet/EthernetServiceImpl.java | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/java/com/android/server/ethernet/EthernetServiceImpl.java b/java/com/android/server/ethernet/EthernetServiceImpl.java index 3fc6aab..c06f61e 100644 --- a/java/com/android/server/ethernet/EthernetServiceImpl.java +++ b/java/com/android/server/ethernet/EthernetServiceImpl.java @@ -87,6 +87,8 @@ public class EthernetServiceImpl extends IEthernetManager.Stub { @Override public String[] getAvailableInterfaces() throws RemoteException { + enforceAccessPermission(); + return mTracker.getInterfaces(checkUseRestrictedNetworksPermission()); } |