xLua: try to dispose a LuaEnv with C# callback!

该错误主要是在调用LuaEnv.Dispose()时引起,一般用于游戏重启时将整个lua虚拟机重置.
报错的可能原因有:

  1. 在同一个方法中释放了delegate的引用,但又立即调用了LuaEnv.Dispose().
  2. 没有在C#侧释放标记有[CSharpCallLua]的delegate(赋值为null).

详细说明可以在xLua自带的FAQ(xLua/Doc/faq.md)中找到.

  • 对于第1个问题, 可以将LuaEnv.Dispose()的释放代码单独放到另一个方法中, 参考 https://github.com/Tencent/xLua/issues/634

  • 对于第2个问题:
    xlua提供了一个lua方法(该方法位于XLua/Resources/xlua/util.lua.txt中): print_func_ref_by_csharp()来打印所有通过delegate注入到C#中的lua方法. 通过得到的信息可以依次检查lua往C#中的哪些地方注入了方法,这样就在C#中去把那些delegate类型的变量赋值为null进行释放了.

但该方式的信息依然不够全面,且当大量使用C#调用lua这种逻辑时,排查问题会很麻烦,所以如果想在报错时确切的知道到底是lua的哪个调用处没有释放,就需要对xLua源码进行额外的处理(v2.1.13测试没问题).

报错的地方是在LuaEnv.cs的Dispose(bool dispose)方法中抛出的, 如下图:

image.png

该处又是调用了translator.AllDelegateBridgeReleased()来得到的检查结果, translator是ObjectTranslator.cs.

在生成的各种wrap文件中,对于delegate的获取处理有如下类似代码:

ObjectTranslator translator = ObjectTranslatorPool.Instance.Find(L);
XXClass gen_to_be_invoked = (XXClass)translator.FastGetCSObj(L, 1);
gen_to_be_invoked.DelegateHandler = translator.GetDelegate<XXClass.DelegateType>(L, 2);

translator依然是ObjectTranslator,其中的GetDelegate()定义如下图:

image.png

对于delegate类型,lua传入肯定是个function,所以一定会执行到红圈处的代码,即CreateDelegateBridge().
修改逻辑如下:

  1. 在CreateDelegateBridge()时获取delegate的包装对象WeakReference; 同时获取lua处的调用堆栈信息,即通过lua的debug.traceback()(也可以是debug.getinfo())来得到lua传入function到C#时的文件和代码行号. 将这2个信息保存起来.
  2. 在ReleaseLuaBase()中移除保存的信息(该方法被LuaEnv.Tick()调用).
  3. 在AllDelegateBridgeReleased()中,当检查到某个delegate没有释放时,就打印其WeakReference(在第1步中)关联保存的lua信息,即可精确得到具体是哪里没有释放.

具体修改代码如下(在ObjectTranslator.cs中新增和修改如下代码):

        Dictionary<int, WeakReference> delegate_bridges = new Dictionary<int, WeakReference>();

#if UNITY_EDITOR
        // 保存weakReference的hash
        private List<int> bridgeLuaRefs = new List<int>();
        // 保存对应的lua stack info
        private List<string> bridgeLuaRefs2 = new List<string>();
        // 开关
        public bool debugDelegateBridgeRelease = false;
#endif

        public object CreateDelegateBridge(RealStatePtr L, Type delegateType, int idx)
        {
            WeakReference weakRef = null;
#if UNITY_EDITOR
            object obj;
            if (this.debugDelegateBridgeRelease)
            {
                string luaStack = this._GetLuaStack(L);
                obj = this._CreateDelegateBridge(L, delegateType, idx, out weakRef);
                if (weakRef != null)
                {
                    // bridgeLuaRefs.Add(weakRef.Target.GetHashCode());
                    bridgeLuaRefs.Add(weakRef.GetHashCode());
                    bridgeLuaRefs2.Add(luaStack);
                }
            }
            else
            {
                obj = this._CreateDelegateBridge(L, delegateType, idx, out weakRef);
            }
            return obj;
#else
            return this._CreateDelegateBridge(L, delegateType, idx, out weakRef);
#endif
        }

#if UNITY_EDITOR
        private string _GetLuaStack(RealStatePtr L)
        {
            var oldTop = LuaAPI.lua_gettop(L);

            // @see https://www.lua.org/pil/25.2.html
            int debug = LuaAPI.xlua_getglobal(L, "debug");
            // LuaAPI.xlua_pushasciistring(L, "getinfo");
            // LuaAPI.xlua_pgettable(L, -2); // 获取getinfo function
            // LuaAPI.xlua_pushinteger(L, 2); // 传入getinfo()第1个参数
            // LuaAPI.xlua_pushasciistring(L, "Sl"); // 传入getinfo()第2个参数
            // var strIndex = LuaAPI.lua_pcall(L, 2, 1, 0); // 执行函数,此时在栈上的结果是一个table
            // // @see https://www.lua.org/pil/25.1.html
            // LuaAPI.lua_pushstring(L, "source"); // 要获取一个table某个key下的值, 要先传入key的值='source'
            // LuaAPI.xlua_pgettable(L, -2); // 获取table[key]返回的值,此时是一个string
            // var luasource = LuaAPI.lua_tostring(L, -1);
            // LuaAPI.lua_pop(L, 1); // 弹出结果

            // LuaAPI.lua_pushstring(L, "currentline");
            // LuaAPI.xlua_pgettable(L, -2);
            // var currentline = LuaAPI.lua_tostring(L, -1);
            // LuaAPI.lua_pop(L, 1);
            // LuaAPI.lua_settop(L, oldTop);
            // string luaStack = luasource + ":" + currentline;

            // 修改为使用traceback()的信息
            LuaAPI.xlua_pushasciistring(L, "traceback");
            LuaAPI.xlua_pgettable(L, -2);
            var strIndex = LuaAPI.lua_pcall(L, 0, 1, 0);
            string luaStack = LuaAPI.lua_tostring(L, -1);
            LuaAPI.lua_pop(L, 1);
            LuaAPI.lua_settop(L, oldTop);

            return luaStack;
        }
#endif

        private object _CreateDelegateBridge(RealStatePtr L, Type delegateType, int idx, out WeakReference weakRef)
        {
            LuaAPI.lua_pushvalue(L, idx);
            LuaAPI.lua_rawget(L, LuaIndexes.LUA_REGISTRYINDEX);
            if (!LuaAPI.lua_isnil(L, -1))
            {
                int referenced = LuaAPI.xlua_tointeger(L, -1);
                LuaAPI.lua_pop(L, 1);

                weakRef = delegate_bridges[referenced];
                if (weakRef.IsAlive)
                {
                    if (delegateType == null)
                    {
                        return weakRef.Target;
                    }
                    DelegateBridgeBase exist_bridge = weakRef.Target as DelegateBridgeBase;
                    Delegate exist_delegate;
                    if (exist_bridge.TryGetDelegate(delegateType, out exist_delegate))
                    {
                        return exist_delegate;
                    }
                    else
                    {
                        exist_delegate = getDelegate(exist_bridge, delegateType);
                        exist_bridge.AddDelegate(delegateType, exist_delegate);
                        return exist_delegate;
                    }
                }
            }
            else
            {
                LuaAPI.lua_pop(L, 1);
            }

            LuaAPI.lua_pushvalue(L, idx);
            int reference = LuaAPI.luaL_ref(L);
            LuaAPI.lua_pushvalue(L, idx);
            LuaAPI.lua_pushnumber(L, reference);
            LuaAPI.lua_rawset(L, LuaIndexes.LUA_REGISTRYINDEX);
            DelegateBridgeBase bridge;
            try
            {
#if (UNITY_EDITOR || XLUA_GENERAL) && !NET_STANDARD_2_0
                if (!DelegateBridge.Gen_Flag)
                {
                    bridge = Activator.CreateInstance(delegate_birdge_type, new object[] { reference, luaEnv }) as DelegateBridgeBase;
                }
                else
#endif
                {
                    bridge = new DelegateBridge(reference, luaEnv);
                }
            }
            catch (Exception e)
            {
                LuaAPI.lua_pushvalue(L, idx);
                LuaAPI.lua_pushnil(L);
                LuaAPI.lua_rawset(L, LuaIndexes.LUA_REGISTRYINDEX);
                LuaAPI.lua_pushnil(L);
                LuaAPI.xlua_rawseti(L, LuaIndexes.LUA_REGISTRYINDEX, reference);
                throw e;
            }
            if (delegateType == null)
            {
                weakRef = new WeakReference(bridge);
                delegate_bridges[reference] = weakRef;
                return bridge;
            }
            try
            {
                var ret = getDelegate(bridge, delegateType);
                bridge.AddDelegate(delegateType, ret);
                weakRef = new WeakReference(bridge);
                delegate_bridges[reference] = weakRef;
                return ret;
            }
            catch (Exception e)
            {
                bridge.Dispose();
                throw e;
            }
        }

        public bool AllDelegateBridgeReleased()
        {
#if UNITY_EDITOR
            var _bridgeLuaRefs = this.bridgeLuaRefs;
            var _bridgeLuaRefs2 = this.bridgeLuaRefs2;
            this.bridgeLuaRefs = new List<int>();
            this.bridgeLuaRefs2 = new List<string>();
#endif
            foreach (var kv in delegate_bridges)
            {
                if (kv.Value.IsAlive)
                {
#if UNITY_EDITOR
                    if (this.debugDelegateBridgeRelease)
                    {
                        var hash = kv.Value.GetHashCode();
                        for (int i = 0; i < _bridgeLuaRefs.Count; i++)
                        {
                            if (_bridgeLuaRefs[i] == hash)
                            {
                                UnityEngine.Debug.Log("未释放lua引用: " + _bridgeLuaRefs2.Count + " : " + _bridgeLuaRefs2[i]);
                                break;
                            }
                        }
                    }
#endif
                    return false;
                }
            }
            return true;
        }

        public void ReleaseLuaBase(RealStatePtr L, int reference, bool is_delegate)
        {
            if (is_delegate)
            {
                LuaAPI.xlua_rawgeti(L, LuaIndexes.LUA_REGISTRYINDEX, reference);
                if (LuaAPI.lua_isnil(L, -1))
                {
                    LuaAPI.lua_pop(L, 1);
                }
                else
                {
                    LuaAPI.lua_pushvalue(L, -1);
                    LuaAPI.lua_rawget(L, LuaIndexes.LUA_REGISTRYINDEX);
                    if (LuaAPI.lua_type(L, -1) == LuaTypes.LUA_TNUMBER && LuaAPI.xlua_tointeger(L, -1) == reference) //
                    {
                        //UnityEngine.Debug.LogWarning("release delegate ref = " + luaReference);
                        LuaAPI.lua_pop(L, 1);// pop LUA_REGISTRYINDEX[func]
                        LuaAPI.lua_pushnil(L);
                        LuaAPI.lua_rawset(L, LuaIndexes.LUA_REGISTRYINDEX); // LUA_REGISTRYINDEX[func] = nil
                    }
                    else //another Delegate ref the function before the GC tick
                    {
                        LuaAPI.lua_pop(L, 2); // pop LUA_REGISTRYINDEX[func] & func
                    }
                }

                LuaAPI.lua_unref(L, reference);

                WeakReference weakRef = null;
                delegate_bridges.TryGetValue(reference, out weakRef);
                delegate_bridges.Remove(reference);

#if UNITY_EDITOR
                if (this.debugDelegateBridgeRelease && weakRef != null)
                {
                    // if (weakRef.Target != null){
                    // var hash = weakRef.Target.GetHashCode();
                    // }
                    var hash = weakRef.GetHashCode();
                    for (int i = 0; i < this.bridgeLuaRefs.Count; i++)
                    {
                        if (bridgeLuaRefs[i] == hash)
                        {
                            bridgeLuaRefs.RemoveAt(i);
                            bridgeLuaRefs2.RemoveAt(i);
                            break;
                        }
                    }
                }
#endif
            }
            else
            {
                LuaAPI.lua_unref(L, reference);
            }
        }

在使用var luaEnv = new LuaEnv()的地方设置luaEnv.translator.debugDelegateBridgeRelease = true, 打开开关,当进行luaEnv.Dispose()时,如果lua侧有未释放的引用,则会打印出lua的对应堆栈信息.

lua堆栈参考: https://blog.csdn.net/zhuzhuyule/article/details/41086745

转载请注明出处: https://www.jianshu.com/p/58d20d46560a

最后编辑于
©著作权归作者所有,转载或内容合作请联系作者
  • 序言:七十年代末,一起剥皮案震惊了整个滨河市,随后出现的几起案子,更是在滨河造成了极大的恐慌,老刑警刘岩,带你破解...
    沈念sama阅读 157,298评论 4 360
  • 序言:滨河连续发生了三起死亡事件,死亡现场离奇诡异,居然都是意外死亡,警方通过查阅死者的电脑和手机,发现死者居然都...
    沈念sama阅读 66,701评论 1 290
  • 文/潘晓璐 我一进店门,熙熙楼的掌柜王于贵愁眉苦脸地迎上来,“玉大人,你说我怎么就摊上这事。” “怎么了?”我有些...
    开封第一讲书人阅读 107,078评论 0 237
  • 文/不坏的土叔 我叫张陵,是天一观的道长。 经常有香客问我,道长,这世上最难降的妖魔是什么? 我笑而不...
    开封第一讲书人阅读 43,687评论 0 202
  • 正文 为了忘掉前任,我火速办了婚礼,结果婚礼上,老公的妹妹穿的比我还像新娘。我一直安慰自己,他们只是感情好,可当我...
    茶点故事阅读 52,018评论 3 286
  • 文/花漫 我一把揭开白布。 她就那样静静地躺着,像睡着了一般。 火红的嫁衣衬着肌肤如雪。 梳的纹丝不乱的头发上,一...
    开封第一讲书人阅读 40,410评论 1 211
  • 那天,我揣着相机与录音,去河边找鬼。 笑死,一个胖子当着我的面吹牛,可吹牛的内容都是我干的。 我是一名探鬼主播,决...
    沈念sama阅读 31,729评论 2 310
  • 文/苍兰香墨 我猛地睁开眼,长吁一口气:“原来是场噩梦啊……” “哼!你这毒妇竟也来了?” 一声冷哼从身侧响起,我...
    开封第一讲书人阅读 30,412评论 0 194
  • 序言:老挝万荣一对情侣失踪,失踪者是张志新(化名)和其女友刘颖,没想到半个月后,有当地人在树林里发现了一具尸体,经...
    沈念sama阅读 34,124评论 1 239
  • 正文 独居荒郊野岭守林人离奇死亡,尸身上长有42处带血的脓包…… 初始之章·张勋 以下内容为张勋视角 年9月15日...
    茶点故事阅读 30,379评论 2 242
  • 正文 我和宋清朗相恋三年,在试婚纱的时候发现自己被绿了。 大学时的朋友给我发了我未婚夫和他白月光在一起吃饭的照片。...
    茶点故事阅读 31,903评论 1 257
  • 序言:一个原本活蹦乱跳的男人离奇死亡,死状恐怖,灵堂内的尸体忽然破棺而出,到底是诈尸还是另有隐情,我是刑警宁泽,带...
    沈念sama阅读 28,268评论 2 251
  • 正文 年R本政府宣布,位于F岛的核电站,受9级特大地震影响,放射性物质发生泄漏。R本人自食恶果不足惜,却给世界环境...
    茶点故事阅读 32,894评论 3 233
  • 文/蒙蒙 一、第九天 我趴在偏房一处隐蔽的房顶上张望。 院中可真热闹,春花似锦、人声如沸。这庄子的主人今日做“春日...
    开封第一讲书人阅读 26,014评论 0 8
  • 文/苍兰香墨 我抬头看了看天上的太阳。三九已至,却和暖如春,着一层夹袄步出监牢的瞬间,已是汗流浃背。 一阵脚步声响...
    开封第一讲书人阅读 26,770评论 0 192
  • 我被黑心中介骗来泰国打工, 没想到刚下飞机就差点儿被人妖公主榨干…… 1. 我叫王不留,地道东北人。 一个月前我还...
    沈念sama阅读 35,435评论 2 269
  • 正文 我出身青楼,却偏偏与公主长得像,于是被迫代替她去往敌国和亲。 传闻我的和亲对象是个残疾皇子,可洞房花烛夜当晚...
    茶点故事阅读 35,312评论 2 260