When creating new safes, in either subAccounts or main console accounts, a nonce is used to create the salt used by the SafeDeployer:
SafeDeployer::_createSafe and _genNonce:
If the wallet creation fails, it retries with a new nonce (and thus a new salt) until it succeeds. This opens up an unlikely, but possible, attack vector where a rich malevolent actor could front run wallet creation by creating a lot of wallets increasing the gas cost for the victim due to the iterations.
This of course costs even more gas for the attacker as they need to actually create the wallets making this attack unlikely; but its still possible.
Since the wallets created by the attacker would be identical, a front run here doesnt really do anything else than what the victim wanted; creating a wallet. Hence, the wallet created by a possible attacker would work just as well for the victim.
Consider adding a boolean flag if the user wishes to iterate over nonces when creating a safe. That way, a user can opt-in to having the looping logic or not.
