In permission review mode, always request user's consent to toggle BT. (6251e6a7) · Commits · e / os / android_packages_modules_Bluetooth

service/java/com/android/server/bluetooth/BluetoothManagerService.java

+9 −15

Original line number	Diff line number	Diff line
		@@ -685,9 +685,8 @@ class BluetoothManagerService extends IBluetoothManager.Stub {
		mContext.enforceCallingOrSelfPermission(BLUETOOTH_ADMIN_PERM,
		"Need BLUETOOTH ADMIN permission");

		if (!isEnabled() && mPermissionReviewRequired
		&& startConsentUiIfNeeded(packageName, callingUid,
		BluetoothAdapter.ACTION_REQUEST_ENABLE)) {
		if (!isEnabled() && mPermissionReviewRequired) {
		startConsentUi(packageName, callingUid, BluetoothAdapter.ACTION_REQUEST_ENABLE);
		return false;
		}
		}
		@@ -721,9 +720,8 @@ class BluetoothManagerService extends IBluetoothManager.Stub {
		mContext.enforceCallingOrSelfPermission(BLUETOOTH_ADMIN_PERM,
		"Need BLUETOOTH ADMIN permission");

		if (isEnabled() && mPermissionReviewRequired
		&& startConsentUiIfNeeded(packageName, callingUid,
		BluetoothAdapter.ACTION_REQUEST_DISABLE)) {
		if (isEnabled() && mPermissionReviewRequired) {
		startConsentUi(packageName, callingUid, BluetoothAdapter.ACTION_REQUEST_DISABLE);
		return false;
		}
		}
		@@ -743,8 +741,8 @@ class BluetoothManagerService extends IBluetoothManager.Stub {
		return true;
		}

		private boolean startConsentUiIfNeeded(String packageName,
		int callingUid, String intentAction) throws RemoteException {
		private void startConsentUi(String packageName, int callingUid, String intentAction)
		throws RemoteException {
		try {
		// Validate the package only if we are going to use it
		ApplicationInfo applicationInfo = mContext.getPackageManager()
		@@ -756,16 +754,12 @@ class BluetoothManagerService extends IBluetoothManager.Stub {
		+ " not in uid " + callingUid);
		}

		// Legacy apps in permission review mode trigger a user prompt
		if (applicationInfo.targetSdkVersion < Build.VERSION_CODES.M) {
		// Permission review mode, trigger a user prompt
		Intent intent = new Intent(intentAction);
		mContext.startActivity(intent);
		return true;
		}
		} catch (PackageManager.NameNotFoundException e) {
		throw new RemoteException(e.getMessage());
		}
		return false;
		}

		public void unbindAndFinish() {