github/reth
1
1
Fork 0
mirror of https://github.com/paradigmxyz/reth.git synced 2026-02-10 06:55:10 -05:00
Code Issues Packages Projects Releases Wiki Activity

fix: ambiguous deposit mint value in arbitrary (#9216)

Browse Source
This commit is contained in:
Roman Krasiuk
2024-07-01 04:26:44 -07:00
committed by GitHub
parent e6842fb8bd
commit 068bf57c06
1 changed files with 10 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
crates/primitives/src/transaction/mod.rs
View File

@@ -1478,6 +1478,16 @@ impl<'a> arbitrary::Arbitrary<'a> for TransactionSigned {
if tx_eip_4844.to != Address::default() { Some(()) } else { None };
}
#[cfg(feature = "optimism")]
// Both `Some(0)` and `None` values are encoded as empty string byte. This introduces
// ambiguity in roundtrip tests. Patch the mint value of deposit transaction here, so that
// it's `None` if zero.
if let Transaction::Deposit(ref mut tx_deposit) = transaction {
if tx_deposit.mint == Some(0) {
tx_deposit.mint = None;
}
}
let signature = Signature::arbitrary(u)?;
#[cfg(feature = "optimism")]