HomePhabricator

tests: fix spurious windows test failures after 012598880c
f0b8afc88ba4Unpublished

Unpublished Commit ยท Learn More

Repository Importing: This repository is still importing.

Description

tests: fix spurious windows test failures after 012598880c

on rare occasions, rand() was returning duped values, causing duplicate
transactions.

BuildMerkleTree happily used these, but CPartialMerkleTree caught them and
returned a null merkle root.

Rather than taking changes with rand(), use the loop counter to guarantee
unique values.

At sipa's request, also remove the remaining uses of rand().

Details

Provenance
Cory Fields <cory-nospam-@coryfields.com>Authored on Jan 7 2015, 01:11
schancelPushed on Jan 5 2018, 21:58
schancelPushed on Jan 5 2018, 21:39
schancelPushed on Jan 5 2018, 21:17
Parents
rSTAGING3b95808428c1: Merge pull request #5520
Branches
Unknown
Tags
Unknown

Event Timeline

Cory Fields <cory-nospam-@coryfields.com> committed rSTAGINGf0b8afc88ba4: tests: fix spurious windows test failures after 012598880c (authored by Cory Fields <cory-nospam-@coryfields.com>).Jan 7 2015, 01:18