记一次支付异步校验逻辑漏洞,之前支付出现问题很长时间了,最后通过日志发现是最后核对金额时出现了问题。
举例,当用户使用微信支付1笔商品价格为19.9元的订单时,微信支付平台异步回调时会得到具体的用户支付金额。
如1990(以分为单位,即19.90元)。
校验逻辑
<?php
$notify_money = 1990;//微信服务器返回的金额
$db_money = 19.90;//数据库存储的订单金额
if(intval($notify_money) === intval($db_money*100)){
echo 'SUCCESS';
}else{
echo 'FAIL';
}
这个逻辑,第一眼看起来可能没什么问题,在实际的使用情况下,有的订单则会通过(SUCCESS),有的则可能会失败(FAIL)。
经过测试,发现这两个值的intval结果是完全不同的。
最后查阅了一番发现这是浮点数处理不当引起的问题。
http://php.net/manual/en/function.intval.php#60793
http://www.laruence.com/2013/03/26/2884.html
拜读了鸟哥的讲解之后,心里终于明白是怎么回事了。
最终的解决办法是,换做round或者先将浮点转换到字符串再使用intval进行即可。
修改后的一个逻辑
<?php
$notify_money = 1990;//微信服务器返回的金额
$db_money = 19.90;//数据库存储的订单金额
if(round($notify_money) === round($db_money*100)){
echo 'SUCCESS';
}else{
echo 'FAIL';
}
//或
if(intval($notify_money) === intval((string)($db_money*100))){
echo 'SUCCESS';
}else{
echo 'FAIL';
}
这个问题可能不只仅限于微信支付或者其他支付都有可能遇到,往往就在于多个业务的金额单位不一致导致的。