▌背景
这是让项目归零的那笔交易:
https://etherscan.io/tx/0xad89ff16fd1ebe3a0a7cf4ed282302c06626c1af33221ebe0d3a470aba4a660f
我们可以看到执行的方法是 batchTransfer。
那这个方法是干嘛的呢?(给指定的几个地址,发送相同数量的代币)。
▌整体逻辑是
你传几个地址给我(receivers),然后再传给我你要给每个人多少代币(value);
然后你要发送的总金额 = 发送的人数* 发送的金额;
然后 要求你当前的余额大于 发送的总金额;
然后扣掉你发送的总金额;
然后 给 receivers 里面的每个人发送 指定的金额(value)。
从逻辑上看,这边是没有任何问题的,你想给别人发送代币,那么你本身的余额一定要大于发送的总金额的!
但是这段代码却犯了一个很傻的错!
▌代码解释
这个方法会传入两个参数:
_receivers
_value
_receivers 的值是个列表,里面有两个地址:
0x0e823ffe018727585eaf5bc769fa80472f76c3d7
0xb4d30cac5124b46c2df0cf3e3e1be05f42119033
_value 的值是:
8000000000000000000000000000000000000000000000000000000000000000
我们再查看代码(如下图):
我们一行一行地来解释:
uintcnt = _receivers.length;
是获取 _receivers 里面有几个地址,我们从上面可以看到 参数里面只有两个地址,所以 cnt=2,也就是 给两个地址发送代币。
uint256amount = uint256(cnt) * _value;
▌uint256
首先 uint256(cnt) 是把 cnt 转成了 uint256 类型。那么,什么是 uint256 类型?或者说 uint256 类型的取值范围是多少?
uintx 类型的取值范围是 0 到 2 的 x 次方 -1。也就是,假如是 uint8 的话,则 uint8 的取值范围是 0 到 2 的 8 次方 -1,即 0 到 255。
那么,uint256 的取值范围是:
0 - 2 的 256 次方 -1 ,也就是 0 到 115792089237316195423570985008687907853269984665640564039457584007913129639935
Python 算 2 的 256 次方是多少?
那么假如说 设置的值超过了 取值范围怎么办?这种情况称为“溢出”。
举个例子来说明:
因为 uint256 的取值太大了,所以用 uint8 来 举例。
从上面我们已经知道了 uint8 最小是 0,最大是 255。
那么当我 255 + 1 的时候,结果是啥呢?结果会变成 0。
那么当我 255 + 2 的时候,结果是啥呢?结果会变成 1。
那么当我 0 - 1 的时候,结果是啥呢?结果会变成 255。
那么当我 0 - 2 的时候,结果是啥呢?结果会变成 254。
那么,我们回到上面的代码中:
amount= uint256(cnt) * _value
则 amount = 2* _value。
但是此时 _value 是 16 进制的,我们把它转成 10 进制:
(Python 16 进制转 10 进制)
可以看到 _value = 57896044618658097711785492504343953926634992332820282019728792003956564819968
那么 amount = _value*2 = 115792089237316195423570985008687907853269984665640564039457584007913129639936
可以在查看上面看到 uint256 取值范围最大为 115792089237316195423570985008687907853269984665640564039457584007913129639935
此时,amout 已经超过了最大值,溢出 则 amount = 0
下一行代码 require(cnt > 0 && cnt <= 20); require 语句是表示该语句一定要是正确的,也就是 cnt 必须大于 0 且 小于等于 20
我们的 cnt 等于 2,通过!
require(_value >0&& balances[msg.sender] >= amount);
这句要求 value 大于 0,我们的 value 是大于 0 的 且,当前用户拥有的代币余额大于等于 amount,因为 amount 等于 0,所以 就算你一个代币没有,也是满足的!
balances[msg.sender] = balances[msg.sender].sub(amount);
这句是当前用户的余额 - amount
当前 amount 是 0,所以当前用户代币的余额没有变动。
for(uinti =0; i < cnt; i++) {
balances[_receivers[i]] = balances[_receivers[i]].add(_value);
Transfer(msg.sender, _receivers[i], _value);
}
这句是遍历 _receivers 中的地址, 对每个地址做以下操作:
balances[_receivers[i]] = balances[_receivers[i]].add(_value); _receivers 中的地址 的余额 = 原本余额+value
所以 _receivers 中地址的余额 则加了 57896044618658097711785492504343953926634992332820282019728792003956564819968 个代币!!!
Transfer(msg.sender, _receivers[i], _value); } 这句则只是把赠送代币的记录存下来!!!
▌总结
就一个简单的溢出漏洞,导致 BEC 代币的市值接近归 0。
那么,开发者有没有考虑到溢出问题呢?
其实他考虑了,可以看如下的截图:
除了 amount 的计算外, 其他的给用户转钱都用了 safeMath 的方法(sub,add)。
那么,为啥就偏偏这一句没有用 safeMath 的方法呢。。。
这就要问写代码的人了。。。
▌啥是 safeMath
safeMath 是为了计算安全 而写的一个 library。
我们看看它干了啥?为啥能保证计算安全。
functionmul(uint256 a, uint256 b)internalconstantreturns(uint256){
uint256 c = a * b;
assert(a ==0|| c / a == b);
returnc;
}
如上面的乘法。他在计算后,用 assert 验证了下结果是否正确!
如果在上面计算 amount 的时候,用了 mul 的话, 则 c / a == b 也就是 验证 amount / cnt == _value。
这句会执行报错的,因为 0 / cnt 不等于 _value。
所以程序会报错!
也就不会发生溢出了...
那么,还有一个小问题,这里的 assert 好 require 好像是干的同一件事 —— 都是为了验证某条语句是否正确!
那么它俩有啥区别呢?
用了 assert 的话,则程序的 gas limit 会消耗完毕;
而 require 的话,则只是消耗掉当前执行的 gas。
▌总结
只要涉及到计算,一定要用 safeMath