Page MenuHomePhabricator

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

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

Details

Summary

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

Repository
rEFL core/efl
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
stefan_schmidt created this revision.Mon, Nov 18, 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: https://phab.enlightenment.org/w/maintainers_reviewers/

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

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

bu5hm4n accepted this revision.Tue, Nov 26, 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.Tue, Nov 26, 6:17 AM
This revision was automatically updated to reflect the committed changes.