TempleGold
TempleDAO
Foundry
25,000 USDC
View results
Previous Next
Submission Details
Severity: medium
Invalid

Fee on transfer token calculation is incorrect

ke1caM

Vulnerability details

Fee on Transfer token is a token that takes a percentage of sent amount as a fee.

Bid function in SpiceAuction contract correctly calculates the amount of received tokens by recipient. However the following check leads to denial of service every time these tokens are used.

uint256 _bidTokenAmountBefore = IERC20(bidToken).balanceOf(_recipient);
IERC20(bidToken).safeTransferFrom(msg.sender, _recipient, amount);
uint256 _bidTokenAmountAfter = IERC20(bidToken).balanceOf(_recipient);
// fee on transfer tokens
if (amount != _bidTokenAmountAfter - _bidTokenAmountBefore) { revert CommonEventsAndErrors.InvalidParam(); }

When user places a bid the amount of tokens are transfered from his address. When this token is fee on tranfer token, a fee will be subtracted from that amount. The recipient will receive amount - fee tokens. _bidTokenAmountAfter - _bidTokenAmountBefore will always be different (smaller) than amount. This function will always revert for fee on transfer tokens.

Impact

Spice auction will not work with fee on transfer tokens. Current implementation of bid function leads to DoS every time a fee on transfer token is used.

Recommended Mitigation Steps

Here is a pseudocode with possible fix:

function bid(uint256 amount) external virtual override {
/// @dev Cache, gas savings
uint256 epochId = _currentEpochId;
EpochInfo storage info = epochs[epochId];
​
if(!info.isActive()) { revert CannotDeposit(); }
if (amount == 0) { revert CommonEventsAndErrors.ExpectedNonZero(); }
​
SpiceAuctionConfig storage config = auctionConfigs[epochId];
(address bidToken,) = _getBidAndAuctionTokens(config);
address _recipient = config.recipient;
uint256 _bidTokenAmountBefore = IERC20(bidToken).balanceOf(_recipient);
IERC20(bidToken).safeTransferFrom(msg.sender, _recipient, amount);
uint256 _bidTokenAmountAfter = IERC20(bidToken).balanceOf(_recipient);
// fee on transfer tokens
- if (amount != _bidTokenAmountAfter - _bidTokenAmountBefore) { revert CommonEventsAndErrors.InvalidParam(); }
- depositors[msg.sender][epochId] += amount;
+ amount = _bidTokenAmountAfter - _bidTokenAmountBefore;
+ depositors[msg.sender][epochId] += amount;
​
​
info.totalBidTokenAmount += amount;
emit Deposit(msg.sender, epochId, amount);
}
Updates

Lead Judging Commences

inallhonesty Lead Judge over 1 year ago
Submission Judgement Published
Invalidated
Reason: Known issue

Support

FAQs

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