signer: avoid large value copy inside loop

This commit is contained in:
9547 2024-06-10 22:29:58 +08:00
parent 2eb185c92b
commit 567630dfc9

View file

@ -243,13 +243,14 @@ func (args *SendTxArgs) validateTxSidecar() error {
// Generate commitment and proof. // Generate commitment and proof.
commitments := make([]kzg4844.Commitment, n) commitments := make([]kzg4844.Commitment, n)
proofs := make([]kzg4844.Proof, n) proofs := make([]kzg4844.Proof, n)
for i, b := range args.Blobs { for i := range args.Blobs {
c, err := kzg4844.BlobToCommitment(&b) b := &args.Blobs[i]
c, err := kzg4844.BlobToCommitment(b)
if err != nil { if err != nil {
return fmt.Errorf("blobs[%d]: error computing commitment: %v", i, err) return fmt.Errorf("blobs[%d]: error computing commitment: %v", i, err)
} }
commitments[i] = c commitments[i] = c
p, err := kzg4844.ComputeBlobProof(&b, c) p, err := kzg4844.ComputeBlobProof(b, c)
if err != nil { if err != nil {
return fmt.Errorf("blobs[%d]: error computing proof: %v", i, err) return fmt.Errorf("blobs[%d]: error computing proof: %v", i, err)
} }
@ -258,8 +259,8 @@ func (args *SendTxArgs) validateTxSidecar() error {
args.Commitments = commitments args.Commitments = commitments
args.Proofs = proofs args.Proofs = proofs
} else { } else {
for i, b := range args.Blobs { for i := range args.Blobs {
if err := kzg4844.VerifyBlobProof(&b, args.Commitments[i], args.Proofs[i]); err != nil { if err := kzg4844.VerifyBlobProof(&args.Blobs[i], args.Commitments[i], args.Proofs[i]); err != nil {
return fmt.Errorf("failed to verify blob proof: %v", err) return fmt.Errorf("failed to verify blob proof: %v", err)
} }
} }