Sparkn

CodeFox Inc.
DeFiFoundryProxy
15,000 USDC
View results
Submission Details
Severity: low
Valid

Remove out commented code of no use

Summary

Remove out commented code of no use

Vulnerability Details

The ProxyFactory.sol contains lines of code (require statements) that are commented out but serve no purpose. Commented-out code segments do not contribute to the functionality of the contracts but can clutter the codebase, making it harder for developers to understand the actual logic of the contracts.

// require(saltToCloseTime[salt] == 0, "Contest is not registered");
// require(saltToCloseTime[salt] < block.timestamp + EXPIRATION_TIME, "Contest is not expired");

Impact

It will make it harder for developers to understand the actual logic of the contracts.

Tools Used

Manual Analysis

Recommendations

Remove the commented line of code.

Support

FAQs

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