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

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

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

代码如下:

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

 

 

 奔溃的地方如下

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

 

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

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

 

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

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

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

 

上一篇:php – 如何在WordPress / WooCommerce 3中的注释表单中添加自定义字段


下一篇:PHP – Magento管理员屏幕添加新的审查按钮不起作用,并生成404错误