Merge pull request #2213 from dusk-network/2089-consensus-candidate-m…

…essages-do-not-sign-consensusheader Add ConsensusHeader to Candidate signature
dusk-network · Sep 7, 2024 · d1ef5fa · d1ef5fa
2 parents 004e58e + e798025
commit d1ef5fa
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/node-data/src/message.rs b/node-data/src/message.rs
@@ -1410,7 +1410,11 @@ impl StepMessage for Candidate {
         &mut self.sign_info
     }
     fn signable(&self) -> Vec<u8> {
-        self.candidate.header().hash.to_vec()
+        let mut signable = self.header.signable();
+        signable.extend_from_slice(Self::SIGN_SEED);
+        let candidate_hash = self.candidate.header().hash.to_vec();
+        signable.extend_from_slice(&candidate_hash);
+        signable
     }
     fn header(&self) -> &ConsensusHeader {
         &self.header