From ccf684f1bae5cc542d49fb5b46e296b6e465d9ca Mon Sep 17 00:00:00 2001
From: cui <cuiweixie@gmail.com>
Date: Fri, 15 Aug 2025 15:10:44 +0800
Subject: [PATCH] core/vm: refactor to use bitutil.TestBytes (#32434)

---
 core/vm/common.go    | 9 ---------
 core/vm/contracts.go | 3 ++-
 2 files changed, 2 insertions(+), 10 deletions(-)

diff --git a/core/vm/common.go b/core/vm/common.go
index 658803b820..2990f58972 100644
--- a/core/vm/common.go
+++ b/core/vm/common.go
@@ -84,12 +84,3 @@ func toWordSize(size uint64) uint64 {
 
 	return (size + 31) / 32
 }
-
-func allZero(b []byte) bool {
-	for _, byte := range b {
-		if byte != 0 {
-			return false
-		}
-	}
-	return true
-}
diff --git a/core/vm/contracts.go b/core/vm/contracts.go
index 21307ff5ac..d286150277 100644
--- a/core/vm/contracts.go
+++ b/core/vm/contracts.go
@@ -30,6 +30,7 @@ import (
 	"github.com/consensys/gnark-crypto/ecc/bls12-381/fp"
 	"github.com/consensys/gnark-crypto/ecc/bls12-381/fr"
 	"github.com/ethereum/go-ethereum/common"
+	"github.com/ethereum/go-ethereum/common/bitutil"
 	"github.com/ethereum/go-ethereum/core/tracing"
 	"github.com/ethereum/go-ethereum/crypto"
 	"github.com/ethereum/go-ethereum/crypto/blake2b"
@@ -289,7 +290,7 @@ func (c *ecrecover) Run(input []byte) ([]byte, error) {
 	v := input[63] - 27
 
 	// tighter sig s values input homestead only apply to tx sigs
-	if !allZero(input[32:63]) || !crypto.ValidateSignatureValues(v, r, s, false) {
+	if bitutil.TestBytes(input[32:63]) || !crypto.ValidateSignatureValues(v, r, s, false) {
 		return nil, nil
 	}
 	// We must make sure not to modify the 'input', so placing the 'v' along with