Dria

Summary

Consider using safeTransfer/safeTransferFrom instead of transfer/transferFrom.

Vulnerability Details

Using safeTransfer() instead of transfer() is a best practice in Solidity because safeTransfer() includes extra checks and error handling that ensure compatibility with both standard and non-standard ERC20 tokens.

Impact

transfer() may not revert on failure while safeTransfer() reverts if the transfer fails. Using safeTransfer() ensures that the contract is robust and compatible with a broader range of ERC20 tokens, reducing the risk of failed transfers and enhancing security.

Tools Used

Manual audit

Recommendations

Import SafeERC20 library from OpenZeppelin:

Use safeTransfer() / safeTransferFrom() with IERC20 casting: