Merge pull request #20 from penumbra-zone/clamp_reserves

Clamp reserves posted in positions to 80-bits
penumbra-zone · Jul 26, 2023 · 753131b · 753131b
2 parents 1aac7b5 + 90049d9
commit 753131b
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/src/trader.rs b/src/trader.rs
@@ -332,8 +332,10 @@ where
         // assets. so we should really have _half the total available reserves divided
         // amongst the positions involving that asset_
         let reserves = Reserves {
-            r1: reserves_1 / 2u32.into(),
-            r2: reserves_2 / 2u32.into(),
+            // We need to clamp the reserves to 80 bits, which is the max the node
+            // software can handle.
+            r1: (reserves_1 / 2u32.into()).clamp(0u32.into(), (u128::pow(2, 80) - 1).into()),
+            r2: (reserves_2 / 2u32.into()).clamp(0u32.into(), (u128::pow(2, 80) - 1).into()),
         };
 
         if reserves.r1 == 0u32.into() && reserves.r2 == 0u32.into() {