Merge pull request #1233 from nspcc-dev/fix/gasverify

core: restrict maximum gas allow for verification

pkg/core/blockchain.go

@@ -1305,6 +1305,11 @@ func (bc *Blockchain) verifyHashAgainstScript(hash util.Uint160, witness *transa
 		return err
 	}
 
+	gasPolicy := bc.contracts.Policy.GetMaxVerificationGas(interopCtx.DAO)
+	if gas > gasPolicy {
+		gas = gasPolicy
+	}
+
 	vm := SpawnVM(interopCtx)
 	vm.SetPriceGetter(getPrice)
 	vm.GasLimit = gas

pkg/core/native/policy.go

@@ -25,6 +25,7 @@ const (
 	defaultMaxBlockSize            = 1024 * 256
 	defaultMaxTransactionsPerBlock = 512
 	defaultFeePerByte              = 1000
+	defaultMaxVerificationGas      = 50000000
 )
 
 var (
@@ -51,6 +52,7 @@ type Policy struct {
 	maxTransactionsPerBlock uint32
 	maxBlockSize            uint32
 	feePerByte              int64
+	maxVerificationGas      int64
 }
 
 var _ interop.Contract = (*Policy)(nil)
@@ -149,6 +151,7 @@ func (p *Policy) Initialize(ic *interop.Context) error {
 	p.maxTransactionsPerBlock = defaultMaxTransactionsPerBlock
 	p.maxBlockSize = defaultMaxBlockSize
 	p.feePerByte = defaultFeePerByte
+	p.maxVerificationGas = defaultMaxVerificationGas
 
 	return nil
 }
@@ -221,6 +224,11 @@ func (p *Policy) GetFeePerByteInternal(dao dao.DAO) int64 {
 	return p.getInt64WithKey(dao, feePerByteKey)
 }
 
+// GetMaxVerificationGas returns maximum gas allowed to be burned during verificaion.
+func (p *Policy) GetMaxVerificationGas(_ dao.DAO) int64 {
+	return p.maxVerificationGas
+}
+
 // getBlockedAccounts is Policy contract method and returns list of blocked
 // accounts hashes.
 func (p *Policy) getBlockedAccounts(ic *interop.Context, _ []stackitem.Item) stackitem.Item {