{"content":{"title":"智能合约审计-重入漏洞","body":"# 一、什么是重入攻击\r\n重入攻击是 Solidity 智能合约中最具破坏性的攻击之一。当一个函数对另一个不受信任的合约进行外部调用时，就会发生重入攻击。然后，不受信任的合约递归回调原始函数，试图耗尽资金。\r\n\r\n当合约在发送资金之前未能更新其状态时，攻击者可以不断调用 withdraw 函数来耗尽合约的资金。现实世界中一个著名的重入攻击是造成 6000 万美元损失的 DAO 攻击。\r\n\r\n## 以太币转移和回退功能\r\n\r\n以太坊智能合约中重入攻击的可能性来自以太坊上处理价值转移的方式。以太坊对用户和智能合约账户一视同仁，因此两者都可以调用智能合约或接收以太币的转账。\r\n\r\n如果以太币被转移到一个包含智能合约代码的地址，那么该智能合约就有机会运行一些代码。这种“回退功能”可用于根据转账更新内部状态或采取存款触发的其他行动（例如发行一些代币作为响应）。\r\n# 二、如今重入攻击仍然是一个重大问题吗？\r\n虽然重入攻击在这两年算得上是老生常谈了，但也有过这样的案例：\r\n\r\nUniswap/Lendf.Me 黑客攻击（2020 年 4 月）——2500 万美元，被黑客使用重入攻击。\r\nBurgerSwap 黑客攻击（2021 年 5 月）——720 万美元，原因是伪造的代币合约和可重入漏洞。\r\nSURGEBNB 黑客攻击（2021 年 8 月）——400 万美元似乎是一次基于重入的价格操纵攻击。\r\nCREAM FINANCE 黑客攻击（2021 年 8 月）——1880 万美元，重入漏洞允许利用者进行第二次借贷。\r\nSiren 协议黑客攻击（2021 年 9 月）——350 万美元，AMM 矿池通过重入攻击被利用。\r\nParaluni黑客攻击（2022 年 3 月）： Paraluni 智能合约在 2022 年 3 月遭到黑客攻击，利用重入漏洞和对不受信任的用户输入的验证不当窃取了约 170 万美元的代币。\r\nFei 协议黑客攻击 （2022 年 4 月） ——Fei 协议成为一次价值约 8000 万美元的黑客攻击的受害者，这次黑客攻击是通过使用包含重入漏洞的第三方代码实现的。\r\n# 三、重入漏洞分析\r\n## 单个函数的重入\r\n这个漏洞的第一个版本涉及可以在函数的第一次调用完成之前重复调用的函数。这可能会导致函数的不同调用以破坏性方式进行交互。\r\n\r\n```\r\n// INSECURE\r\nmapping (address => uint) private userBalances;\r\n\r\nfunction withdrawBalance() public {\r\n    uint amountToWithdraw = userBalances[msg.sender];\r\n    (bool success, ) = msg.sender.call.value(amountToWithdraw)(\"\"); // At this point, the caller's code is executed, and can call withdrawBalance again\r\n    require(success);\r\n    userBalances[msg.sender] = 0;\r\n}\r\n```\r\n由于用户的余额直到函数的最后才设置为 0，第二次（和以后的）调用仍然会成功，并且会一遍又一遍地提取余额。\r\n2016 年 6 月 17日，The DAO遭到黑客攻击，360 万以太币（5000 万美元）在第一次重入攻击中被盗。以太坊基金会发布了一个重要更新来回滚黑客攻击。这导致以太坊被分叉为以太坊经典和以太坊。\r\n\r\n在给出的例子中，防止这种攻击的最好方法是确保在完成所有需要做的内部工作之前不要调用外部函数：\r\n\r\n```\r\nmapping (address => uint) private userBalances;\r\n\r\nfunction withdrawBalance() public {\r\n    uint amountToWithdraw = userBalances[msg.sender];\r\n    userBalances[msg.sender] = 0;\r\n    (bool success, ) = msg.sender.call.value(amountToWithdraw)(\"\"); // The user's balance is already 0, so future invocations won't withdraw anything\r\n    require(success);\r\n}\r\n```\r\n请注意，如果您有另一个调用 的函数withdrawBalance()，它可能会受到相同的攻击，因此您必须将调用不受信任合约的任何函数视为本身不受信任。有关潜在解决方案的进一步讨论，请参见下文。\r\n## 跨函数重入\r\n攻击者还可以使用共享相同状态的两个不同函数进行类似的攻击。\r\n\r\n```\r\n// INSECURE\r\nmapping (address => uint) private userBalances;\r\n\r\nfunction transfer(address to, uint amount) {\r\n    if (userBalances[msg.sender] >= amount) {\r\n       userBalances[to] += amount;\r\n       userBalances[msg.sender] -= amount;\r\n    }\r\n}\r\n\r\nfunction withdrawBalance() public {\r\n    uint amountToWithdraw = userBalances[msg.sender];\r\n    (bool success, ) = msg.sender.call.value(amountToWithdraw)(\"\"); // At this point, the caller's code is executed, and can call transfer()\r\n    require(success);\r\n    userBalances[msg.sender] = 0;\r\n}\r\n```\r\n在这种情况下，攻击者在他们的代码中外部调用执行withdrawBalance的时候调用transfer()，由于他们的余额尚未设置为 0，因此即使他们已经收到提款，他们也可以转移代币。该漏洞也被用于 DAO 攻击。\r\n相同的解决方案将起作用，但有相同的注意事项。另请注意，在此示例中，两个函数都是同一个合约的一部分。但是，如果多个合约共享状态，则相同的错误可能会出现在多个合约中。\r\n# 四、 重入解决方案中的陷阱\r\n由于重入可能发生在多个函数甚至多个合约中，任何旨在防止单个函数重入的解决方案都是不够的。\r\n\r\n相反，我们建议首先完成所有内部工作（即状态更改），然后才调用外部函数。如果仔细遵守此规则，您将可以避免由于重入而导致的漏洞。但是，您不仅要避免过早调用外部函数，还要避免调用调用外部函数的函数。例如，以下是不安全的：\r\n\r\n```\r\n// INSECURE\r\nmapping (address => uint) private userBalances;\r\nmapping (address => bool) private claimedBonus;\r\nmapping (address => uint) private rewardsForA;\r\n\r\nfunction withdrawReward(address recipient) public {\r\n    uint amountToWithdraw = rewardsForA[recipient];\r\n    rewardsForA[recipient] = 0;\r\n    (bool success, ) = recipient.call.value(amountToWithdraw)(\"\");\r\n    require(success);\r\n}\r\n\r\nfunction getFirstWithdrawalBonus(address recipient) public {\r\n    require(!claimedBonus[recipient]); // Each recipient should only be able to claim the bonus once\r\n\r\n    rewardsForA[recipient] += 100;\r\n    withdrawReward(recipient); // At this point, the caller will be able to execute getFirstWithdrawalBonus again.\r\n    claimedBonus[recipient] = true;\r\n}\r\n```\r\n即使getFirstWithdrawalBonus()不直接调用外部合约，调用 withdrawReward()也足以使其容易受到重入的影响。因此，您需要将其 withdrawReward()视为不受信任。\r\n\r\n```\r\nmapping (address => uint) private userBalances;\r\nmapping (address => bool) private claimedBonus;\r\nmapping (address => uint) private rewardsForA;\r\n\r\nfunction untrustedWithdrawReward(address recipient) public {\r\n    uint amountToWithdraw = rewardsForA[recipient];\r\n    rewardsForA[recipient] = 0;\r\n    (bool success, ) = recipient.call.value(amountToWithdraw)(\"\");\r\n    require(success);\r\n}\r\n\r\nfunction untrustedGetFirstWithdrawalBonus(address recipient) public {\r\n    require(!claimedBonus[recipient]); // Each recipient should only be able to claim the bonus once\r\n\r\n    claimedBonus[recipient] = true;\r\n    rewardsForA[recipient] += 100;\r\n    untrustedWithdrawReward(recipient); // claimedBonus has been set to true, so reentry is impossible\r\n}\r\n```\r\n除了使重新进入变得不可能的修复之外， 还标记了不受信任的功能。同样的模式在每个级别重复：因为untrustedGetFirstWithdrawalBonus()调用 untrustedWithdrawReward()，它调用外部合约，所以您也必须将 untrustedGetFirstWithdrawalBonus()视为不安全的。\r\n通常建议的另一种解决方案是mutex。这允许您“锁定”某些状态，因此它只能由锁的所有者更改。一个简单的示例可能如下所示：\r\n\r\n\r\n```\r\n// Note: This is a rudimentary example, and mutexes are particularly useful where there is substantial logic and/or shared state\r\nmapping (address => uint) private balances;\r\nbool private lockBalances;\r\n\r\nfunction deposit() payable public returns (bool) {\r\n    require(!lockBalances);\r\n    lockBalances = true;\r\n    balances[msg.sender] += msg.value;\r\n    lockBalances = false;\r\n    return true;\r\n}\r\n\r\nfunction withdraw(uint amount) payable public returns (bool) {\r\n    require(!lockBalances && amount > 0 && balances[msg.sender] >= amount);\r\n    lockBalances = true;\r\n\r\n    (bool success, ) = msg.sender.call.value(amount)(\"\");\r\n\r\n    if (success) { // Normally insecure, but the mutex saves it\r\n      balances[msg.sender] -= amount;\r\n    }\r\n\r\n    lockBalances = false;\r\n    return true;\r\n}\r\n```\r\n如果用户在第一次调用完成之前尝试再次调用withdraw()，锁将阻止它产生任何效果。这可能是一种有效的模式，但当您有多个合约时，它会变得棘手。以下是不安全的：\r\n\r\n```\r\n// INSECURE\r\ncontract StateHolder {\r\n    uint private n;\r\n    address private lockHolder;\r\n\r\n    function getLock() {\r\n        require(lockHolder == address(0));\r\n        lockHolder = msg.sender;\r\n    }\r\n\r\n    function releaseLock() {\r\n        require(msg.sender == lockHolder);\r\n        lockHolder = address(0);\r\n    }\r\n\r\n    function set(uint newState) {\r\n        require(msg.sender == lockHolder);\r\n        n = newState;\r\n    }\r\n}\r\n```\r\n攻击者可以调用getLock()，然后永远不会调用releaseLock()。如果他们这样做，那么合同将永远被锁定，并且无法进行进一步的更改。如果您使用互斥体来防止重入，您将需要仔细确保没有任何方法可以声明锁并且永远不会释放锁。\r\n\r\n# 五、 总结\r\nDeFi 智能合约容易遭受重入攻击的一个共同点是，它们在易受攻击的代码启动之前没有经过安全审计。重入漏洞是一种众所周知的威胁，应该作为安全审计的一部分进行识别和修复。  保护web3世界的安全我们义不容辞！"},"author":{"user":"https://learnblockchain.cn/people/12138","address":null},"history":null,"timestamp":1673934723,"version":1}