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

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

代码如下：

 

 

 奔溃的地方如下

 

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

 

 

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

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

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