Closed Bug 1641581 Opened 4 years ago Closed 4 years ago

Provide Address Sanitizer builds of Thunderbird Daily

Categories

(Thunderbird :: Build Config, enhancement)

enhancement

Tracking

(thunderbird78 affected)

RESOLVED FIXED
Thunderbird 80.0
Tracking Status
thunderbird78 --- affected

People

(Reporter: tdulcet, Assigned: rjl)

References

()

Details

Attachments

(3 files)

Provide Address Sanitizer (ASan) builds of Thunderbird Daily and port the ASan reporter add-on, to make fuzzing Thunderbird and reporting security vulnerabilities easier. Also see:

Assignee: nobody → rob

For what it's worth, I tried building ASAN build of DEBUG version of TB using GCC locally several weeks ago.
This was motivated due to the frustration of not being able to run mochitest under valgrind.
(|make mozmill| ran under valgrind and that helped me find many memory-related errors in the past including one that got me security bounty even.)

Specifically, there is a link time error caused by recently added OTR-related function.
Maybe I should report it if it is still there.
The added library function caused many other issues. So, I thought I would wait for the dust to settle down and would try again. Now that this bugzilla caught my eye, it is a good time to try again.
But I need to fix my local patches to cope with the massive format-only changes that landed in the last few days first.

Does the attached configuration change create ASAN build of TB on taskcluster (that uses clang)?
If so, maybe I should not bother to try creating GCC-based binary of ASAN TB.

TIA

Flags: needinfo?(rob)

(In reply to ISHIKAWA, Chiaki from comment #2)

Does the attached configuration change create ASAN build of TB on taskcluster (that uses clang)?

Yes it does, I'll get it landed today so that you can take a look at it.

Flags: needinfo?(rob)
Status: UNCONFIRMED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED

The reporter addon will have to be covered in a separate bug, that's a pretty big project.
Other platforms can be built if there is desire and someone intends to make use of them.

Target Milestone: --- → Thunderbird 79.0

Should I create a new bug for porting the ASan reporter add-on?

I would be interested in (and would use) a Windows build. Firefox currently supports both Linux and Windows.

(In reply to tdulcet from comment #6)

Should I create a new bug for porting the ASan reporter add-on?
Yes please do.

I would be interested in (and would use) a Windows build. Firefox currently supports both Linux and Windows.

Great! I'll see what I can do.

Target Milestone: Thunderbird 79.0 → Thunderbird 80.0

Yes please do.

OK, I created Bug 1648731.

Great! I'll see what I can do.

Thanks for creating the Windows build! Where is the correct place to download both the Linux and Windows ASan builds? I am not seeing them listed with the other Daily/Nightly builds, although I can see that they are being built on Treeherder. Maybe these links should also be listed somewhere in the Developer documentation, like Firefox does.

On treeherder, click the B of the build, like https://treeherder.mozilla.org/#/jobs?repo=comm-central&selectedTaskRun=KMkcDbtNToG0CkmQMZFLUg.0 - then in the "Job details" below find the appropriate artifact to download.

Attached image error.png (deleted) —

OK, I downloaded the Windows installer, but I get this error during the installation and when I try to run Thunderbird afterwords. The installation completes, but Thunderbird does not open.

I'll take care of that. See bug 1646557.

Bug 1646557 was marked as invalid two weeks ago and I am still getting this error.

Flags: needinfo?(rob)

That's not the right bug number. It's bug 1649557.
I've reopened it since it's not working as expected.

Flags: needinfo?(rob)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: