View Issue Details

IDProjectCategoryView StatusLast Update
0031619RunnerWindows (YYC)Public2020-03-04 09:14
ReporterDanAssigned ToRussell Kay 
PriorityMediumSeverityB - MajorReproducibility100%
Status ClosedResolutionFixed 
PlatformWindows YYCOSWindows 10OS Version1909
Product Version2.2.5 
Target Version2.3.0Fixed in Version2.3.0 
Summary0031619: Functions: [YYC] Doesn't always validate the number of arguments passed to script_execute(), can cause knock-on errors
DescriptionYYC doesn't always validate the number of arguments passed to a script via script_execute(), which can then can cause knock-on errors and in some cases crashes. You can see from the sample that in some cases it will catch this, but it appears to be that if the first time script_execute() is called it is given the correct number of arguments or more than is required, then YYC will attempt to always allow it thereafter. As soon as you then call the script with too few arguments it will give the wrong result.

VM will always show a code-error in-game when the script is called if the number of arguments is not correct.

Expected: YYC does the same as VM and always errors in-game if the argument count is wrong OR we at least we fix why the wrong result is being returned in this sample and we also write out a debug message every time that script_execute() is called with the wrong number of arguments but was allowed to carry on.
Steps To Reproduce1. Run the attached sample for Windows VM and observe the in-game error that your arguments are wrong for the line where one has been omitted.
2. Run it again for Windows YYC and observe the project runs, but that the second result is wrong (it's giving 4 instead of 3, for some unknown reason).
Repro: 100%
Additional InformationThis came from a YYC crash in https://yoyogames.zendesk.com/agent/tickets/166737 - have asked for the code which caused the crash and will add that seperately later, but the reporter has confirmed this issue was the root of the problem in their project.
TagsNo tags attached.
1.4 Found In
2.x Runtime Found In2.2.5.378
2.x Runtime Verified In9.9.1.1707

Activities

Dan

2020-02-12 17:46

Adminstrator  

testScriptExecuteWrongArgCount.yyz (20,950 bytes)

Russell Kay

2020-02-18 22:04

Manager   ~0065373

Fixed in gitlab