Alchemix Transmuter

Summary
Deprecated in favor of safeIncreaseAllowance() and safeDecreaseAllowance(). If only setting the initial allowance to the value that means infinite, safeIncreaseAllowance() can be used instead

Vulnerability Details

https://github.com/Cyfrin/2024-12-alchemix/blob/main/src/StrategyArb.sol#L27

https://github.com/Cyfrin/2024-12-alchemix/blob/main/src/StrategyArb.sol#L37

https://github.com/Cyfrin/2024-12-alchemix/blob/main/src/StrategyArb.sol#L44

https://github.com/Cyfrin/2024-12-alchemix/blob/main/src/StrategyMainnet.sol#L35

https://github.com/Cyfrin/2024-12-alchemix/blob/main/src/StrategyMainnet.sol#L45

https://github.com/Cyfrin/2024-12-alchemix/blob/main/src/StrategyOp.sol#L27

https://github.com/Cyfrin/2024-12-alchemix/blob/main/src/StrategyOp.sol#L39

https://github.com/Cyfrin/2024-12-alchemix/blob/main/src/StrategyOp.sol#L50

Tools Used

Manual

Recommendations

witching to safeIncreaseAllowance() and safeDecreaseAllowance() is safer and future-proof.