use safeTransfer when moving protocol fees, for non-compliant tokens

hoytech · hoytech · commit 3d924ebb532b · 2025-04-29T10:44:08.000-04:00
diff --git a/src/libraries/FundsLib.sol b/src/libraries/FundsLib.sol
@@ -79,7 +79,7 @@ library FundsLib {
             uint256 protocolFeeAmount = feeAmount * p.protocolFee / 1e18;
 
             if (protocolFeeAmount != 0) {
-                IERC20(asset).transfer(p.protocolFeeRecipient, protocolFeeAmount);
+                IERC20(asset).safeTransfer(p.protocolFeeRecipient, protocolFeeAmount);
                 amount -= protocolFeeAmount;
                 feeAmount -= protocolFeeAmount;
             }

Original file line number	Diff line number	Diff line change
`@@ -79,7 +79,7 @@ library FundsLib {`
`79`	`79`	`uint256 protocolFeeAmount = feeAmount * p.protocolFee / 1e18;`
`80`	`80`
`81`	`81`	`if (protocolFeeAmount != 0) {`
`82`		`- IERC20(asset).transfer(p.protocolFeeRecipient, protocolFeeAmount);`
	`82`	`+ IERC20(asset).safeTransfer(p.protocolFeeRecipient, protocolFeeAmount);`
`83`	`83`	`amount -= protocolFeeAmount;`
`84`	`84`	`feeAmount -= protocolFeeAmount;`
`85`	`85`	`}`