QuantAMM

QuantAMM
49,600 OP
View results
Submission Details
Severity: high
Valid

loss of ownerFee as there is no way to retrieve them

Summary

In UpliftOnlyExample::onAfterSwap there is fees paid to owner that sent to the UpliftOnlyExample contract that has no function to retrieve the fees from it leading to fees being stuck in the contract forever

Vulnerability Details

In UpliftOnlyExample::onAfterSwap if it is ownerFee it being sent to the address of the router which has no function to take out those fees

File: UpliftOnlyExample.sol
342: if (ownerFee > 0) {
343: _vault.sendTo(feeToken, address(this), ownerFee);

Impact

  • Loss of funds

  • funds stuck in the contract forever

Tools Used

manual review

Recommendations

implement a function to retrieve the fees

Updates

Lead Judging Commences

n0kto Lead Judge 10 months ago
Submission Judgement Published
Validated
Assigned finding tags:

finding_ownerFee_cannot_be_withdrawn

Likelihood: High, every swap. Impact: High, funds are stuck.

Support

FAQs

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

Give us feedback!