KingDefi收益计算逻辑漏洞分析

漏洞原因

近日，据业内人士提供的有关信息，名为KingDefi的项目合约存在漏洞，并提示其他用户谨慎操作，提取资金并取消授权。知道创宇区块链安全实验室调研发现，KingDeFi是一个DeFi项目，主要功能包含对BSC、Solana链上DeFi的收益聚合分析、用户DeFi收益追踪以及项目原生代币的抵押挖矿。

在查看BSC链上的KrownMaster合约源码后发现，该合约确实存在逻辑漏洞，会导致用户收益率受到影响，在相应的计算逻辑存在疏漏，以下为详细解释。合约链上地址如下：https://bscscan.com/address/0x56a65a3736e65349e5b0737cb2c5eb7d5ccbbbe3#code如下图所示，我们注意到在项目用户奖励更新算法逻辑的处理过程中存在对investor数组的一个遍历，此处investor地址存在被重复遍历并且修改对应奖励的可能性。

全战三国Three Kingdoms上线涨幅稳定10倍:据官方消息，MDIS生态第二个项目Total War: Three Kingdoms 全战三国已开启流动性挖矿，目前MDIS-TWTK MDIS 流动性挖矿年化达10000%，且TWTK币价距开盘价格已经稳定在10倍，全战三国将在9点开启派系出征挖矿，用户可以解锁派系参与挖矿。

Total War: Three Kingdoms 是MDIS开发的继聚合交易MiaoO后的第二个生态应用，MDIS致力于在HECO上建设全生态去中心化交易所，打造包括聚合交易、NFT、借贷、预言机、Unicrypt在内的多个基础性生态应用。[2021/4/6 19:51:21]

动态 | Kik CEO称有团队有意购买Kik并欲将其与Kin项目整合:Kik应用程序可能并没有死，正如该公司的首席执行官Ted Livingston最近宣布的那样——Livingston称，有一个团队有兴趣购买这款应用并继续开发，同时欲将其与Kin项目整合。 Kik聊天应用的消失对KIN的ICO来说是最后的打击，因为它依赖于该旗舰产品。没有Kik，这个项目就只剩下KIN代币，目前其价格徘徊在最低点。但KIN代币目前仍在使用，根据Blocktivity数据，KIN实际上每天都有交易。KIN生态系统每天处理超过100万笔微型APP和奖励交易。据此前消息，Kik曾宣布将关闭应用程序，并裁员至19人。（Cryptovest）[2019/10/8]

如下图所示，用户在通过deposit调用进行抵押的时候，判断当用户抵押数量为0时，可作为investor地址加入投资收益列表从而获得抵押收益，而该判断可被黑客利用。

声音 | Sunny King：美元稳定币不会取代比特币:PoS共识机制发明人、被V神称为“唯一一个最具原创精神的数字货币开发者”的Sunny King今日在“王峰十问”表示，与美元锚定的稳定币不会取代稳定币，法币固然有其地位，但比特币打开的是个人财产世界。[2018/9/21]

如下图所示，黑客可通过调用withdraw或者withdrawAll函数将指定pid池子中的抵押数量提现，从而使得user.amount为0，进而该地址可以在再次deposit抵押的时候通过相应检查进入investor列表，从而在updatePool函数中对黑客investor地址进行重复遍历并且增加多次抵押奖励，使得抵押奖励分配不均，影响到其他用户的抵押挖矿收益。

动态 | Kinit推出IOS版:据btcmanager消息，近日软件商KiK宣布旗下的消息应用软件Kinit已推出IOS版本。Kinit是一款消息传递软件，它允许用户通过使用其服务来赚取数字货币。[2018/8/25]

通过查看项目github发现，KingDefi项目方当前已对该问题进行了修改。漏洞修复

那么项目方如何修复该漏洞？查看项目的github地址(https://github.com/kingdefi/Krown-Contracts/tree/main/Farm)，发现其在18个小时前曾更新过代码，对比一下更新代码。

发现项目方已经删除了用于存储用户地址的数组，改为了rewardsPerShare变量，该变量表示单位抵押代币所对应的奖励代币；同时项目方也更改了奖励的计算方式(updatePool函数)：由原来循环所有用户地址来按比例分配奖励改为更新rewardsPerShare变量来计算用户奖励代币。

对比两种奖励方式，后者已经不会产生前者因为重复计算奖励的问题，这种奖励方式类似于sushiswap的奖励计算方式，同时也避免了前者因为循环次数太多导致的gas销毁过大的问题。漏洞总结

Kingdefi这次的漏洞影响到的是用户的奖励代币数量，攻击者可不断抵押提取来提高自身奖励的分配数量，但是用户的抵押代币是不受任何影响，可以正确安全提取出来。从项目方的修复结果来看，其换了一种常规奖励计算方式，该方式符合抵押挖矿逻辑，用户可正常且正确提取抵押和奖励代币。在此提醒广大项目方，在上线Defi挖矿项目前一定要做好代码审计，不同的计算方式在吸引新用户的同时也会大大增加犯错的风险！i

标签：KININGKINGEFIDeFi Kingdombiking音标King ArthurValuedefi vSWAP

莱特币热门资讯