Open Bug 1771212 Opened 3 years ago Updated 3 years ago

Make test_crash_abort.js actually test calls to abort()

Categories

(Toolkit :: Crash Reporting, defect)

Unspecified
All
defect

Tracking

()

People

(Reporter: gsvelto, Unassigned)

Details

I didn't notice it before but the test is not actually testing what it's supposed to test:

  • On Windows it's disabled because abort() calls __fastfail() but I should be able to fix that with WER
  • On Linux and macOS abort() gets redirected to mozalloc_abort() which - at least on macOS - doesn't exercise the SIGABRT handler but rather the regular EXC_BAD_ACCESS exception catcher

So we should enable this on Windows (and make it pass) and make sure we're really, really calling the system abort() on macOS and Linux.

The severity field is not set for this bug.
:gsvelto, could you have a look please?

For more information, please visit auto_nag documentation.

Flags: needinfo?(gsvelto)
Severity: -- → S3
Flags: needinfo?(gsvelto)
You need to log in before you can comment on or make changes to this bug.