MultiSig Timelock

First Flight #55
Beginner FriendlyWallet
100 EXP
Submission Details
Impact: low
Likelihood: medium

Bug Static Confirmation Threshold Can Permanently Deadlock Governance in MultiSigTimelock.sol

Author Revealed upon completion

Root + Impact

Description

  • REQUIRED_CONFIRMATIONS is hardcoded to 3.

  • If signer count drops to 2, no transaction can ever execute.

uint256 private constant REQUIRED_CONFIRMATIONS = 3; // @>

Risk

Likelihood:

  • Signers revoked

  • Lost keys

Impact:

  • Permanent governance deadlock


Mitigation

uint256 public requiredConfirmations;
function updateConfirmations(uint256 newValue) external onlyOwner {
require(newValue <= s_signerCount);
requiredConfirmations = newValue;
}

Support

FAQs

Can't find an answer? Chat with us on Discord, Twitter or Linkedin.

Give us feedback!