Respect AppOpsManager.opAllowsReset

jclehner · May 14, 2015 · 8abb5cb · 8abb5cb
1 parent 356455f
commit 8abb5cb
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 3 deletions.
diff --git a/src/at/jclehner/appopsxposed/AppListFragment.java b/src/at/jclehner/appopsxposed/AppListFragment.java
@@ -508,10 +508,11 @@ private void resetChangedOps()
 			final int uid = pi.packageInfo.applicationInfo.uid;
 			final String packageName = pi.packageInfo.packageName;
 
-			for(OpEntryWrapper op : pi.changedOps)
+			for(OpEntryWrapper entry : pi.changedOps)
 			{
-				appOps.setMode(op.getOp(), uid, packageName,
-						AppOpsManagerWrapper.opToDefaultMode(op.getOp()));
+				final int op = entry.getOp();
+				if(AppOpsManagerWrapper.opAllowsReset(op))
+					appOps.setMode(op, uid, packageName, AppOpsManagerWrapper.opToDefaultMode(op));
 			}
 		}
 

diff --git a/src/at/jclehner/appopsxposed/util/AppOpsManagerWrapper.java b/src/at/jclehner/appopsxposed/util/AppOpsManagerWrapper.java
@@ -160,6 +160,19 @@ public static int opToDefaultMode(int op)
 		}
 	}
 
+	public static boolean opAllowsReset(int op)
+	{
+		try
+		{
+			return callStatic(AppOpsManager.class, "opAllowsReset", new Class<?>[] { int.class }, op);
+		}
+		catch(ReflectiveException e)
+		{
+			Util.debug(e);
+			return op != OP_WRITE_SMS;
+		}
+	}
+
 	public static int opFromName(String opName)
 	{
 		if(!opName.startsWith("OP_"))