Skip to content

Commit 605201e

Browse files
committed
Constructor sanity check prevented creation of fully one-sided curves (pointed out by @MiloTruck)
1 parent 6e8a8c8 commit 605201e

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

src/EulerSwap.sol

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -85,8 +85,8 @@ contract EulerSwap is IEulerSwap, EVCUtil, UniswapHook {
8585
s.reserve1 = initialState.currReserve1;
8686

8787
require(CurveLib.verify(p, s.reserve0, s.reserve1), CurveLib.CurveViolation());
88-
require(!CurveLib.verify(p, s.reserve0 > 0 ? s.reserve0 - 1 : 0, s.reserve1), CurveLib.CurveViolation());
89-
require(!CurveLib.verify(p, s.reserve0, s.reserve1 > 0 ? s.reserve1 - 1 : 0), CurveLib.CurveViolation());
88+
if (s.reserve0 != 0) require(!CurveLib.verify(p, s.reserve0 - 1, s.reserve1), CurveLib.CurveViolation());
89+
if (s.reserve1 != 0) require(!CurveLib.verify(p, s.reserve0, s.reserve1 - 1), CurveLib.CurveViolation());
9090

9191
// Configure external contracts
9292

0 commit comments

Comments
 (0)