Use call instead of transfer

ChorusOne · Jan 4, 2024 · 45442bf · 45442bf
1 parent 8f95f93
commit 45442bf
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/src/FeeRewardsManager.sol b/src/FeeRewardsManager.sol
@@ -38,7 +38,8 @@ library CalculateAndSendRewards {
         );
         // This can be used to call this contract again (reentrancy)
         // but since all funds from this contract are used for the owner
-        payable(owner).transfer(ownerAmount);
+        (bool ownerSent, ) = payable(owner).call{value: ownerAmount}("");
+        require(ownerSent, "Failed to send Ether back to owner contract");
         (bool sent, ) = payable(withdrawalCredential).call{
             value: returnedAmount
         }("");