{"content":{"title":"DAO投票漏洞","body":"作者：Konstantin Nekrasov - MixBytes 的安全研究员\r\n\r\n![](https://img.learnblockchain.cn/2025/03/09/25.jpg)\r\n\r\n## 代币投票\r\n\r\n去中心化自治组织（DAO）在区块链上运行，并通过投票进行管理。代币投票是最受欢迎的方式：DAO的成员提出建议，其他代币持有者以代币投票表示同意。当提案达到法定人数时，可以执行其脚本。\r\n\r\n这种方法存在合理的批评，维塔利克·布特林在以下链接中进行了强调 [\\[1\\]](https://learnblockchain.cn/article/12449) [\\[2\\]](https://learnblockchain.cn/article/11648) [\\[3\\]](https://learnblockchain.cn/article/11481)：\r\n\r\n1. 小规模富有参与者（“鲸鱼”）在成功执行决策方面优于大量小额持有者。\r\n2. 代币投票治理赋予代币持有者的利益，而牺牲了社区的其他部分。\r\n3. 对于同样持有与该平台互动的其他 DeFi 平台代币的代币持有者，可能会出现利益冲突。\r\n4. 代币投票对攻击者根本性的脆弱性：投票买卖 [\\[4\\]](https://blog.mangata.finance/blog/2022-10-08-council-incident-report/) [\\[5\\]](https://bean.money/blog/beanstalk-governance-exploit) [\\[7\\]](https://gitlab.com/relyt29/votebuying-carbonvote)，投票借贷 [\\[9\\]](https://www.placeholder.vc/blog/2020/1/7/how-much-does-a-crypto-vote-cost) 和鲸鱼串通 [\\[6\\]](https://decrypt.co/38050/steem-steemit-tron-justin-sun-cryptocurrency-war)。\r\n5. 暴露于复杂的博弈论攻击 [\\[10\\]](https://hackingdistributed.com/2016/05/27/dao-call-for-moratorium/)。\r\n\r\n这些都是非技术问题。此外，程序员可能因分心或对区块链工作原理的知识不足而在代码中引入纯技术漏洞。\r\n\r\n有许多 DAO 使用代币投票：基于 Aragon 的 DAO、X-DAO、Nexus Mutual、Showball Finance、Pickle Finance、Spirit Swap、Keep3r Network 等等。\r\n\r\n在本文中，我们将检查代币投票中可能出现的技术漏洞，并检查它们在上述一些 DAO 中是否存在。\r\n\r\n## 攻击 #1. 闪电贷\r\n\r\n如果黑客能够在同一区块内投票并执行提案（例如，通过紧急方法），DAO 可能存在漏洞。这类漏洞在 Beanstalk [\\[5\\]](https://bean.money/blog/beanstalk-governance-exploit) 和 MakerDAO [\\[11\\]](https://www.theblock.co/post/82721/makerdao-issues-warning-after-a-flash-loan-is-used-to-pass-a-governance-vote) 等项目中曾遇到过。\r\n\r\nAragon\r\n\r\nAragon 使用 [MiniMeToken 的](https://github.com/aragon/minime/blob/1d5251fc88eee5024ff318d95bc9f4c5de130430/contracts/MiniMeToken.sol) balanceOfAt() 在提案创建的前一区块计算用户余额。这使得闪电贷攻击不可能。\r\n\r\nX-DAO\r\n\r\nX-DAO 在转移时会回滚 [\\[→\\]](https://github.com/xdao-app/xdao-contracts/blob/855c2d6aa4016d09ae357abaff945ceb8af82347/contracts/core/Dao.sol#L580)：\r\n\r\n```cs\r\nfunction transfer(address, uint256) public pure override returns (bool) {\r\n    revert(\"GT: transfer is prohibited\");\r\n}\r\n```\r\n\r\n因此，不可能借出代币、在 DEX 上出售它，也不可能借入或购买，因此闪电贷攻击是不可能的。\r\n\r\nNexus Mutual\r\n\r\nNexus Mutual 的提案分为由顾问委员会投票的提案和由普通成员投票的提案。在顾问委员会投票中，每个参与者的权重相等于一。成员投票是普通代币投票。我们的重点仅在于代币投票。\r\n\r\n如果所有其他成员都已经投票，则可以在一个事务中投票并执行提案 [\\[→\\]](https://github.com/NexusMutual/smart-contracts/blob/283f40208ec5bc0fc529cedeacf307c19086ccde/contracts/modules/governance/Governance.sol#L711)：\r\n\r\n```php\r\nfunction canCloseProposal(uint _proposalId)\r\n    ...\r\n    if (numberOfMembers == proposalVoteTally[_proposalId].voters\r\n      || dateUpdate.add(_closingTime) <= now)\r\n      return 1;\r\n```\r\n\r\n但有一个复杂性——用户的代币在每次投票后在接下来的 7 天内不可转移 [\\[→\\]](https://github.com/NexusMutual/smart-contracts/blob/283f40208ec5bc0fc529cedeacf307c19086ccde/contracts/modules/governance/Governance.sol#L911)：\r\n\r\n```css\r\ntokenInstance.lockForMemberVote(msg.sender, tokenHoldingTime);\r\n```\r\n\r\n因此，从技术上讲，黑客可以进行闪电贷，从市场上拉取 NXM 代币，在同一事务中投票并执行提案。但他们仍然需要偿还贷款，这需要从执行的提案中获得至少与闪电贷相同的利润。然而，由于在发现数据验证漏洞后引入的各种限制，似乎很难提出成功的攻击 [\\[14\\]](https://learnblockchain.cn/article/12424)。\r\n\r\nKeep3r Network\r\n\r\n该项目使用 [类似 MiniMeToken 的代币](https://github.com/keep3r-network/keep3r.network/blob/a6897007db6e656e0e310ed7ee4ad42904fe2794/contracts/Keep3r.sol#L633)。它在提案创建区块之前的一块计算投票权重通过 getPriorVotes()。可以得出相同的结论。\r\n\r\n## 攻击 #2. 错误的重投\r\n\r\n如果一个合同允许用户对提案进行重投，但以不正确的方式扣除用户的旧投票，则可能会出现漏洞。\r\n\r\n建议检查以下危险场景：\r\n\r\n- 投票给不在提案列表中的提案；\r\n- 投票 → 转让 → 投票；\r\n- 在创建的同一块对提案投票；\r\n- 对同一提案 ID 使用错误的参数进行投票；\r\n- 重放离线交易。\r\n\r\n这种类型的漏洞在 MakerDAO [\\[12\\]](https://blog.openzeppelin.com/makerdao-critical-vulnerability/) 和 KP3R Network [\\[13\\]](https://statemind.io/blog/2022/09/27/gauge-proxy-bug.html) 等项目中曾遭遇过。\r\n\r\nAragon\r\n\r\n在 Aragon 中可以重投，但它正确地增加/减少上一次的投票权 [\\[→\\]](https://github.com/aragon/aragon-apps/blob/b72da2c6606a361d0160d5d78fb534018ba3ce91/apps/voting/contracts/Voting.sol#L316)：\r\n\r\n```cpp\r\n// 这可能会重入，尽管我们可以假设治理代币不是恶意的。\r\nuint256 voterStake = token.balanceOfAt(_voter, vote_.snapshotBlock);\r\nVoterState state = vote_.voters[_voter];\r\n\r\n// 如果选民之前投过票，则减少数量。\r\nif (state == VoterState.Yea) {\r\n    vote_.yea = vote_.yea.sub(voterStake);\r\n} else if (state == VoterState.Nay) {\r\n    vote_.nay = vote_.nay.sub(voterStake);\r\n}\r\n```\r\n\r\n投票给不在提案列表中的提案\r\n\r\n在投票方法中有一个 voteExists 修饰符，因此不可能对不存在的提案进行投票 [\\[→\\]](https://github.com/aragon/aragon-apps/blob/b72da2c6606a361d0160d5d78fb534018ba3ce91/apps/voting/contracts/Voting.sol#L154)：\r\n\r\n```javascript\r\nfunction vote(\r\n    uint256 _voteId,\r\n    bool _supports,\r\n    bool _executesIfDecided\r\n) external voteExists(_voteId)\r\n```\r\n\r\n其他情况\r\n\r\n用户的投票权不能转移并用于同一提案，因为 Aragon 在提案创建前的一块使用 MinimeToken 的余额。\r\n\r\nX-DAO\r\n\r\nX-DAO 的投票发生在链外，没有重新投票的机制。如果用户签署了拒绝，他们仍然可以签署批准，批准将被计算。但不能以相反的方式进行——只有正面的投票会被计算 [\\[→\\]](https://github.com/xdao-app/xdao-contracts/blob/855c2d6aa4016d09ae357abaff945ceb8af82347/contracts/core/Dao.sol#L281)：\r\n\r\n```cpp\r\nfor (uint256 i = 0; i < signers.length; i++) {\r\n    share += balanceOf(signers[i]);\r\n}\r\n\r\nif (share * 100 < totalSupply() * quorum) {\r\n    return false;\r\n}\r\n```\r\n\r\n已签署的投票不能被计数两次 [\\[→\\]](https://github.com/xdao-app/xdao-contracts/blob/855c2d6aa4016d09ae357abaff945ceb8af82347/contracts/core/Dao.sol#L278)：\r\n\r\n```coffeescript\r\nrequire(!_hasDuplicate(signers), \"DAO: signatures are not unique.\");\r\n```\r\n\r\n重放攻击\r\n\r\n为了投票，用户签署一个提案，这只是一个数据集：目标地址、 calldata、msg 值、nonce、时间戳、block.chainid 和 X-DAO 实例地址。因此，在不同的以太坊链上、在另一个 X-DAO 实例上，甚至在具有不同 nonce 的另一个提案上重放用户签署的投票都是不可能的 [\\[→\\]](https://github.com/xdao-app/xdao-contracts/blob/855c2d6aa4016d09ae357abaff945ceb8af82347/contracts/core/Dao.sol#L240)：\r\n\r\n```php\r\nfunction getTxHash\r\n...\r\nreturn\r\n    keccak256(abi.encode(\r\n        address(this),\r\n        _target,\r\n        _data,\r\n        _value,\r\n        _nonce,\r\n        _timestamp,\r\n        block.chainid\r\n    ));\r\n}\r\n```\r\n\r\n提案也无法重放 [\\[→\\]](https://github.com/xdao-app/xdao-contracts/blob/855c2d6aa4016d09ae357abaff945ceb8af82347/contracts/core/Dao.sol#L197)：\r\n\r\n```coffeescript\r\nrequire(!executedTx[txHash], \"DAO: voting already executed.\");\r\n```\r\n\r\nSnowball Finance\r\n\r\n用户不能投票给不在提案列表中的提案，但他们可以对相同的提案进行重投。合同在重投时正确地扣除了他们的先前决定。用户的代币被锁定在托管合约中，无法转移。\r\n\r\nSpirit Swap\r\n\r\nSpirit Swap 没有提案：用户只能选择并投票代币的权重。用户可以每周投票一次，而无法重投。\r\n\r\nKeep3r Network\r\n\r\nKeep3r Network 的行为类似于 Aragon，但用户不能对同一提案投票两次。\r\n\r\n对不存在提案的投票在要求中失败 [\\[→\\]](https://github.com/keep3r-network/keep3r.network/blob/a6897007db6e656e0e310ed7ee4ad42904fe2794/contracts/Keep3rGovernance.sol#L473)：\r\n\r\n```php\r\nfunction _castVote...\r\n    ...\r\n    require(state(proposalId) == ProposalState.Active, \"Governance::_castVote: voting is closed.\");\r\n```\r\n\r\n同样，不可能重投 [\\[→\\]](https://github.com/keep3r-network/keep3r.network/blob/a6897007db6e656e0e310ed7ee4ad42904fe2794/contracts/Keep3rGovernance.sol#L476)：\r\n\r\n```coffeescript\r\nrequire(receipt.hasVoted == false, \"Governance::_castVote: voter already voted.\");\r\n```\r\n\r\nNexus Mutual\r\n\r\n不可能对不在提案列表中的提案进行投票，因为以下要求将失败 [\\[→\\]](https://github.com/NexusMutual/smart-contracts/blob/283f40208ec5bc0fc529cedeacf307c19086ccde/contracts/modules/governance/Governance.sol#L282)：\r\n\r\n```cs\r\nfunction submitVote(uint _proposalId, uint _solutionChosen) external {\r\n    ...\r\n    require(allProposalData[_proposalId].propStatus == uint(Governance.ProposalStatus.VotingStarted), \"Not allowed\");\r\n```\r\n\r\n用户只能对同一提案进行一次投票 [\\[→\\]](https://github.com/NexusMutual/smart-contracts/blob/283f40208ec5bc0fc529cedeacf307c19086ccde/contracts/modules/governance/Governance.sol#L859)：\r\n\r\n```javascript\r\nfunction _submitVote(uint _proposalId, uint _solution) internal {\r\n    ...\r\n    require(memberProposalVote[msg.sender][_proposalId] == 0, \"Not allowed\");\r\n    ...\r\n    memberProposalVote[msg.sender][_proposalId] = totalVotes;\r\n```\r\n\r\n## 攻击 #3. 缺少提案验证\r\n\r\n如果提案的属性没有完全验证，黑客可能会创造出看似良善的破坏性提案，存在社会工程的机会。\r\n\r\n回答以下问题：\r\n\r\n1. 提案如何在网站上显示？\r\n2. 黑客能否在提案中提供任意脚本或 calldata？\r\n3. 黑客能否在恶意提案中提供任意良善描述？\r\n4. 普通用户判断提案的脚本实际做了什么是否困难？\r\n\r\n在 Beanstalk [\\[5\\]](https://bean.money/blog/beanstalk-governance-exploit) 和 Nexus Mutual [\\[14\\]](https://learnblockchain.cn/article/12424) 等项目中曾遇到过验证不足的问题。\r\n\r\nAragon\r\n\r\n在 Aragon 中，新的提案接受任意执行脚本和元数据 [\\[→\\]](https://github.com/aragon/aragon-apps/blob/b72da2c6606a361d0160d5d78fb534018ba3ce91/apps/voting/contracts/Voting.sol#L126)：\r\n\r\n```javascript\r\nfunction newVote(bytes _executionScript, string _metadata) external auth(CREATE_VOTES_ROLE) returns (uint256 voteId)\r\n```\r\n\r\n元数据被发出，但没有保存在存储中，因此暗示后台将解析区块链中的事件，以显示提案作者地址和描述 [\\[→\\]](https://github.com/aragon/aragon-apps/blob/b72da2c6606a361d0160d5d78fb534018ba3ce91/apps/voting/contracts/Voting.sol#L298)：\r\n\r\n```cpp\r\nemit StartVote(voteId, msg.sender, _metadata);\r\n```\r\n\r\n因此，可以提供任意执行脚本。因此，验证数据的责任落在投票者身上。\r\n\r\nX-DAO\r\n\r\n在 X-DAO 中没有链上的方法来创建提案。\r\n\r\n可以在 [xdao.app](https://www.xdao.app/137/ecosystem) 上创建离线提案，并且有两点需要注意：\r\n\r\n1. 目标地址及其 calldata 默认是隐藏的，因此用户可能会误解恶意意图。提案示例见 [链接](https://www.xdao.app/137/dao/0x60620f047049511b6c7734bd6715800de22d3a73/votingPage/0x99455859248fc849abbed4363012657ce9c3646aed1e98e8b15db67484926360)\r\n2. 可以为提案指定任意标题和描述：[链接](https://www.xdao.app/137/dao/0x60620f047049511b6c7734bd6715800de22d3a73/votingPage/0x99455859248fc849abbed4363012657ce9c3646aed1e98e8b15db67484926360)\r\n\r\n因此，验证数据的责任落在投票者身上。\r\n\r\nSnowball Finance\r\n\r\n在 Snowball Finance 中，新提案接收以下参数：\r\n\r\n```cpp\r\nfunction propose(\r\n    string calldata _title,\r\n    string calldata _metadata,\r\n    uint256 _votingPeriod,\r\n    address _target,\r\n    uint256 _value,\r\n    bytes memory _data\r\n)\r\n```\r\n\r\n并将所有这些参数存储到合同存储中，包括 msg.sender。\r\n\r\n对 calldata 没有限制。所以，验证数据的责任落在投票者身上。\r\n\r\nSpirit Swap\r\n\r\nSpirit Swap 在投票中没有 ID 或元数据，因此此部分无关。\r\n\r\nKeep3r Network\r\n\r\nKeep3r Network 中的新提案接受一组目标、msg 值、calldata 和其他元数据 [\\[→\\]](https://github.com/keep3r-network/keep3r.network/blob/a6897007db6e656e0e310ed7ee4ad42904fe2794/contracts/Keep3rGovernance.sol#L340)：\r\n\r\n```css\r\nfunction propose(\r\n    address[] memory targets,\r\n    uint256[] memory values,\r\n    string[] memory signatures,\r\n    bytes[] memory calldatas,\r\n    string memory description\r\n)\r\n```\r\n\r\n攻击者仍然可以提供任意 calldata。因此，验证数据的责任落在投票者身上。\r\n\r\nNexus Mutual\r\n\r\nNexus Mutual 的提案分为不同类别。每个类别都有预定的操作地址和方法签名。网站上的提案清楚地显示了要调用的方法及其参数： [https://app.nexusmutual.io/governance/view?proposalId=175](https://app.nexusmutual.io/governance/view?proposalId=175)\r\n\r\n因此，似乎不存在社会工程机会。\r\n\r\n## 攻击 #4. 无转移验证\r\n\r\n代币锁定机制应检查批准的 transferFrom 调用（及其他类似的转移方法）的返回值：\r\n\r\n- Aragon Minime 代币在调用 transferFrom 函数失败时返回 false。\r\n\r\n这种类型的漏洞曾在 ForceDAO 中遇到过 [\\[15\\]](https://halborn.com/explained-the-forcedao-hack-april-2021/)。\r\n\r\nNexus Mutual\r\n\r\nNexus Mutual 通过 tokenInstance.lockForMemberVote() 方法锁定代币，而不使用 transferFrom [\\[→\\]](https://github.com/NexusMutual/smart-contracts/blob/283f40208ec5bc0fc529cedeacf307c19086ccde/contracts/modules/token/NXMToken.sol#L252)：\r\n\r\n```cs\r\nfunction lockForMemberVote(\r\n    address _of,\r\n    uint _days\r\n) public onlyOperator {\r\n    if (_days.add(now) > isLockedForMV[_of])\r\n        isLockedForMV[_of] = _days.add(now);\r\n}\r\n```\r\n\r\nAragon、X-DAO 和 Keep3r Network：无效适用\r\n\r\nAragon、X-DAO 和 Keep3r Network 没有锁定机制。\r\n\r\nSnowball Finance 和 Spirit Swap：托管\r\n\r\nSnowball Finance 和 Spirit Swap 使用类似的托管合约来在一段时间内锁定代币。两个合约都检查转移的结果 [\\[→\\]](https://github.com/Layer3Org/spiritswap-core/blob/f730bcb4a96088f4878dffc6b578afdfe42db945/SpiritV2/contracts/SpiritV1/inSpirit.vy#L377)：\r\n\r\n```css\r\nassert ERC20(self.token).transferFrom(_addr, self, _value)\r\n...\r\nassert ERC20(self.token).transfer(msg.sender, value)\r\n```\r\n\r\n## 攻击 #5. 小投票窗口\r\n\r\n对某些提案持消极态度的用户和否决权持有者可能没有时间作出反应。尤其是当法定人数少于 50% 时。\r\n\r\nAragon\r\n\r\n投票和执行的开放时间为 voteTime [\\[→\\]](https://github.com/aragon/aragon-apps/blob/b72da2c6606a361d0160d5d78fb534018ba3ce91/apps/voting/contracts/Voting.sol#L406)：\r\n\r\n```javascript\r\nfunction _isVoteOpen(Vote storage vote_) internal view returns (bool) {\r\n    return getTimestamp64() < vote_.startDate.add(voteTime) && !vote_.executed;\r\n}\r\n```\r\n\r\n这是一个全局参数，仅初始化一次。因此，社会工程的可能性取决于特定项目的初始化。\r\n\r\nX-DAO\r\n\r\n投票和执行的开放时间为 3 天 [\\[→\\]](https://github.com/xdao-app/xdao-contracts/blob/855c2d6aa4016d09ae357abaff945ceb8af82347/contracts/core/Dao.sol#L191)：\r\n\r\n```cpp\r\nuint32 public constant VOTING_DURATION = 3 days;\r\n...\r\nrequire(\r\n    _timestamp + VOTING_DURATION >= block.timestamp,\r\n    \"DAO: voting is over.\"\r\n);\r\n```\r\n\r\n在我们看来，这应该足以让 DAO 的活跃参与者投票。\r\n\r\nSnowball Finance\r\n\r\nSnowball Finance 的时间周期是可变但有限的，可以由治理设置：\r\n\r\n- 投票期从 1 天到 30 天不等；\r\n- 执行延迟从 30 秒到 30 天不等；\r\n- 过期时间为 14 天。\r\n\r\n在我们看来，这应该足以让 DAO 的活跃参与者投票。\r\n\r\nSpirit Swap\r\n\r\nSpirit Swap 每周允许投票一次 [\\[→\\]](https://github.com/Layer3Org/spiritswap-core/blob/f730bcb4a96088f4878dffc6b578afdfe42db945/SpiritV2/contracts/SpiritV2/StableGaugeProxy.sol#L713)：\r\n\r\n```cpp\r\nuint256 public voteDelay = 604800;\r\n\r\n...\r\n\r\nmodifier hasVoted(address voter) {\r\n    uint256 time = block.timestamp - lastVote[voter];\r\n    require(time > voteDelay, \"You voted in the last 7 days.\");\r\n    _;\r\n}\r\n```\r\n\r\n在我们看来，这应该足以让 DAO 的活跃参与者投票。\r\n\r\nKeep3r Network\r\n\r\nKeep3r Network 的时间周期是可变但有限的，可以由治理设置：\r\n\r\n- 投票期从 1 天到 30 天不等；\r\n- 执行期为 14 天。\r\n\r\n在我们看来，这应该足以让 DAO 的活跃参与者投票。\r\n\r\nNexus Mutual\r\n\r\n提案可在 _closingTime 经过后关闭并执行 [\\[→\\]](https://github.com/NexusMutual/smart-contracts/blob/283f40208ec5bc0fc529cedeacf307c19086ccde/contracts/modules/governance/Governance.sol#L711)：\r\n\r\n```php\r\nfunction canCloseProposal(uint _proposalId)\r\n...\r\nif (numberOfMembers == proposalVoteTally[_proposalId].voters\r\n  || dateUpdate.add(_closingTime) <= now)\r\n  return 1;\r\n```\r\n\r\n可以在网站上看到，参数从 3 到 7 天不等，具体取决于提案类别：\r\n\r\n[https://app.nexusmutual.io/governance/categories](https://app.nexusmutual.io/governance/categories)\r\n\r\n在我们看来，这应该足以让 DAO 的活跃参与者投票。\r\n\r\n## 攻击 #6. 双重投票\r\n\r\n黑客是否可以对同一提案投票两次，使用相同的代币？\r\n\r\n建议检查以下场景：\r\n\r\n- 投票 → 转让 → 再次投票；\r\n- 投票 → 委托 → 再次投票；\r\n- 调整 vote() 参数以增加额外的投票权；\r\n- 检查重入性。\r\n\r\nAragon\r\n\r\n投票-转移-投票\r\n\r\n可以在用户之间移动代币，但只在提案创建前的一块才重要，因此没有人能够双重投票。\r\n\r\n投票-委托-投票\r\n\r\n默认的 Aragon 合约中没有委托机制。\r\n\r\n参数调整\r\n\r\n没有参数可调整：\r\n\r\n```javascript\r\nfunction vote(\r\n    uint256 _voteId,\r\n    bool _supports,\r\n    bool _executesIfDecided\r\n) external voteExists(_voteId)\r\n```\r\n\r\n重入性\r\n\r\n投票可能会通过 \\_unsafeExecuteVote() 进行外部调用，但代码遵循检查-影响-交互模式，因此对重入性是免疫的。\r\n\r\nX-DAO\r\n\r\n投票-转移-投票\r\n\r\n可以签署投票并将代币转移到另一个账户，以便他们可以签署另一个投票。但是 execute() 方法仅计数最终的代币分配，因此黑客的场景不适用。\r\n\r\n其他情况\r\n\r\n没有委托机制，也没有链上 vote() 方法，因此没有参数可以调整或尝试重新进入。\r\n\r\nSpirit Swap\r\n\r\nSpirit Swap 投票只会更改协议中的代币权重，并且在用户投票时应用更改。\r\n\r\n投票-转移-投票\r\n\r\n可以投票，等待托管合约中的代币解锁，将代币转移到另一个账户，再次锁定，然后为同一提案投票。但是计算表明，如果代币在整个过程中被简单地锁定，所应用的总投票权将是相同的。没有利益。\r\n\r\n投票-委托-投票\r\n\r\n用户在托管合约中锁定的投票权无法委托给其他用户。\r\n\r\n参数调整\r\n\r\nvote()方法接受代币和权重的数组，因此需要检查这些参数是否可以以某种方式被调整，以增加某些代币的额外投票权。如果你传递一个包含相同代币两次的数组会发生什么？\r\n\r\n看起来投票方法会正确考虑其所有参数，并且在数组中传递相同的代币不会影响计算的正确性，因为所有权重都按总权重之和划分 [\\[→\\]](https://github.com/Layer3Org/spiritswap-core/blob/f730bcb4a96088f4878dffc6b578afdfe42db945/SpiritV2/contracts/SpiritV2/StableGaugeProxy.sol#L834)：\r\n\r\n```cpp\r\nfor (uint256 i = 0; i < _tokenCnt; i++) {\r\n    _totalVoteWeight = _totalVoteWeight + _weights[i];\r\n}\r\n```\r\n\r\n重入性\r\n\r\n接下来检查的是是否存在重入性。vote() 方法中存在外部调用 [\\[→\\]](https://github.com/Layer3Org/spiritswap-core/blob/f730bcb4a96088f4878dffc6b578afdfe42db945/SpiritV2/contracts/SpiritV2/StableGaugeProxy.sol#L792)：\r\n\r\n```css\r\nIBribe(bribes[gauges[_token]])._withdraw(...);\r\n...\r\nIBribe(bribes[_gauge])._deposit(uint256(_tokenWeight), _owner);\r\n```\r\n\r\nIBribe 合约可以视为可信，因此，即使这里可能存在漏洞，也不会构成威胁。\r\n\r\nSnowball Finance\r\n\r\n投票-转移-投票\r\n\r\n可以投票，等待托管合约中的代币解锁，将代币转移到另一个账户，再次锁定，然后为同一提案投票。但是计算表明，如果代币在整个过程中被简单地锁定，所应用的总投票权将是相同的。没有利益。\r\n\r\n投票-委托-投票\r\n\r\n没有委托。\r\n\r\n参数调整\r\n\r\n没有参数可调整：\r\n\r\n```javascript\r\nfunction vote(uint256 _proposalId, bool _support)\r\n```\r\n\r\n重入性\r\n\r\nvote() 方法中没有不可信的外部调用。\r\n\r\nNexus Mutual\r\n\r\n投票-转移-投票\r\n\r\n这是不可能的，因为用户的转移在每次投票后被锁定 [\\[→\\]](https://github.com/NexusMutual/smart-contracts/blob/283f40208ec5bc0fc529cedeacf307c19086ccde/contracts/modules/governance/Governance.sol#L911)：\r\n\r\n```cs\r\nfunction _setVoteTally(uint _proposalId, uint _solution, uint mrSequence) internal {\r\n    ...\r\n    tokenInstance.lockForMemberVote(msg.sender, tokenHoldingTime);\r\n```\r\n\r\n投票-委托-投票\r\n\r\n目前不允许委托 [\\[→\\]](https://github.com/NexusMutual/smart-contracts/blob/283f40208ec5bc0fc529cedeacf307c19086ccde/contracts/modules/governance/Governance.sol#L396)：\r\n\r\n```javascript\r\nfunction delegateVote(address _add) external isMemberAndcheckPause checkPendingRewards {\r\n    revert(\"Delegations are not allowed.\");\r\n```\r\n\r\n但即使没有回滚，还有另一个要求，即如果用户刚刚进行了投票，他们不能委托其投票权 [\\[→\\]](https://github.com/NexusMutual/smart-contracts/blob/283f40208ec5bc0fc529cedeacf307c19086ccde/contracts/modules/governance/Governance.sol#L414)：\r\n\r\n```perl\r\nif (allVotesByMember[msg.sender].length > 0) {\r\n    require((allVotes[allVotesByMember[msg.sender][allVotesByMember[msg.sender].length - 1]].dateAdd).add(tokenHoldingTime) < now);\r\n}\r\n```\r\n\r\n参数调整\r\n\r\n没有参数可调整：\r\n\r\n```cs\r\nfunction submitVote(uint _proposalId, uint _solutionChosen)\r\n```\r\n\r\n重入性\r\n\r\nsubmitVote() 方法的代码遵循检查-影响-交互模式，因此没有重入性。\r\n\r\nKeep3r Network\r\n\r\n投票-转移-投票\r\n\r\nKeep3r 的行为与 Aragon 类似，检查用户在提案创建前的一块的投票权。因此此情况不适用。\r\n\r\n参数调整\r\n\r\n没有参数可调整：\r\n\r\n```javascript\r\nfunction castVote(uint256 proposalId, bool support)\r\n```\r\n\r\n重入性\r\n\r\ncastVote() 方法中没有不可信的外部调用。\r\n\r\n## 攻击 #7. 双重执行\r\n\r\nexecute() 方法中是否存在重入性？可以在同一区块内调用两次吗？\r\n\r\nAragon、X-DAO、Nexus Mutual 和 Keep3r Network\r\n\r\nAragon、X-DAO 和 Keep3r Network 遵循检查-影响-交互模式，因此它们的执行方法不易受重入性攻击。\r\n\r\nSnowball Finance\r\n\r\nSnowball Finance 实现了 nonReentrant 修饰符，其执行方法也不易受重入性攻击。\r\n\r\nSpirit Swap\r\n\r\nSpirit Swap 没有执行() 方法：它在 vote() 方法中应用更改。\r\n\r\n## 结论\r\n\r\n这是在测试具有投票功能的 DAO 时值得检查的基本列表。欢迎使用！\r\n\r\n###  链接\r\n\r\n01. 区块链治理笔记\r\n\r\n    [https://learnblockchain.cn/article/12449](https://learnblockchain.cn/article/12449)\r\n02. 在不知情的人中，区块链投票被高估，但在知情的人中被低估\r\n\r\n    [https://learnblockchain.cn/article/11648](https://learnblockchain.cn/article/11648)\r\n03. 超越代币投票治理\r\n\r\n    [https://learnblockchain.cn/article/11481](https://learnblockchain.cn/article/11481)\r\n04. 2022年10月6日，Mangata X 受到治理攻击，导致攻击者获得链上理事会的投票权\r\n\r\n    [https://blog.mangata.finance/blog/2022-10-08-council-incident-report/](https://blog.mangata.finance/blog/2022-10-08-council-incident-report/)\r\n05. 2022年4月17日，犯罪者使用闪电贷利用 Beanstalk 的治理机制\r\n\r\n    [https://bean.money/blog/beanstalk-governance-exploit](https://bean.money/blog/beanstalk-governance-exploit)\r\n06. Tron 基金会 CEO Justin Sun 与大型加密交易所勾结，利用客户的代币为 Steem 网络投票，该网络的绝大多数社区成员强烈反对\r\n\r\n    [https://decrypt.co/38050/steem-steemit-tron-justin-sun-cryptocurrency-war](https://decrypt.co/38050/steem-steemit-tron-justin-sun-cryptocurrency-war)\r\n07. 碳投票以太坊区块链投票实现的投票购买合同的概念证明\r\n\r\n    [https://gitlab.com/relyt29/votebuying-carbonvote](https://gitlab.com/relyt29/votebuying-carbonvote)\r\n08. 一个 uniswap 池，用于购买 $TRIBE——一个治理 Fei Protocol 的代币\r\n\r\n    [https://info.uniswap.org/#/tokens/0xc7283b66eb1eb5fb86327f08e1b5816b0720212b](https://info.uniswap.org/#/tokens/0xc7283b66eb1eb5fb86327f08e1b5816b0720212b)\r\n09. 一次加密投票的成本是多少？\r\n\r\n    [https://www.placeholder.vc/blog/2020/1/7/how-much-does-a-crypto-vote-cost](https://www.placeholder.vc/blog/2020/1/7/how-much-does-a-crypto-vote-cost)\r\n10. 对 DAO 的暂时叫停的呼吁\r\n\r\n    （发现了多种博弈论漏洞）\r\n\r\n    [https://hackingdistributed.com/2016/05/27/dao-call-for-moratorium/](https://hackingdistributed.com/2016/05/27/dao-call-for-moratorium/)\r\n11. 在闪电贷被用来通过治理投票后，MakerDAO 发布了警告\r\n\r\n    [https://www.theblock.co/post/82721/makerdao-issues-warning-after-a-flash-loan-is-used-to-pass-a-governance-vote](https://www.theblock.co/post/82721/makerdao-issues-warning-after-a-flash-loan-is-used-to-pass-a-governance-vote)\r\n12. MakerDAO 治理中的关键漏洞技术描述\r\n\r\n    [https://blog.openzeppelin.com/makerdao-critical-vulnerability/](https://blog.openzeppelin.com/makerdao-critical-vulnerability/)\r\n13. KP3R 漏洞报告：Statemind 如何在 Keep3r Network 中发现两年前的漏洞\r\n\r\n    [https://statemind.io/blog/2022/09/27/gauge-proxy-bug.html](https://statemind.io/blog/2022/09/27/gauge-proxy-bug.html)\r\n14. Nexus Mutual——calldata 验证漏洞\r\n\r\n    [https://learnblockchain.cn/article/12424](https://learnblockchain.cn/article/12424)\r\n15. 2021年4月4日，ForceDAO DeFi 聚合器被一名白帽和四名黑帽黑客利用。恶意攻击者能够偷走 FORCE 代币。\r\n\r\n    [https://halborn.com/explained-the-forcedao-hack-april-2021/](https://halborn.com/explained-the-forcedao-hack-april-2021/)\r\n\r\n- MixBytes 是谁？\r\n\r\n\r\n\r\n[MixBytes](https://mixbytes.io/) 是一个专业的区块链审计和安全研究团队，专注于为兼容 EVM 和基于 Substrate 的项目提供全面的智能合约审计和技术咨询服务。请加入我们在 [X](https://twitter.com/MixBytes) 中，以获取最新的行业趋势和见解。\r\n\r\n>- 原文链接： [mixbytes.io/blog/dao-vot...](https://mixbytes.io/blog/dao-voting-vulnerabilities)\r\n>- 登链社区 AI 助手，为大家转译优秀英文文章，如有翻译不通的地方，还请包涵～"},"author":{"user":"https://learnblockchain.cn/people/24680","address":null},"history":"bafkreic4y54f6owmsidwnq37ue6snzwsza4ffwssoxroiifnzhaxk7hi2e","timestamp":1741495154,"version":1}