Submission Details
Severity:
Unintended behaviour because of Executed operation can again and again Re-schedule in scheduleEmergencyAction function
Summary
In TimelockController.sol because of Executed operation can again and again Re-schedule in scheduleEmergencyAction function. That will disrupt the functionality.
Vulnerability Details
/**
* @notice Schedules an emergency action
* @dev Only callable by addresses with EMERGENCY_ROLE
* @param id Operation ID for the emergency action
*/
function scheduleEmergencyAction(bytes32 id) external onlyRole(EMERGENCY_ROLE) {
@>>> There should be a check point . That will check is that Operation Id is executed before or not .
_emergencyActions[id] = true;
emit EmergencyActionScheduled(id, block.timestamp);
}
Impact
Increase Gas cost and lead to miscalculation.
Tools Used
manual review
Recommendations
There should be a check point . That will check is that Operation Id is executed before or not .
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.