#58 AssertResults array not checked for upper bounds

v1.1.0.0
wont-fix
aliazzz
None
2022-12-10
2022-02-08
srigan1986
No

I use the CfUnit testframework (not yet CoUnit) and run loop over indexed test cases with TRY, CATCH on the testee FB. At some point of time, exception occured and it was showing the method AssAssertResult as the culprit for exception - RTS_ACCESS_VIOLATION.

I suspect if it could be a phenomenon when the number of asserts per testsuite exceed 1000 (MaxNumberOfAssertsForEachTestSuite).
I have attached screenshots of code snippets. Please verify if it could be the cause of exception and would be glad to hear from you.

2 Attachments

Discussion

  • hermsen

    hermsen - 2022-02-08

    Thank you for using cfUnit.
    Sadly cfunit v1.1.0.0 will not receive bugfixes as it has been superseeded by coUnit. A slightly different name will only bring you some minor code changes if you use refactoring wisely, however coUnit v1.2.0.0 also brings new features, improvements and bugfixes over its predecessor. So step 1 is to update to coUnit.

     
  • hermsen

    hermsen - 2022-02-13

    Since I have not received feedback and you make use of an older version, I asume this has been resolved. If somehow your issue persists in the latest version ( v1.2) you can open a new ticket.

     

    Last edit: hermsen 2022-02-15
  • hermsen

    hermsen - 2022-02-15
    • status: open --> wont-fix
    • private: Yes --> No
     
  • sgronchi - 2022-11-29

    I had the same problem just today with coUnit v1.2.0.0.

    Library parameters are not editable so I cannot increment the related array size.

     
  • hermsen

    hermsen - 2022-12-10
    • discussion: enabled --> disabled
     
  • hermsen

    hermsen - 2022-12-10

    The library is open source, you can edit the source code if you wish.

     
    👍
    1
  • hermsen

    hermsen - 2022-12-10
    • discussion: disabled --> enabled
     

Log in to post a comment.