521551 - recursion testcases are crashing on windows mobile

Status: VERIFIED FIXED

(Tamarin Graveyard :: Virtual Machine, defect, P3)

Description

[Brent Baker]

Testcases added for bug 502589 are crashing on windows mobile.

test/acceptance/recursion/*

Comment 1

[Brent Baker]

marked recursion/* testcases as expectedfail on windows mobile via changeset 2757:4baea8cc222c

Comment 2

[Brent Baker]

Rick: 
This appears to be a JIT issue as -Dinterp results in 
'Error: Error #1023', whereas we get a crash otherwise.

Its unclear which change created the failure since Lars 
probably tested on versions earlier than my recent 
ARM changes and the buildbot hasn't been running 
to completion for a couple of days.

In any event, its most likely something I've done, so
I suggest we disable these tests until I can resolve it.

Comment 3

[Rick Reitmaier]

I've gone back to Sep 18th [d18863e7f76c] and am still able to see the crash (even after applying the patch).

I'm starting to suspect that this issue has been in existence for quite a while and that the newly introduced test cases are just now tickling it.

Comment 4

[Lars T Hansen]

(In reply to comment #3)
> I've gone back to Sep 18th [d18863e7f76c] and am still able to see the crash
> (even after applying the patch).
> 
> I'm starting to suspect that this issue has been in existence for quite a while
> and that the newly introduced test cases are just now tickling it.

Likely, and it's my fault - some of these test cases have really deep recursions, but I neglected to fix whether the checks for all those recursions have actually landed!  Ouch.  There are further PCRE fixes pending for bug #502589.  Disabling the tests on winmo is the correct thing to do for the time being.

Comment 5

[Lars T Hansen]

Recursion tests, plus rewritten test cases, landed as redux changesets:

changeset:   2886:d1e23d56e9c7 (test cases)
changeset:   2887:fec599faa108 (pcre overflow checking)
changeset:   2888:a34e717a8864 (xml overflow checking)
changeset:   2889:d355f00be039 (re-enable test cases on winmo)

Comment 6

[Brent Baker]

Verified that testcases are running and passing in build 2894