jleony/nanoreth
1
0
Fork 0
mirror of https://github.com/hl-archive-node/nanoreth.git synced 2025-12-06 10:59:55 +00:00
Code Issues Packages Projects Releases Wiki Activity

chore: use safe math when calculating missing senders (#7099)

Browse Source
This commit is contained in:
Matthias Seitz
2024-03-11 17:48:39 +01:00
committed by GitHub
parent 1156be213d
commit 82b6504ef4
1 changed files with 9 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
crates/storage/provider/src/providers/database/provider.rs
View File

@ -56,7 +56,7 @@ use std::{
sync::{mpsc, Arc},
time::{Duration, Instant},
};
use tracing::{debug, warn};
use tracing::{debug, error, warn};
/// A [`DatabaseProvider`] that holds a read-only database transaction.
pub type DatabaseProviderRO<DB> = DatabaseProvider<<DB as Database>::TX>;
@ -542,10 +542,16 @@ impl<TX: DbTxMut + DbTx> DatabaseProvider<TX> {
// NOTE: Transactions are always guaranteed to be in the database whereas
// senders might be pruned.
if senders.len() != transactions.len() {
senders.reserve(transactions.len() - senders.len());
if senders.len() > transactions.len() {
error!(target: "providers::db", senders=%senders.len(), transactions=%transactions.len(),
first_tx=%first_transaction, last_tx=%last_transaction,
"unexpected senders and transactions mismatch");
}
let missing = transactions.len().saturating_sub(senders.len());
senders.reserve(missing);
// Find all missing senders, their corresponding tx numbers and indexes to the original
// `senders` vector at which the recovered senders will be inserted.
let mut missing_senders = Vec::with_capacity(transactions.len() - senders.len());
let mut missing_senders = Vec::with_capacity(missing);
{
let mut senders = senders.iter().peekable();