use safe erc20 functions

ERC721Payable.handlePayment core-contracts/contracts/ERC721Payable.sol50-56 ignores return value by payableToken.transferFrom core-contracts/contracts/ERC721Payable.sol54 consider safeTransferFrom to prevent any locks or loss of funds --- The text was updated successfully, but these errors were...

User can mint without paying if the token used doesn't revert on failure

Lines of code Vulnerability details Impact User can mint and if he doesn't have enough tokens the transfer will fail and if the token doesn't revert it will return false and the minting will continue. The user will mint everything for free. Proof of Concept Some ERC20 tokens like USDT don't rever...

Possible problem when mintToken and the user wants to purchase a token

Lines of code Vulnerability details Impact A User can mint tokens even if the payment doesn't succeed. Proof of Concept the contract ERC721Payable uses erc20 to handle payments. When this contract handles the payment of the user , calls the function transferFrom from the ERC20 payableToken variab...