feat: allow at most 4 inputs for Bn256Pairing precompile (#581)

* feat: allow at most 4 inputs for `Bn256Pairing` precompile * mv core/vm/testdata/precompiles/bn256Pairing.json core/vm/testdata/precompiles/fail-bn256Pairing.json
scroll-tech · Nov 24, 2023 · 17bd8b2 · 17bd8b2
1 parent 563e36e
commit 17bd8b2
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 0 deletions.
diff --git a/core/vm/contracts.go b/core/vm/contracts.go
@@ -585,6 +585,10 @@ var (
 // runBn256Pairing implements the Bn256Pairing precompile, referenced by both
 // Byzantium and Istanbul operations.
 func runBn256Pairing(input []byte) ([]byte, error) {
+	// Allow at most 4 inputs
+	if len(input) > 4*192 {
+		return nil, errBadPairingInput
+	}
 	// Handle some corner cases cheaply
 	if len(input)%192 > 0 {
 		return nil, errBadPairingInput

diff --git a/...vm/testdata/precompiles/bn256Pairing.json → ...stdata/precompiles/fail-bn256Pairing.json b/...vm/testdata/precompiles/bn256Pairing.json → ...stdata/precompiles/fail-bn256Pairing.json