Page MenuHomePhabricator

ci: travis: add job to build efl and run tests with address sanitizer

Authored by stefan_schmidt on Nov 18 2019, 6:59 AM.



A first step to have addrress sanitizer coverage on our EFL build. It
already catched some use after free errors and memory corruptions. Still
we have some limititation how we are running this right now. We
currently ignore some leaks that are exposed durign the normal build
wile haveing the short living runs of edje_cc, elm_prefs_cc, etc.
For now the whole leak detection is disabled on the testsuite as well.

This is still very valuable, especially for all non-leak cases it finds,
thus we are goign to run this on our daily build.

Diff Detail

rEFL core/efl
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.
stefan_schmidt created this revision.Nov 18 2019, 6:59 AM

It seems that this patch has no reviewers specified. If you are unsure who can review your patch, please check this wiki page and see if anyone can be added:

stefan_schmidt requested review of this revision.Nov 18 2019, 6:59 AM

Use the same check retry logic as we have for the normal runs

bu5hm4n accepted this revision.Nov 26 2019, 6:17 AM

I really like this change.

But i am a bit concerned about how it is picked up by devs, if accidental new mem-leaks are introduced.

Can you land that on your own, it seems to not apply here correctly.

This revision is now accepted and ready to land.Nov 26 2019, 6:17 AM
This revision was automatically updated to reflect the committed changes.