fix(inventory): check permission to prevent securityException

Signed-off-by: Teclib <skita@teclib.com>
glpi-project · Oct 22, 2019 · ac3b161 · ac3b161
1 parent 810c7f5
commit ac3b161
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/inventory/src/main/java/org/flyve/inventory/Utils.java b/inventory/src/main/java/org/flyve/inventory/Utils.java
@@ -27,6 +27,7 @@
 package org.flyve.inventory;
 
 import android.content.Context;
+import android.content.pm.PackageManager;
 import android.os.Build;
 import android.os.Environment;
 import android.telephony.TelephonyManager;
@@ -188,7 +189,8 @@ protected static String createJSON(Context context, ArrayList<Categories> catego
             jsonQuery.put("versionClient", appVersion);
             jsonQuery.put("deviceId", getDeviceId());
             TelephonyManager telephonyManager = (TelephonyManager) context.getSystemService(Context.TELEPHONY_SERVICE);
-            if (telephonyManager != null) {
+            if (telephonyManager != null &&
+                    (context.checkPermission(android.Manifest.permission.READ_PHONE_STATE, android.os.Process.myPid(), android.os.Process.myUid())) == PackageManager.PERMISSION_GRANTED) {
                 jsonQuery.put("IMEI", telephonyManager.getDeviceId());
             }
             jsonQuery.put("content", content);