DittoETH

Summary

Hardcoding the min delay could have unintended consequences in the contract.

Vulnerability Details

In the DittoTimelockController.sol contract we have this line

but if we see the Natspec above this line which are

we can see that arg[0] is the min delay which is 2 days, but in the above line we can see that minimum delay is set to 0 which could have unintended consequences.

Impact

Hardcoding 0 could have unintended consequences

Tools Used

Manual Review

Recommendations

do not hardcode 0 instead take it as perimeter