XLuaHook::invoke() must copy all Varargs elements used by the hook.

The Varargs object passed to invoke() seems to be reused by the Lua
interpreter and capturing it as is when defining the hook leads to
accessing other elements than intended when the hook runs.
Instead, copy all the function object in fun and the the extra
arguments in xargs, and use these when defining the hook.

Author: F-i-f

app/src/main/java/eu/faircode/xlua/XLua.java

@@ -829,6 +829,10 @@ public Varargs invoke(final Varargs args) {
             String m = args.arg(2).checkjstring();
             args.arg(3).checkfunction();
             Log.i(TAG, "Dynamic hook " + cls.getName() + "." + m);
+            final LuaValue fun = args.arg(3);
+            final List<LuaValue> xargs = new ArrayList<>();
+            for (int i = 4; i <= args.narg(); i++)
+                xargs.add(args.arg(i));
 
             XposedBridge.hookAllMethods(cls, m, new XC_MethodHook() {
                 @Override
@@ -846,9 +850,9 @@ private void execute(String when, MethodHookParam param) {
                     List<LuaValue> values = new ArrayList<>();
                     values.add(LuaValue.valueOf(when));
                     values.add(CoerceJavaToLua.coerce(new XParam(context, param, settings)));
-                    for (int i = 4; i <= args.narg(); i++)
-                        values.add(args.arg(i));
-                    args.arg(3).invoke(values.toArray(new LuaValue[0]));
+                    for (int i = 0; i < xargs.size(); i++)
+                        values.add(xargs.get(i));
+                    fun.invoke(values.toArray(new LuaValue[0]));
                 }
             });