The Standard

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

Update `positions[msg.sender].TST` and `positions[msg.sender].EUROs` in `LiquidationPool::decreasePosition()` function before the transfer makes it more resilient to re-entrancy

Summary

Updating the positions[msg.sender].TST and positions[msg.sender].EUROs in LiquidationPool::decreasePosition() function before the transfer makes it more resilient to re-entrancy.

if (_tstVal > 0) {
+ positions[msg.sender].TST -= _tstVal;
IERC20(TST).safeTransfer(msg.sender, _tstVal);
- positions[msg.sender].TST -= _tstVal;
}
if (_eurosVal > 0) {
+ positions[msg.sender].EUROs -= _eurosVal;
IERC20(EUROs).safeTransfer(msg.sender, _eurosVal
- positions[msg.sender].EUROs -= _eurosVal;
}
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.