Use safeTransfer and safeTransferFrom from SafeERC20

[code423n4]

Lines of code

https://github.com/code-423n4/2022-07-fractional/blob/main/src/modules/protoforms/BaseVault.sol#L65
https://github.com/code-423n4/2022-07-fractional/blob/main/src/references/TransferReference.sol#L22

Vulnerability details

Impact

Some tokens fail silently, not throwing an error. The functions safeTransfer and safeTransferFrom has checks for that returns and throw an error if not possible to transfer the tokens

Recommended Mitigation Steps

+ using SafeERC20 for IERC20;

-  IERC20(token).transfer(_to, _amount)
+  IERC20(token).safeTransfer(_to, _amount)

-  IERC20(token).transferFrom(_from, _to, _amount);
+  IERC20(token).safeTransferFrom(_from, _to, _amount);

[stevennevins]

Duplicate of #312

[HardlyDifficult]

Agree with the sponsor that this is a non-critical best practice. Transfers may fail but no events are emitted, balance doesn't change, and no other negative consequences were identified.

Merging with the warden's QA report #231