[BUG] HookParam call timing problem

[cesaryuan]

复现步骤：

1. 创建模块，代码如下，启用模块

loadApp(name = "com.wibo.bigbang.ocr") {
  findClass(name = "okhttp3.Request\$Builder").hook {
      injectMember {
          method {
              name = "addHeader"
              param(StringClass,StringClass)
              returnType = "okhttp3.Request\$Builder"
          }
          beforeHook {
              val a1 = args().first().string();
              loggerD("checkEqual", "", LoggerType.LOGD)
              val a2 = args().first().string();
              loggerD(
                  tag = "checkEqual",
                  msg = "${a1 == a2} $a1 $a2"
              )
          }
      }
  }
}

2. 安装宿主app：布丁扫描，启动宿主APP

3. 观察Logcat，发现有好几次输出了false，按理来说应该始终是true才对
   

4. 把a1后面的loggerD("checkEqual", "", LoggerType.LOGD)删掉后，程序运行正常（始终输出true
   

[fankes]

试试 XposedBridge.log 以及 Log.d

[cesaryuan]

试试 XposedBridge.log 以及 Log.d

谢谢大佬的回复，我试了下换成 Log.d 之后也还是一样的问题，但是出现 false 的几率变小了很多，然后我进一步测试，发现把 loggerD 换成 Thread.sleep(500) 后有一样的效果，出现 false 的几率特别大；但是换成 Thread.sleep(1) 之后就小了很多（还是会有 false）

[cesaryuan]

补充一下，发现一个临时的解决方案，在最前面加上 val args = args

[fankes]

因为 Log 会导致延迟，这个时候宿主的代码和你执行的时间不是同步的，执行的时间就会比你靠前，出现这种情况大概是正常的，这不是 API 的问题

因为 loggerD 会同时调用两个 Logd 耗时会更长

[fankes]

这大概不是 API 的管辖范围内，所以关闭了，不能认为是 bug

[cesaryuan]

感觉有点反直觉诶，为什么在同一次hook中args会发生变化呢，那这样的话感觉是一个很严重的问题呀，理论上每一次hook执行不应该是相互隔离的嘛。

执行的时间就会比你靠前

这里没太懂，beforehook不是总在originmethod之前运行，并且beforehook运行完后origin method才会执行嘛

beforehook(param)
orginMethod(param.args)
afterhook(param)

我以为的大概是这个流程，那按理说每一次运行到beforehook时，param.args就应该不会发生改变了才对。现在的情况看起来是不同的method调用之间的param.args之间会互相混在一起？

所以其实对于每一次method调用，共享的都是一个param？所以如果method调用的比较快就会导致后一次调用的param把前一次的覆盖掉？

[fankes]

你用原生 Xposed API 试一下，如果有这个问题你再告诉我

[cesaryuan]

你用原生 Xposed API 试一下，如果有这个问题你再告诉我

大佬我试了下原生Xposed，不会出现这个问题，代码如下

package com.example.test_yuki_bug;

import android.util.Log;

import de.robv.android.xposed.IXposedHookLoadPackage;
import de.robv.android.xposed.XC_MethodHook;
import de.robv.android.xposed.XposedHelpers;
import de.robv.android.xposed.callbacks.XC_LoadPackage;

public class MainHook implements IXposedHookLoadPackage {
    @Override
    public void handleLoadPackage(XC_LoadPackage.LoadPackageParam lpparam) throws Throwable {
        if (!lpparam.packageName.equals("com.wibo.bigbang.ocr")) return;
        hook(lpparam);
    }

    private void hook(XC_LoadPackage.LoadPackageParam lpparam) {
        XposedHelpers.findAndHookMethod("okhttp3.Request$Builder", 
                lpparam.classLoader, "addHeader", String.class, String.class, new XC_MethodHook() {
                    @Override
                    protected void beforeHookedMethod(MethodHookParam param) throws InterruptedException {
                        String a1 = (String) param.args[0];
                        Thread.sleep(500);
                        String a2 = (String) param.args[0];
                        Log.d("checkEqual", a1.equals(a2) + " a1: " + a1 + " a2: " + a2);
                    } 
                });
    }
}

[fankes]

那你把复现 apk 和 demo 都发上来，我有空试试

[fankes]

在注解声明 isUsingResourcesHook = false 再试试，无论如何先试试这个

如果依然不行，参考这个问题 #23，你可以切到这个版本试试可不可以修复，分别测试 1.1.4 和 1.1.5 版本

告诉我每个测试准确的结果。

[cesaryuan]

分别测试 1.1.4 和 1.1.5 版本

在这里修改就可以嘛

[cesaryuan]

在注解声明 isUsingResourcesHook = false 再试试

已测试，问题存在，且问题症状无明显区别（false出现几率相同）

分别测试 1.1.4 和 1.1.5 版本

已测试，都存在问题，且问题症状无明显区别（false出现几率相同）

你用原生 Xposed API 试一下

已测试，不存在问题，始终为 true

模块demo (Yuki)：yuki_bug_test.zip
模块demo (原Xposed)：同上（放在一个单独的Module里了）
宿主APK：（安装后需点击同意用户协议后运行到登录界面，但无需登录即可复现bug。如果出现闪退，禁用模块后运行一次宿主APP然后再启用模块即可）

• 123链接：https://www.123pan.com/s/oWqA-tk2od.html
• 蓝奏链接：https://wwrr.lanzoul.com/ixOt818cn3yh