The Standard

The Standard
DeFiHardhat
20,000 USDC
View results
Submission Details
Severity: low
Invalid

LiquidationPool::stake can accept Position as callData instead of memory

Summary

Using calldata instead of memory is more gas efficient.

Vulnerability Details

LiquidationPool::stake() can accept position as callData instead of memory to save some gas.

Impact

Saving gas.

Tools Used

Manual Review

Recommendations

Pass position as call data to stake function as below.

function stake(Position calldata _position) private pure returns (uint256) {
return _position.TST > _position.EUROs ? _position.EUROs : _position.TST;
}
Updates

Lead Judging Commences

hrishibhat Lead Judge over 1 year ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity
Assigned finding tags:

informational/invalid

Support

FAQs

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