代码review还是需要再仔细点-一次crash异常分析的总结

版本发布之后,外网新增了一些crash, 从mini dump看,的确是有异常的,619行crash了。

代码如下:

 奔溃的地方如下

 可以看到是Zip add的时候指针空了, 为什么呢?

 

结合代码,可以发现这个地方算便宜是有问题的,万一length 小于size呢?、、

修复方案就不另说了,其实这个问题我一开始从dump 没看出来,还是实际该代码重现才发现的。

最终找到问题了再来看,经验就是: windbg不会骗人,如果有源代码,就多review吧。不是每次都有条件重现的。

猜你喜欢

转载自www.cnblogs.com/khacker/p/11508047.html