[[ Bug 19771 ]] Fix incorrect validation method for JumpIf

runrevmark · runrevmark · commit cbaade219ecf · 2017-06-15T13:18:52.000+01:00
The validator for the JumpIf bytecode was using the wrong
argument to compute the address. This has been fixed.
diff --git a/libscript/src/script-bytecode.hpp b/libscript/src/script-bytecode.hpp
@@ -121,7 +121,7 @@ struct MCScriptBytecodeOp_JumpIf
 		ctxt.CheckArity(2);
 		ctxt.CheckRegister(ctxt.GetArgument(0));
 		ctxt.CheckAddress(ctxt.GetAddress() +
-						  ctxt.GetSignedArgument(0));
+						  ctxt.GetSignedArgument(1));
 	}
 	
 	static void Execute(MCScriptExecuteContext& ctxt)

Original file line number	Diff line number	Diff line change
`@@ -121,7 +121,7 @@ struct MCScriptBytecodeOp_JumpIf`
`121`	`121`	`ctxt.CheckArity(2);`
`122`	`122`	`ctxt.CheckRegister(ctxt.GetArgument(0));`
`123`	`123`	`ctxt.CheckAddress(ctxt.GetAddress() +`
`124`		`- ctxt.GetSignedArgument(0));`
	`124`	`+ ctxt.GetSignedArgument(1));`
`125`	`125`	`}`
`126`	`126`
`127`	`127`	`static void Execute(MCScriptExecuteContext& ctxt)`