fix(BalancerV3Executor): Do data.length check at the beginning of swap

Took 16 minutes
2025-06-06 17:24:07 +01:00
parent 941879d5d6
commit 4f58977007
2 changed files with 5 additions and 6 deletions
--- a/foundry/test/executors/BalancerV3Executor.t.sol
+++ b/foundry/test/executors/BalancerV3Executor.t.sol
@@ -68,7 +68,7 @@ contract BalancerV3ExecutorTest is Constants, TestUtils {
        assertEq(receiver, BOB);
    }

-    function testDecodeParamsInvalidDataLength() public {
+    function testSwapInvalidDataLength() public {
        bytes memory invalidParams = abi.encodePacked(
            osETH_ADDR,
            waEthWETH_ADDR,
@@ -77,7 +77,7 @@ contract BalancerV3ExecutorTest is Constants, TestUtils {
        );

        vm.expectRevert(BalancerV3Executor__InvalidDataLength.selector);
-        balancerV3Exposed.decodeParams(invalidParams);
+        balancerV3Exposed.swap(1 ether, invalidParams);
    }

    function testSwap() public {