Merge pull request #1467 from filecoin-project/fix/charge-gas

use charge gas safe in places we return errors
filecoin-project · Mar 27, 2020 · 6dddc42 · 6dddc42
2 parents 4dcb810 + 19553a6
commit 6dddc42
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/chain/vm/vm.go b/chain/vm/vm.go
@@ -210,7 +210,9 @@ func (vm *VM) send(ctx context.Context, msg *types.Message, parent *Runtime,
 		}()
 	}
 
-	rt.ChargeGas(rt.Pricelist().OnMethodInvocation(msg.Value, msg.Method))
+	if aerr := rt.chargeGasSafe(rt.Pricelist().OnMethodInvocation(msg.Value, msg.Method)); aerr != nil {
+		return nil, aerrors.Wrap(aerr, "not enough gas for method invocation"), rt
+	}
 
 	toActor, err := st.GetActor(msg.To)
 	if err != nil {