Closed Bug 1718226 Opened 3 years ago Closed 3 years ago

Do not capture crashes intercepted via Windows Error Reporting that aren't flagged as fatal

Categories

(Toolkit :: Crash Reporting, task)

Unspecified
Windows
task

Tracking

()

RESOLVED FIXED
91 Branch
Tracking Status
firefox91 --- fixed

People

(Reporter: gsvelto, Assigned: gsvelto)

References

Details

Attachments

(1 file)

This is an attempt at fixing issues such as bug 1716120, bug 1709423 (and thus bug 1714334 too). It's a bit of a shot in the dark because WER documentation is full of holes.

I've noticed that the WER_RUNTIME_EXCEPTION_INFORMATION structure has a field called bIsFatal. I guess that this might be used to tell crashes apart from hangs and the like. Since the meaning of field is not documented anywhere I can only guess that its name is descriptive.

I'll cook up a patch to ignore crashes that have it set to false and we'll see what happens.

Our Windows Error Reporting runtime module seems to be notified of all
sorts of non-fatal exceptions. Since there is no documentation clarifying
how to tell them apart from actual crashes we'll try using the bIsFatal
field in the WER_RUNTIME_EXCEPTION_INFORMATION structure for this
purpose. There is no documentation describing the contents of the field
so we can only assume that what its name implies is what we're looking
for.

Assignee: nobody → gsvelto
Status: NEW → ASSIGNED
Pushed by gsvelto@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/341fcf114fa1
Only capture crash reports for fatal exceptions intercepted via WER r=KrisWright
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 91 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: