记一个 BUG 的排查过程:一个类里有一个成员方法,用着用着, GC 把这个方法杀了,类实例还活着

2019-06-18 12:48:00 +08:00
 yulitian888

挺有意思的一个 bug,症状是运行了一小段时间之后,程序进程直接死掉,哪怕所有代码都用了 try...catch 都管不住。

故障发生的原因在于, [直接] 使用了实例方法当作非托管函数的回调函数参数使用。 类代码如下: public class SnapInfo { internal int CallBack_KaKou(IntPtr lAnalyzerHandle, uint dwEventType, IntPtr pEventInfo, IntPtr pBuffer, uint dwBufSize, IntPtr dwUser, int nSequence, IntPtr reserved) { ...... }

}

调用大华卡口摄像头抓拍照片并识别车牌的时候用的,调用 SDK 代码如下: NETClient.RealLoadPicture(loginId, ChannelNum, (uint)EM_EVENT_IVS_TYPE.TRAFFIC_MANUALSNAP, true, currentSnap.CallBack_KaKou, loginId, IntPtr.Zero);

其中 currentSnap 是 SnapInfo 类的实例。 执行一段时间之后,呵呵呵~~~

解决办法:

public class SnapInfo {

public SnapInfo(CameraDevice cameraDevice) { this.卡口回调 = new fAnalyzerDataCallBack(this.CallBack_KaKou); }

public fAnalyzerDataCallBack 卡口回调 { get; private set; }

internal int CallBack_KaKou(IntPtr lAnalyzerHandle, uint dwEventType, IntPtr pEventInfo, IntPtr pBuffer, uint dwBufSize, IntPtr dwUser, int nSequence, IntPtr reserved) { ...... }

}

把成员方法包装成成员属性,对非托管方法传参的时候替换为属性“卡口回调”,问题解决。

挺有趣的一个排错过程,记一下

8010 次点击
所在节点    C#
4 条回复
geelaw
2019-06-18 12:58:09 +08:00
这并不会让你的代码正确。

真正的原因是非托管代码和 GC 之间互相不认识,所以你需要 GC.KeepAlive(currentSnap) 或其他做得 currentSnap 仍然可达的方式来确保 currentSnap 在成员方法没有被取消监听之前不会被回收。
yulitian888
2019-06-18 13:21:18 +08:00
@geelaw GC.KeepAlive(currentSnap) 和 GC.KeepAlive(currentSnap.CallBack_KaKou) 都试过了,修改成属性才是最后生效的方式。
实际上,使用 KeepAlive 甚至都没有能够做到延缓进程死亡时间的效果。
当委托作为成员属性存在的时候,只要实例不被回收,GC 是不会对单独的属性下黑手的。而 GC 显然是在形参实参传递的过程中误判了委托的引用关系才会杀掉方法的引用。
geelaw
2019-06-18 14:45:23 +08:00
@yulitian888 #2 你需要的是

DelegateType dlg = new DelegateType(currentSnap.CallBack_KaKou);
ExternalObject.SetCallback(dlg);
...
GC.KeepAlive(dlg);

因为是委托对象被咔擦了。把委托对象作为属性是一种不符合对象本身表达意思的方式。
yulitian888
2019-06-18 16:01:29 +08:00
@geelaw 然而你说的这种写法并没有生效,这也是我第一时间想到的写法,然并卵

这是一个专为移动设备优化的页面(即为了让你能够在 Google 搜索结果里秒开这个页面),如果你希望参与 V2EX 社区的讨论,你可以继续到 V2EX 上打开本讨论主题的完整版本。

https://www.v2ex.com/t/575061

V2EX 是创意工作者们的社区,是一个分享自己正在做的有趣事物、交流想法,可以遇见新朋友甚至新机会的地方。

V2EX is a community of developers, designers and creative people.

© 2021 V2EX