Beginner FriendlyFoundryBridge
100 EXP
View results
Submission Details
Severity: low
Invalid

Comment Should be clear

Summary

Whenever writing a comment or docstring for code, it should be clear about that function for the auditor to audit

Vulnerability Details

L1BossBridge.sol
depositTokensToL2 Function:
It seems the amount check for the deposit limit is done on the vault's balance, but the comment mentions that it's for the DEPOSIT_LIMIT. It might be more clear if you directly check against DEPOSIT_LIMIT.

Impact

The auditor or other dev will be confused , if the comment not clear

Tools Used

None

Recommendations

The Dev should should review about his comments too, whether this will understand by other Dev or Auditor or not!

Updates

Lead Judging Commences

0xnevi Lead Judge
about 2 years ago
0xnevi Lead Judge about 2 years ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity

Support

FAQs

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