NoOne - use `safeMint` instead `mint`

[sherlock-admin4]

NoOne

medium

use safeMint instead mint

Summary

In ArrakisMetaVaultFactory.sol function deployPrivateVault use mint instead safeMint

Vulnerability Detail

Impact

Using ERC721::_mint() can mint ERC721 tokens to addresses which don't support ERC721 tokens, while ERC721::_safeMint() ensures that ERC721 tokens are only minted to addresses which support them.

Code Snippet

https://github.com/sherlock-audit/2024-03-arrakis/blob/main/arrakis-modular/src/ArrakisMetaVaultFactory.sol#L245

Tool used

Manual Review

Recommendation

use safeMint instead mint