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

Data type discrepancy between v1 and v2

Summary

S_feePrecision is differing in the two contract versions

Vulnerability Details

Changing variables is unsafe practice.

Ln 96 in ThunderLoan.sol (v1) : uint256 private s_feePrecision;
Ln 97 in ThunderLoan.sol(v2) : uint256 public constant FEE_PRECISION = 1e18;

Notice the difference (private vs public constant)

Impact

Low impact, but could open up to exploits potentially

Tools Used

Manual inspection

Recommendations

Make the variables match

Updates

Lead Judging Commences

0xnevi Lead Judge
almost 2 years ago
0xnevi Lead Judge almost 2 years ago
Submission Judgement Published
Invalidated
Reason: Other

Support

FAQs

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