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

Commit 3f1d4557 authored by Nate Myren's avatar Nate Myren
Browse files

Log invalid Perm Groups

Test: attempt to create a PermGroupLiveData with an invalid permission
name

Change-Id: Ic6d66091463f3281be3b9cab49dfe8fab98a1e9e
parent c0d3f088
Loading
Loading
Loading
Loading
+5 −0
Original line number Diff line number Diff line
@@ -22,6 +22,7 @@ import android.content.pm.PackageManager
import android.content.pm.PermissionGroupInfo
import android.content.pm.PermissionInfo
import android.os.UserHandle
import android.util.Log
import com.android.permissioncontroller.permission.data.PackageInfoRepository.getPackageBroadcastReceiver
import com.android.permissioncontroller.permission.data.PackageInfoRepository.getPackageInfoLiveData
import com.android.permissioncontroller.permission.model.livedatatypes.LightPermGroupInfo
@@ -43,6 +44,8 @@ class PermGroupLiveData(
) : SmartUpdateMediatorLiveData<PermGroup>(),
    PackageBroadcastReceiver.PackageBroadcastListener {

    private val LOG_TAG = this::class.java.simpleName

    private val context = app.applicationContext!!

    /**
@@ -84,6 +87,7 @@ class PermGroupLiveData(
        val permissionInfos = mutableMapOf<String, LightPermInfo>()

        groupInfo = Utils.getGroupInfo(groupName, context) ?: run {
            Log.e(LOG_TAG, "Invalid permission group $groupName")
            value = null
            return
        }
@@ -94,6 +98,7 @@ class PermGroupLiveData(
                    Utils.getInstalledRuntimePermissionInfosForGroup(context.packageManager,
                        groupName)
                } catch (e: PackageManager.NameNotFoundException) {
                    Log.e(LOG_TAG, "Invalid permission group $groupName")
                    value = null
                    return
                }