sherlock-audit / 2022-11-telcoin-judging

0 stars 0 forks source link

Met - Warning - should submit work without fee? #71

Closed sherlock-admin closed 1 year ago

sherlock-admin commented 1 year ago

Met

informational

Warning - should submit work without fee?

Summary

This is just informational as I don't know exactly how 1 inch, or the "owner" of FeeBuyback works.

Is it possible that the owner calls submit without providing the eth for the fee? Perhaps the fee can be rounded to 0 for some very small amounts? What if it is (later becomes) desirable to make swaps w/o a fee? Is the swap then going to return false because of 0 amount and revert the whole transaction?

Vulnerability Detail

Impact

Code Snippet

https://github.com/sherlock-audit/2022-11-telcoin/blob/main/contracts/fee-buyback/FeeBuyback.sol#L77-L78

Tool used

Manual Review

Recommendation