TcUnit icon indicating copy to clipboard operation
TcUnit copied to clipboard

AddAssertResult not checking limits of AssertResults array

Open doerk77 opened this issue 3 years ago • 1 comments

Describe the bug When increasing TotalAsserts in method AddAssertResult the array limits of AssertResults are not being checked, causing PLC crash due to invalid array index.

Expected behavior Prevent crash by limit checks and sending error through ADSLOGSTR. Remark: Prevent overflow of event logger (error should only be raised once)!!!

Screenshots image

Software versions v1.2.0.0

has been mentioned earlier in #153

doerk77 avatar Jun 15 '22 09:06 doerk77

Pull request welcome.

sagatowski avatar Jun 20 '22 05:06 sagatowski

Solved in #193 & in commit 4a48bdd

sagatowski avatar Nov 30 '23 12:11 sagatowski