Update RefInvoker

limpoxe · limpoxe · commit b25279c7b6ab · 2016-01-28T14:10:24.000+08:00
diff --git a/PluginCore/src/com/plugin/core/proxy/MethodHandler.java b/PluginCore/src/com/plugin/core/proxy/MethodHandler.java
@@ -31,7 +31,9 @@ public synchronized Object invoke(Object proxy, Method method, Object[] args) th
 
         Object invokeResult = null;
         if (before == null) {
-            method.setAccessible(true);
+            if (!method.isAccessible()) {
+                method.setAccessible(true);
+            }
             invokeResult = method.invoke(mTarget, args);
         }
 
diff --git a/PluginCore/src/com/plugin/util/RefInvoker.java b/PluginCore/src/com/plugin/util/RefInvoker.java
@@ -34,7 +34,9 @@ public static Object invokeMethod(Object target, Class clazz, String methodName,
 		try {
 			//LogUtil.e("Method", methodName);
 			Method method = clazz.getDeclaredMethod(methodName, paramTypes);
-			method.setAccessible(true);
+			if (!method.isAccessible()) {
+				method.setAccessible(true);
+			}
 			return method.invoke(target, paramValues);
 		} catch (SecurityException e) {
 			e.printStackTrace();
@@ -54,7 +56,9 @@ public static Object invokeMethod(Object target, Class clazz, String methodName,
 	public static Object getFieldObject(Object target, Class clazz, String fieldName) {
 		try {
 			Field field = clazz.getDeclaredField(fieldName);
-			field.setAccessible(true);
+			if (!field.isAccessible()) {
+				field.setAccessible(true);
+			}
 			return field.get(target);
 		} catch (SecurityException e) {
 			e.printStackTrace();
@@ -108,15 +112,19 @@ public static void setFieldObject(Object target, String className, String fieldN
 	public static void setFieldObject(Object target, Class clazz, String fieldName, Object fieldValue) {
 		try {
 			Field field = clazz.getDeclaredField(fieldName);
-			field.setAccessible(true);
+			if (!field.isAccessible()) {
+				field.setAccessible(true);
+			}
 			field.set(target, fieldValue);
 		} catch (SecurityException e) {
 			e.printStackTrace();
 		} catch (NoSuchFieldException e) {
 			// try supper for Miui, Miui has a class named MiuiPhoneWindow
 			try {
 				Field field = clazz.getSuperclass().getDeclaredField(fieldName);
-				field.setAccessible(true);
+				if (!field.isAccessible()) {
+					field.setAccessible(true);
+				}
 				field.set(target, fieldValue);
 			} catch (Exception superE) {
 				e.printStackTrace();

Original file line number	Diff line number	Diff line change
`@@ -31,7 +31,9 @@ public synchronized Object invoke(Object proxy, Method method, Object[] args) th`
`31`	`31`
`32`	`32`	`Object invokeResult = null;`
`33`	`33`	`if (before == null) {`
`34`		`- method.setAccessible(true);`
	`34`	`+ if (!method.isAccessible()) {`
	`35`	`+ method.setAccessible(true);`
	`36`	`+ }`
`35`	`37`	`invokeResult = method.invoke(mTarget, args);`
`36`	`38`	`}`
`37`	`39`