draft date: Apr 9
Low
[L02] fund admin can use depositFromWithPermit to deposit to arbitrary address
In depositFromWithPermit, the fund admin address can trigger a "deposit" if a user signs a permit signature. However, the recipient is chosen by the admin (msg.sender), not decided or signed by the user.
Given that the admin role already has a lot of power controlling the token balances, this doesn't pose additional risk to the users. However, it adds an unnecessary trust assumption in the deposit step and might make this function less useful.
Suggestion: consider removing the recipient input and just deposit to from to minimize trust assumptions. Or add another signature validation similar to "signed withdrawal".