Open code423n4 opened 3 years ago
0xsanson
The claimEscrow(...) function in Liquidation.sol can be set external instead of public since it's not used in the contract. (code clarity and gas savings)
https://github.com/code-423n4/2021-06-tracer/blob/main/src/contracts/Liquidation.sol#L109
Manual analysis
Change public to external.
Handle
0xsanson
Vulnerability details
Impact
The claimEscrow(...) function in Liquidation.sol can be set external instead of public since it's not used in the contract. (code clarity and gas savings)
Proof of Concept
https://github.com/code-423n4/2021-06-tracer/blob/main/src/contracts/Liquidation.sol#L109
Tools Used
Manual analysis
Recommended Mitigation Steps
Change public to external.