The Standard

The Standard
DeFiHardhat
20,000 USDC
View results
Submission Details
Severity: high
Valid

Contract can be Dos in `LiquidationPool.sol`::`distributeFees` due to unbounded loop

Summary

Contract can be Dos in LiquidationPool.sol::distributeFees due to unbounded loop in distributeFees and pendingStakes

Vulnerability Details

The unbounded loops in distributeFees iterate over all holders and pendingStakes. If these arrays get very large, the function execution could run out of gas or exceed the block gas limit, making it fail or become unusable.

Impact

the function execution could run out of gas or exceed the block gas limit, making it fail or become unusable.

Tools Used

Manual Review

Recommendations

Not a simple solution to implement with the current codebase. Use offchain computation

Updates

Lead Judging Commences

hrishibhat Lead Judge over 1 year ago
Submission Judgement Published
Validated
Assigned finding tags:

pendingstake-dos

hrishibhat Lead Judge over 1 year ago
Submission Judgement Published
Validated
Assigned finding tags:

pendingstake-high

Support

FAQs

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