fix: Compute EigenDA preimage hash using preimage length - go critic fix

Layr-Labs · Aug 15, 2024 · bfd2b0a · bfd2b0a
1 parent fc4808c
commit bfd2b0a
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/eigenda/types.go b/eigenda/types.go
@@ -27,13 +27,13 @@ type EigenDABlobInfo struct {
 Unlike 4844 there's no need to inject a version byte into the 0th offset of the hash
 */
 func (e *EigenDABlobInfo) PreimageHash() (*common.Hash, error) {
-	kzgCommit, err := e.SerializeCommitment()
+	bytes, err := e.SerializeCommitment()
 	if err != nil {
 		return nil, err
 	}
 
-	data := append(kzgCommit, uint32ToBytes(e.BlobHeader.DataLength)...)
-	dataHash := crypto.Keccak256Hash(data)
+	bytes = append(bytes, uint32ToBytes(e.BlobHeader.DataLength)...)
+	dataHash := crypto.Keccak256Hash(bytes)
 
 	return &dataHash, nil
 }