Creating a SANITIZED version of C-C TB on try-comm-central

298 views
Skip to first unread message

ISHIKAWA,chiaki

unread,
Sep 6, 2024, 6:06:50 PM9/6/24
to dev-platform, Thunderbird Developers
Hi,

Is there a way to build and run test of sanitized version of C-C TB
(UBSAN and ASAN) on try-comm-central?

I suspect there is only linux version (?)  Mac and windows versions will
be great.

The following web page does not explain the try syntax for sanitized build.
https://developer.thunderbird.net/thunderbird-development/fixing-a-bug/try-server

However, there is a bugzilla that mentions the creation of Windows ASAN
for thunderbird daily and
so I assume it is possible to build SANITIZED version of C-C TB on
try-comm-central.

Provide Address Sanitizer builds of Thunderbird Daily
https://bugzilla.mozilla.org/show_bug.cgi?id=1641581

C-C TB UBSAN version fails fairly early until the following bug is fixed.
I have a local patch posted to the bugzilla entry to cope with the issue,
and using the fixed version I found other error issues triggered by the
ASAN and UBSAN checks and reported the issues to bugzilla.

[asan perma] SUMMARY: UndefinedBehaviorSanitizer: undefined-behavior
/builds/worker/checkouts/gecko/comm/mailnews/mime/src/mimebuf.cpp:132:10 in

https://bugzilla.mozilla.org/show_bug.cgi?id=1852662

Given that I have not been able to use valgrind during mochitest and
xpcshell test locally, valgrind is the best hope for checking memory issues.

Another reason I want to build ASAN version on try-comm-central is due
to the rust crate version mismatches local compilation often fails even
if I update both the M-C and C-C trees.
Yet, the build works when my local builds fails due to the rust crate
version mismatch or something.

So I am tempted to create the ASAN binary when my local build fails for
more than half a day or so.

This has been a serious hindrance for creating binary patches for C-C TB
(JavaScript patches can be done without re-buliding the binary,
though.)  whether SANITIZED version is wanted or not.


TIA


Chiaki



Reply all
Reply to author
Forward
0 new messages