Submission Details
Severity:
`whenNotPaused` will never work as we don't have a function to flip `paused` to true or false
Summary
whenNotPaused will never work as we don't have a function to flip paused to true or false
Vulnerability Details
paused is initialized, but we lack a function to flip it to true or back to false.
/**
* @notice Whether the contract is paused
*/
bool public paused;
modifier whenNotPaused() {
if (paused) revert ContractPaused();
_;
}
Impact
Contract can never be paused.
paused will always be false
whenNotPaused does not work. If a hack occurs in the future the funds have a higher likelihood to be stolen since the contract cannot be paused.
Tools Used
Manual review
Recommendations
Add a pause/unpause functions that would flit the variable and pause/unpause the contract.
Updates
Lead Judging Commences
inallhonesty
8 months ago
inallhonesty 7 months ago
Submission Judgement Published Assigned finding tags:
veRAACToken lacks the ability to configure `paused` variable
Prize pool breakdown
Total prize
77,280 USDC
nSLOC
3,86420
USDC / LOC
74,000
3,280
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.