Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit e12511f8 authored by TreeHugger Robot's avatar TreeHugger Robot Committed by Automerger Merge Worker
Browse files

Merge "Change SharedConnectivityService's permission check to use... 

Merge "Change SharedConnectivityService's permission check to use checkCallingOrSelfPermission" into udc-dev am: 68e79acf

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



Change-Id: I2b64ecface8728ad939b5e885bf110fcecfa2162
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents b39273ed 68e79acf

wifi/java/src/android/net/wifi/sharedconnectivity/service/SharedConnectivityService.java

+10 −3
Original line number Diff line number Diff line
@@ -130,9 +130,16 @@ public abstract class SharedConnectivityService extends Service { 


            @RequiresPermission(anyOf = {android.Manifest.permission.NETWORK_SETTINGS,

                    android.Manifest.permission.NETWORK_SETUP_WIZARD})

            /**

             * checkPermissions is using checkCallingOrSelfPermission to support CTS testing of this

             * service. This does allow a process to bind to itself if it holds the proper

             * permission. We do not consider this to be an issue given that the process can already

             * access the service data since they are in the same process.

             */

            private void checkPermissions() {

                if (checkCallingPermission(NETWORK_SETTINGS) != PackageManager.PERMISSION_GRANTED

                        && checkCallingPermission(NETWORK_SETUP_WIZARD)

                if (checkCallingOrSelfPermission(NETWORK_SETTINGS)

                        != PackageManager.PERMISSION_GRANTED

                        && checkCallingOrSelfPermission(NETWORK_SETUP_WIZARD)

                        != PackageManager.PERMISSION_GRANTED) {

                    throw new SecurityException("Calling process must have NETWORK_SETTINGS or"

                            + " NETWORK_SETUP_WIZARD permission");