zecfoundation
/
zcashd
mirror of https://github.com/ZcashFoundation/zcashd.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

rust: Explicitly return null hash for pre-v5 auth digests 

Co-authored-by: Daira Hopwood <daira@jacaranda.org>
This commit is contained in:
str4d 2021-06-14 17:47:03 +01:00 committed by Jack Grigg
parent 0a4c4fa721
commit 7195b72ce2
1 changed files with 5 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
src/rust/src/transaction_ffi.rs
View File

@ -43,9 +43,11 @@ pub extern "C" fn zcash_transaction_digests(
} }
if let Some(auth_digest_ret) = unsafe { auth_digest_ret.as_mut() } { if let Some(auth_digest_ret) = unsafe { auth_digest_ret.as_mut() } {
match tx.version() { match tx.version() {
// Pre-NU5 transaction formats don't have auth digests; leave it empty so we // Pre-NU5 transaction formats don't have authorizing data commitments; when
// don't need to know the correct consensus branch ID for them. // included in the authDataCommitment tree, they use the null hash.
TxVersion::Sprout(_) | TxVersion::Overwinter | TxVersion::Sapling => (), TxVersion::Sprout(_) | TxVersion::Overwinter | TxVersion::Sapling => {
*auth_digest_ret = [0; 32]
}
_ => auth_digest_ret.copy_from_slice(tx.auth_commitment().as_bytes()), _ => auth_digest_ret.copy_from_slice(tx.auth_commitment().as_bytes()),
} }
} }