[Malleability C] hotstuff CertifiedBlock

[UlyanaAndrukhiv]

CertifiedBlock is a Block with an acquired quorum. As an implementation of the ID() method is wrong, a byzantine consensus node can change the quorum certificate without being noticed by other nodes. The usage of this data structure should be reviewed to be proven safe. The ID() might be changed by Hash() function which hashes the quorum certificate too.

type CertifiedBlock struct {
	Block        *Block
	CertifyingQC *flow.QuorumCertificate
}

func (b *CertifiedBlock) ID() flow.Identifier {
	return b.Block.BlockID
}

type QuorumCertificate struct {
	View    uint64
	BlockID Identifier
	SignerIndices []byte
	SigData []byte
}