如今，随着人们对DeFi的兴趣日益浓厚，DEX风靡一时。它们解决了常见的CEX问题，那我们也会问，DEX够安全吗？
在上一篇说明了代币本身的安全问题后|DEFI安全问题之基础篇），现在来聊聊DEX在兑换代币时可能产生的安全问题。目前DEX主要面临的安全问题大致可分成两类：
DEX项目本身存在的安全问题。
作为第三方协议，与其他DEFI项目交互时产生的安全问题。
本文将对第一类安全问题进行介绍。
Part.1
-DecentralizedExchange
重入漏洞
重入漏洞在上一篇我们也提到过，它属于需要防范的经典漏洞。与普通代币的重入相比，Uniswap的重入漏洞的主要表现形式为：攻击者在一笔兑换交易中利用Uniswap未及时更新价格前发起二次兑换，由于此时Uniswap未更新价格，使得二次兑换可兑出的代币数量比正常兑换的多。此外，在Uniswap的重入攻击中，攻击者利用单笔交易可能只能获得微小的收益，因此攻击者往往倾向于使用闪电贷或者循环套利扩大战果。
音乐NFT平台Sound.xyz在Optimism上启动:6月13日消息，音乐NFT平台Sound.xyz宣布与Optimism Collective达成合作关系，并在Optimism上启动其平台，允许艺术家在Optimism上传音乐。Sound.xyz还推出了一种使用主网ETH进行支付并将音乐NFT发送到Optimism上钱包的方式。[2023/6/13 21:34:01]
以imBTC攻击事件为例，该事件是由于UniswapV1在调用ERC777系列代币时，未充分考虑合约回调的情况。
具体表现为：攻击者使用imBTC代币兑换ETH时，合约先通过self.getInputPrice函数计算正确的ETH数额并将ETH发送到目标地址，然后调用self.token.transferFrom函数时，会调用imBTC合约的_callTokensToSend函数，而_callTokensToSend函数会调用用户指定存储imBTC代币的合约。因此，如果攻击者部署存储合约，并改写其中TokensToSend函数，那么当兑换代币时，pair(两种代币组成的交易对)合约调用攻击者部署的存储合约，就可以回调pair进行二次兑换，而二次兑换时pair合约账本还未更新，使得计算的ETH数额比正常兑换要多，以此来获利。
Peter Schiff：ChatGPT不建议将资金分配给比特币，证明人工智能还是相当有智慧的:金色财经报道，欧洲太平洋资本（Euro Pacific Capital）CEO兼首席全球策略师Peter Schiff评论称，ChatGPT不建议将资金分配给比特币，证明人工智能还是相当有智慧的。不过，另有媒体报道称，ChatGPT的回答并不意味着反对加密货币。当ChatGPT被问及对“黄金和比特币”的看法时，它表示选择哪种资产取决于购买者的投资目标。
ChatGPT解释称，黄金是一种有形资产，开采难度大且成本高，使其具有一定程度的稀缺性和内在价值，在经济不确定时期被视为避险投资；而比特币没有实物资产，也不受政府的支持，常被许多人认为是一种投机性投资。
它总结道，对于那些寻求稳定、长期投资的人来说，黄金可能是更好的选择，而比特币可能更适合那些寻求高风险、高回报的人。[2023/4/10 13:53:48]
图1Uniswap的tokenToEthInput函数
图2imBTC的transferFrom函数
图3imBTC的__callTokensToSend函数
纽约金融服务部：接管Signature Bank与其加密业务无关:金色财经报道，纽约金融服务部（NYDFS）发言人表示，周日做出接管Signature Bank的决定与该银行的加密货币业务无关。此前美国前众议员Barney Frank批评了监管机构的决定，声称他们“想要传递加密货币有害的信息”。NYDFS一位发言人在一份声明中表示，接管该银行并将其移交给FDIC的决定是基于该银行目前的状况以及它是否有能力以安全和健康的方式开展业务。该发言人表示，他们正在与Signature Bank的高管合作，评估其财务状况、满足取款请求的能力以及是否能够继续正常运营。该银行没有提供“可靠和一致的数据，造成了对该行领导层的严重信任危机”。NYDFS表示，Signature是一家拥有更重要业务的银行，数字资产业务仅占该银行整体业务的一小部分。（The Block）[2023/3/15 13:04:29]
详细攻击流程如下：
图4ETH-imBTC事件流程图
那么，为什么在第二次调用tokenToEthSwapInput函数兑换代币时，发送的ETH会比正常兑换要多呢?我们可以用公式来还原可兑换代币数量的代码逻辑：
比特大陆市场总监：将与Tether探索更多合作机会:11月11日消息，比特大陆坎昆WDMS Global 2022大会正在隆重召开。会上，Tether CTO Paolo就Stablecoin的重要意义发布演讲。比特大陆市场总监Xmei Lin也在会上宣布，比特大陆正致力于PoW生态和社区发展，包括ETC、LTC等，并正在探索与Tether的更多合作机会。[2022/11/11 12:46:40]
首先，正常兑换下，getInputPrice函数计算可兑换的ETH数量为:
且正常第二次可兑换的ETH数量为：
但重入后第二次可兑换的ETH数量为：
由此可知，在重入后第二次兑换中只有ETH的储备量减少，而imBTC储备量未增加。这样在分母不增加的情况下，导致了等量的imBTC可以兑换更多的ETH。
针对此类安全问题，成都链安建议：
当合约涉及到资产转移时，使用“检查-生效-交互”模式来处理逻辑，对关键的业务操作可以使用OpenZeppelin官方的ReentrancyGuard进行修饰。
Part.2
数据：已有两家以太坊验证节点运营商承诺抗审查，占已质押ETH比例38.34%:8月22日消息，Yearn核心开发者banteg在其社交平台表示，目前已有两家以太坊验证节点运营商承诺不会进行审查，分别是Lido和Coinbase。针对Lido是否足够去中心化的质疑，banteg表示，Lido验证节点由30个不同组织运营。
另据Nansen数据显示，这两家运营商控制38.34%的已质押ETH，其中Lido控制了31.1%的质押ETH。[2022/8/22 12:39:53]
-DecentralizedExchange
swap函数未对K值进行校验
Uniswap的核心是常量乘积模型K=x*y，其中的K值是该pair合约持有代币数量的乘积，且要求之后的每一笔交易完成后K值必须增加(考虑手续费)。因此如果不进行K值校验，将很容易成为攻击点。
图5Uniswap的价格波动
以ImpossibleFinance事件为例，该项目是Uniswap的仿盘，实现了两种兑换代币的函数：cheapSwap和swap。其中cheapSwap函数少了k值校验，但是项目方知道缺少K值校验的后果，专门为cheapSwap函数增加了onlyIFRouter做修饰，来限制cheapSwap函数只能被指定的Router合约调用。
图6合约未检查k值的cheapSwap函数
正常情况下，当用户使用Router合约兑换代币时，首先会使用getAmountsOut函数来计算正确的代币数量amounts；然后调用safeTransferFrom将用户的兑换消耗代币转入目标pair合约；最后，通过内部调用_swap函数来执行cheapSwap函数将兑换代币转至目标地址。
图7Router01合约的swapExactTokensForTokens函数
但是，由于cheapSwap函数缺少了K值检验，如果攻击者部署恶意代币合约，在Router合约调用safeTransferFrom函数时，回调正常的pair合约进行同种兑换，由于，回调后的兑换使用的amounts仍是未更新之前的数据，已不符合改变账本状态之后的校验，那么攻击会导致以错误的价格兑换出目标代币，以此获利。
图8合约进行k值校验的Swap函数
该事件的具体攻击步骤如下：
1.在准备阶段攻击者部署了AAA代币合约，并使用闪电贷借来1000WBNB，兑换65140个项目方的IF代币。
2.使用其中一半的IF代币(32570个)与攻击者自己部署的AAA代币构建IF-AAA交易池。
3.执行AAA-IF-BUSD路径的代币兑换，且当Router合约调用AAA代币合约的transferFrom函数时会执行攻击者的恶意代码，重入至IF-BUSD的pair合约，并将另一半IF代币正常兑换出221897个BUSD。
4.回归到AAA-IF-BUSD路径的兑换，将之前计算的amounts值传入_Swap函数中执行这笔兑换，用一半的IF又兑换了2521897个BUSBD。
5.归还闪电贷，完成攻击。
图9事件流程图
针对此类安全问题，成都链安建议：
在关键的兑换函数中必须做k值校验，不要为了节省gas和代码量就将K值校验和安全验证依赖外部验证，做到自身功能完善。
Part.3
-DecentralizedExchange
通缩代币未设置pair为分红例外
通缩代币在交易时会产生额外的分红与手续费。如果交易合约中包含了此类代币，且没有进行特殊处理，那么，就可能导致交易对合约记录的代币储量与实际的代币可用余额不一致。
以XSquid事件为例，XSquid是一种通缩代币，未将其与WHT代币组成的pair合约地址添加奖励例外列表，造成了pair合约除了正常代币兑换和流动性存储外，还存有多余的XSquid分红奖励代币。因此，攻击者就可以调用Swap函数将pair合约多余的XSquid代币转换为WHT提取，或者通过skim函数将多余的XSquid代币直接提取。
图10XSquid交易对合约未添加奖励例外
图11Swap函数可以兑换多余的WHT代币
图12skim函数可以提取大于reserve的部分
针对此类安全问题，成都链安建议：
DEX在添加通缩分红型代币时多注意手续费以及分红的处理情况。在创建通缩分红型代币交易对时，可以添加奖励例外来避免此类代币的分红问题。
此外，以下两类不属于DEX本身的安全问题，但是被项目方借助了DEX的特性实施，所以将其写在文章末尾。
PART.1
交易池
这类问题主要是指项目方在自己发行的代币里留有后门，创建与主流代币的交易池，诱使投资者使用手里存在价值的代币买入项目方代币，并且不断拉盘对投资者进行投资。
以下面的TRTC项目方为例，项目方创建了ETH-TRTC的交易池。但是在TRTC的代币合约对transferFrom函数做了相关限制，要求代币的转出方为owner(管理员)或者为Uniswap。因此对于投资者，仅可以通过Uniswap买入TRTC代币，而不能卖出TRTC代币。最后由项目方把投资者投入的ETH提走跑路，给投资者带来了巨大的损失。
图13TRTC合约的transfer函数
图14TRTC合约的ensure修饰
图15TRTC合约的transferFrom函数
PART.2
项目方RugPull
RugPull是指项目方卷走投资者资金跑路的行为，目前已成为DeFi生态系统的最大局类型，项目方刻意制造代币价格暴涨的假象、许诺为提供流动性的投资者提供高回报等方式来大量聚集资金，一旦时机成熟就移除池子里的流动性或将代币卷走。这样的例子在DeFi屡见不鲜，AnubisDAO、MeerkatFinance、TurtleDEX、Squidtoken鱿鱼币等都是在卷款跑路之后，注销网站和社交媒体销声匿迹，导致投资者承担了巨大的损失。
写在最后
成都链安建议项目方使用锁仓和多重签名来控制代币流动性，避免出现砸盘跑路的情况。投资者不要被天上掉馅饼的事情冲昏头脑，防范虚假宣传。
郑重声明： 本文版权归原作者所有， 转载文章仅为传播更多信息之目的， 如作者信息标记有误， 请第一时间联系我们修改或删除， 多谢。