🐞Fix safeMoveETH (#1475)

* fix testSafeMoveETHViaMover

* fix safeMoveETH

Author: ZumZoom

src/utils/SafeTransferLib.sol

@@ -212,7 +212,8 @@ library SafeTransferLib {
                 }
                 if extcodesize(mover) {
                     let balanceBefore := balance(to) // Check via delta, in case `SELFDESTRUCT` is bricked.
-                    pop(call(gas(), mover, amount, codesize(), 0x00, codesize(), 0x00))
+                    mstore(0x00, to)
+                    pop(call(gas(), mover, amount, 0x00, 0x20, codesize(), 0x00))
                     if iszero(lt(add(amount, balance(to)), balanceBefore)) { break }
                     if lt(selfBalanceBefore, selfbalance()) { invalid() } // Just in case.
                 }

test/SafeTransferLib.t.sol

@@ -1203,7 +1203,7 @@ contract SafeTransferLibTest is SoladyTest {
         assertEq(to.balance, amount0 + amount1);
     }
 
-    function safeMoveETHViaMover(bytes32) public {
+    function testSafeMoveETHViaMover(bytes32) public {
         _deployETHMover();
 
         address to = _randomHashedAddress();
@@ -1219,12 +1219,12 @@ contract SafeTransferLibTest is SoladyTest {
         assertEq(address(this).balance, selfBalanceBefore - amount0);
 
         if (SafeTransferLib.ETH_MOVER.code.length == 0) {
-            address vault = this.safeMoveETH(to, amount0);
+            address vault = this.safeMoveETH(to, amount1);
             assertEq(vault.balance, amount1);
             assertEq(to.balance, amount0);
-            assertEq(address(this).balance, selfBalanceBefore - amount0);
+            assertEq(address(this).balance, selfBalanceBefore - amount0 - amount1);
         } else {
-            assertEq(this.safeMoveETH(to, amount0), address(0));
+            assertEq(this.safeMoveETH(to, amount1), address(0));
             assertEq(to.balance, amount0 + amount1);
             assertEq(address(this).balance, selfBalanceBefore - amount0 - amount1);
         }