search

crashappsec / libcon4m

Base Compiler and Runtime Support for con4m
Apache License 2.0
0 stars 0 forks source link

Retool tests #88

Closed viega closed 1 month ago

viega commented 1 month ago

Retool the test system to fork by default, so that if a test crashes the runner will still complete.

viega commented 1 month ago

I don't get that crash; I'll look at it in a bit.

viega commented 1 month ago

Okay, I see what the issue is; it was there before, and makes sense that it'd only trigger w/ ASAN / UBSAN. One sec.

ee7 commented 1 month ago

It seemed to be the other way around: the crash didn't happen when I enabled sanitizers. The steps in https://github.com/crashappsec/libcon4m/pull/88#pullrequestreview-2185716970 were without sanitizers.

Will check the new commit in a sec.

viega commented 1 month ago

Either way would make sense, and I'm 100% sure it was the problem.