LightCoin合约非一致性检查漏洞分析

2018-07-15 1,285

本文作者莫良,由入驻 安全脉搏 的 BYSEC.IO 投稿。

BYSEC.IO是全球首家基于区块链生态的信息安全社区,致力于打造漏洞平台、安全媒体联盟、行业名片、项目评级等功能模块组成的全方位立体化区块链安全体系。


0x00 项目简述

LIGHT (光链) 是世界上第一个双层区块链(父链与子链)。其中,LIGHT 父链与传统的公链类似,有且仅有一个,保证记录完整、透明、不可被篡改或销毁。

LIGHT子链可以是多个且相互独立存在,基于PoM validation方式,结合In-Memory数据库缓存,有效提高缓存环节吞吐量,QPS可达10万以上,实现性能大幅提高。

LIGHT突破性地解决了当今数字货币在交易时处理速度严重滞后的问题,实现瞬时完成实时交易。

 

0x01 漏洞详情

合约地址:

0xd97579Cea3fE2473682a4C42648134BB982433B9

合约代码地址:

https://etherscan.io/address/0xd97579Cea3fE2473682a4C42648134BB982433B9#code

合约类型:ERC20https://github.com/ethereum/EIPs/issues/20

漏洞函数:transferFrom

漏洞类型:非一致性检查

漏洞危害:可导致绕过授权额度检查,转出超出授权额度代币至任意以太坊账户。

 

0x02 细节分析

 

image.png

 

代码55行在检查授权转账额度是没有问题的,但是在58行减掉已转出的授权额度时,错误的写成了allowed[_from][_to] -= _value

该行代码导致在授权转账完成后,限额依然不变,在下一次操作授权转账时,依然可以继续按照原本限额转出代币。该漏洞属于一个逻辑漏洞,由于校验的条件和操作的条件不一致,导致后续绕过校验。

 

该漏洞逻辑如图所示。

 

image.png

 

0x03 漏洞调试

 

我们使用Remix调试代码,调试的时候我们修改24行为本地Javascript VM的钱包地址,点Deloy部署后,和合约为合约部署者分配2000000Token

首先调用approve函数为Javascript VM的另外一个账户授权一点转账额度。调用参数如下。该调用为参数1地址授权10000额度的转账权限。

 

"0x14723a09acff6d2a60dcdf7aa4aff308fddc160c",10000

 

调用完毕后,我们切换到授权过的这个账户(既第一个参数),调用transferFrom函数进行授权转账测试。调用参数如下,该调用正确情况下,应该由参数1账户转向参数2账户9999Token。同时减除调用者授权额度9999

 

"0xca35b7d915458ef540ade6068dfe2f44e8fa733c","0x4b0897b0513fdc7c541b6d9d7e929c4e5364d2db",9999

 

由于remix不支持查看二级字典的调试功能,为了调试方便我们在59行后加一行var test = allowed[_from][msg.sender]便于我们观察变量的更新情况。调试结果如下图。我们可以看到,转账成功了,但是allowed[_from][msg.sender]的值却并没有变,正确情况下该值应该等于10000-9999

 

image.png

使用相同参数再次调用一下transferFrom,我们会发现,转账仍然会成功,调用结果如下。

 

image.png

 

0x04修复建议

 

58行改为:

allowed[_from][msg.sender] -= _value;


Tags:
评论  (1)
快来写下你的想法吧!

BYSEC.IO

文章数:3 积分: 28

安全问答社区

安全问答社区

脉搏官方公众号

脉搏公众号