Cardinal-Cryptography / AlephBFT

Rust implementation of Aleph consensus protocol
https://cardinal-cryptography.github.io/AlephBFT/index.html
Apache License 2.0
56 stars 28 forks source link

A0-4006: Check initial unit control hashes early #417

Closed timorleph closed 8 months ago

timorleph commented 8 months ago

Control hashes of initial units have to refer to the empty parent set, so their combined hash is actually known a priori and can be checked early. This also changes one test that used to assume a initial unit with a broken hash can be passed into consensus.

The only connection to A0-4006 is that the test in question failed after the refactor, since I didn't want to react to this incongruous situation internally.

github-actions[bot] commented 8 months ago

Please make sure the following happened