ElderDrivers / ElderDrivers/EdXposed

[BUG] Issues hooking method

Open
#786 18 comments 0 reactions 0 assignees View on GitHub
bug
Dominant language
Java
Stars
5.7k
Forks
650
PR merge metrics
No merged PRs in 30d

Description

I'm trying to hook: https://github.com/aosp-mirror/platform_frameworks_base/blob/26b768d9f86633e4a3c23444b662ae66e4e6ffa4/services/core/java/com/android/server/LocationManagerService.java#L3515

```
private boolean canCallerAccessMockLocation(String opPackageName) {
return mAppOps.checkOp(AppOpsManager.OP_MOCK_LOCATION, Binder.getCallingUid(),
opPackageName) == AppOpsManager.MODE_ALLOWED;
}
```

```
try {
XC_MethodHook.Unhook mockProviderUnhooks;
mockProviderUnhooks = findAndHookMethod("com.android.server.LocationManagerService", classLoader, "canCallerAccessMockLocation", String.class, new XC_MethodReplacement() {
@Override
protected Object replaceHookedMethod(MethodHookParam param) throws Throwable {
//SharedPreferences pref = getPref("preferences");
//if (pref.getBoolean("mockProvider", false)) {
XposedBridge.log("section80 :: canCallerAccessMockLocation");
return true;
//}
}
});

if (mockProviderUnhooks != null) {
XposedBridge.log("section80 :: mock provider hook: " + mockProviderUnhooks.getHookedMethod().getName() + " :: " + mockProviderUnhooks.getHookedMethod().getDeclaringClass().getCanonicalName());
//XposedBridge.log("section80 :: packageName: " + loadPackageParam.packageName + " :: processName: " + loadPackageParam.processName);
}
}
catch (Throwable t) {
XposedBridge.log("section80 :: mock provider hook failed");
}
```

Everything seems to be reporting back fine, xposed supposedly has successfully hooked the method.. however the hook definitely is not working.

It's driving me nuts!

I'm running:
- Android: 10
- Magisk: 20.4 (20400)
- EdXposed: v5.1.4 (4655) SandHook.
- Riru: v23.1

Not sure if I've been staring at this for too long and missing something obvious here, but yeah.. no success.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.