Submission Details
Severity:
Remove `uint160` when casting to address
GitHub
https://github.com/Cyfrin/2024-07-biconomy/blob/9590f25cd63f7ad2c54feb618036984774f3879d/contracts/lib/ExecLib.sol#L40
Summary
The decodeDelegateCall function includes an unnecessary cast to uint160 before casting to address.
Impact
Redundant casting can cause confusion and does not improve functionality, potentially leading to misunderstandings.
Recommendation
Remove the unnecessary uint160 cast and directly cast bytes20 to address.
Prize pool breakdown
Total prize
30,000 USDC
nSLOC
1,43321 USDC / LOC
25,000 USDC
2,750 USDC
2,250 USDC
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.