fix(esplora): use saturating_add in update_tx_graph()

This fixes overflow error when calling update_tx_graph() from update_tx_graph_without_keychain().
notmandatory · Sep 6, 2023 · c437ef0 · c437ef0
1 parent 8a46606
commit c437ef0
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/crates/esplora/src/async_ext.rs b/crates/esplora/src/async_ext.rs
@@ -261,7 +261,7 @@ impl EsploraAsyncExt for esplora_client::AsyncClient {
                     }
                 }
 
-                if last_index > last_active_index.map(|i| i + stop_gap as u32) {
+                if last_index > last_active_index.map(|i| i.saturating_add(stop_gap as u32)) {
                     break;
                 }
             }

diff --git a/crates/esplora/src/blocking_ext.rs b/crates/esplora/src/blocking_ext.rs
@@ -252,7 +252,7 @@ impl EsploraExt for esplora_client::BlockingClient {
                     }
                 }
 
-                if last_index > last_active_index.map(|i| i + stop_gap as u32) {
+                if last_index > last_active_index.map(|i| i.saturating_add(stop_gap as u32)) {
                     break;
                 }
             }
-Original file line number
+Diff line change
@@ Expand Up / @@ -261,7 +261,7 @@ impl EsploraAsyncExt for esplora_client::AsyncClient { @@
                         }
                     }
-                    if last_index > last_active_index.map(|i| i + stop_gap as u32) {
+                    if last_index > last_active_index.map(|i| i.saturating_add(stop_gap as u32)) {
                         break;
                     }
                 }
@@ Expand Down @@