首页 币圈新闻 Basdfsncor安全事件分析:简单漏洞引发50万美元损失,代码审计或存问题

Basdfsncor安全事件分析:简单漏洞引发50万美元损失,代码审计或存问题

原文标题:《成都链安:Bancor Network 漏洞事件分析》

原文来源:成都链安

一、事件简述

2020-06-18(文中所提及时间均为 UTC 时间),以太坊上的智能合约 Bancor Network 被爆出存在严重漏洞。该漏洞由 Bancor Network 团队和白帽最先发现,并第一时间对存在被盗风险的资金进行了转移,涉及资金 50W 余美元。

此次事件中,存在漏洞的合约地址主要有如下三个:

0x923cab01e6a4639664aa64b76396eec0ea7d3a5f

0x8dfeb86c7c962577ded19ab2050ac78654fea9f7

0x5f58058c0ec971492166763c8c22632b583f667f

2020-06-18 3:06:48,Bancor Network 团队利用此漏洞对存在被盗风险的用户资金进行转移,截至 2020-06-18 6:56,Bancor Network 团队共计转移资金约合409,656。

另外两个公开邮箱信息的地址:

0x052ede4c2a04670be329db369c4563283391a3ea

0x1ad1099487b11879e6116ca1ceee486d1efa7b00

也于同一时间利用此漏洞对用户资金进行转移,分别转出资金131,889 和2346。

到目前为止(2020-06-19):

EtherScan 已经将存在此漏洞的合约进行标注,如下图所示

Bancor Network 团队也已对此次事件做出了回应,

详情见(https://blog.bancor.network/bancors-response-to-today-s-smart-contract-vulnerability-dc888c589fe4),并于 UTC 时间 2020-06-17 21:35:53 部署了新的 Bancor Network 合约,合约地址为0x2f9ec37d6ccfff1cab21733bdadede11c823ccb0

二、原理分析

漏洞爆出后,成都链安安全团队第一时间对本次事件进行跟踪分析,根据链上分析结果发现,此次事件中漏洞产生的主要原因是智能合约 Bancor Network 存在一个调用权限为 public 的函数 safeTransferFrom,通过调用此函数,可以将用户授权给智能合约 Bancor Network 的资金转出到任意地址。

由源码可以知,该函数为一个 public 函数,详细代码如下图所示:

safeTransferFrom 函数内部调用了 execute 函数。而 execute 函数的功能是调用 safeTransferFrom 参数中的_token 代币智能合约的 transferFrom 函数进行代理转账。execute 函数源码如下图所示:

我们通过一笔链上交易可以清晰的看到调用此函数的具体作用。如地址「0xc802」发起一笔交易,调用了智能合约「0x5f58」(也就是 Bancor Network)的 safeTransferFrom 函数,使地址」0x8a39」中的 0.000000000000000003 代币 MFG 发送到」0x2955」(图中的「Bancor:White Hat S…),具体如下图所示:

以上就是本次事件漏洞的原理分析,该漏洞的原理十分简单,是因为函数的调用权限设置错误,将本应该只有合约本身调用的函数,设置成了任何人都可以调用。而当 Bancor Network 拥有用户的授权额度时,任何人都可以通过调用 safeTransferFrom 函数,以 Bancor Network 的身份对用户的资金进行代理转账。

针对该漏洞,有两点值得我们进行深入分析:

1. 为什么 Bancor Network 合约的 safeTransferFrom 函数会将权限设置成 public。

2. Bancor Network 合约的主要功能是代币转换,并非一个钱包,为什么用户会有未使用的授权。

对于第一个问题,我们追踪了 safeTransferFrom 函数的调用情况,发现 safeTransferFrom 函数只在 handleSourceToken 函数中被调用,用于将用户的代币转移到本合约中。handleSourceToken 函数源码如下图所示:

既然只是在 handleSourceToken 函数中调用,那么显然是没有必要设置成 public 权限,且根据业务来看,Bancor Network 的主要作用是用于代币交换,并不需要用户主动调用 safeTransferFrom,也就没有必要将 safetransferFrom 设置为 public 权限。于是我们对比了合约中其他的几个函数,这几个函数均被设置成了 public 权限。如下图所示:

由此,针对第一个问题,我们推测将这些函数设置为 public 权限可能是因为合约开发人员的疏忽。而目前来看(2020-06-19),Bancor Network 团队新部署的合约也验证了这个推测。根据链上代码显示,官方部署的新合约已经将相关函数权限全部进行了更改(虽然其他的函数与此事件无关)。具体代码如下图所示:

针对第二个问题,我们跟踪了漏洞爆出后被转移资金的地址,发现这些地址在对 Bancor Network 授权额度时,往往超过了所需要兑换代币的数量,且在兑换完成后,并没有收回额度。如下图所示:

「0x624f」开头的地址对 Bancor Network 合约授权了 900000000000000 个 ONG 代币,此额度已经超过了 ONG 的发行量,相当于将「0x624f」所拥有的 ONG 代币全部授权给 Bancor Network 合约。而跟踪「0x624f」开头地址的交易我们发现,在进行完代币兑换后,未使用完所有授权额度,但并未将授权额度收回。

综上所述,我们推测可能是用户考虑到要随时使用,为了方便,一次性将所有代币都授权给了 Bancor Network 合约。也可能是 Bancor Network 的前端对授权金额设定了默认值。但这种行为存在巨大的风险,一旦智能合约爆发漏洞,资金极易被攻击者窃取。而在对后续新的 Bancor Network 合约进行分析时发现,Bancor Network 团队和用户似乎也意识到了此问题存在的巨大风险,并做出了相应的安全防护。如下图九、十为用户对新合约的授权情况,对比两笔线上交易不难看出,两次授权的时间间隔很短,当用户成功兑换完代币后,剩余授权即被收回。

三、结语

Bancor Network 本次爆出的漏洞,是一个较为简单的漏洞,此类漏洞在代码审计过程中十分容易被发现,但此次漏洞的影响却很大,上线仅两天,涉及的资金就已经超过 50w 美元。幸而 Bancor Network 团队及时发现并修复了漏洞,否则后果不堪设想。成都链安-安全研究团队在此提醒各大智能合约运营商,因区块链合约部署后不可更改的特性,对比传统软件,上线前做好充分的代码审计十分重要,此次事件虽未造成太大的经济损失,但势必会让用户对 Bancor Network 团队产品的安全性产生质疑。

另外也提醒广大用户和运营商,在授权资金给第三方合约时,都应保持谨慎,切不可盲目相信「大公司」,对任何合约的授权都应当秉承最小原则,不要使自己的资金安全掌握在他人手中。

关于作者: szhbsd

热门文章