no way to revoke Approval
Summary
The contract MultiSig.sol lacks functionality for owners to revoke approvals on pending transactions. Once an owner approves a transaction, the approval is permanent, even if the owner changes their mind or identifies a mistake.
P O C
Owner1 approves Transaction A.
Owner1 realizes Transaction A is malicious but cannot revoke approval.
Owner2 approves Transaction A, allowing execution.
Vulnerability Details
Impact
Malicious or accidental approvals cannot be undone, forcing owners to either execute unwanted transactions or leave them pending indefinitely
Tools Used
Manual Review
Recommendations
Add a revokeApproval function
Appeal created
Users mistake, only impacting themselves.
Please read the CodeHawks documentation to know which submissions are valid. If you disagree, provide a coded PoC and explain the real likelihood and the detailed impact on the mainnet without any supposition (if, it could, etc) to prove your point.
Rewards breakdown
Live
The contest is live. Earn rewards by submitting a finding.
Appeals
This is your time to appeal against judgements on your submissions.
Appeals Review
Appeals are being carefully reviewed by our judges.