Register the WER runtime exception module at runtime in child processes
Categories
(Toolkit :: Crash Reporting, task)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox91 | --- | fixed |
People
(Reporter: gsvelto, Assigned: gsvelto)
References
Details
Attachments
(1 file)
This is like bug 1682514 but for all child processes. Child processes currently write out their crash annotations at exception time; if we reached WER however the exception handler will have already been bypassed so we need another way to extract annotations and write them to the .extra file. One possibility is to have the runtime exception module write a minimal .extra file like it does for main process crashes plus the process type. This won't be great but should be enough if push comes to shove.
|
Assignee | |
Comment 1•3 years ago
|
||
I've got this pretty much done but there's a lot more surgery involved for implementing bug 1682518.
|
|
Assignee | |
Comment 2•3 years ago
|
||
This patch sets up a few different things that will be used by the WER runtime
exception module when it needs to notify the main process of a child process
crash.
For every child process we allocate a structure in the main process called
WindowsErrorReportingData that contains three things:
- The address of the function used to notify the main process that there's a
pending minidump for a given child process - The PID of said child process
- The name of the minidump that has been generated
The first field is filled up by the main process and will be read by the WER
process when running the runtime exception module, the second and third fields
on the other hand start empty and will be written into by the runtime exception
module after it has generated a minidump.
I know this sounds scary. It is. But bear with me please.
When we register the runtime exception module we can pass it a single
pointer-sized parameter but we need to pass it at least another pointer that
includes data coming from the child process itself (this one is called
InProcessWindowsErrorReportingData). This data currently includes only the
process type but will also include certain annotations in the future
(e.g. bug 1711418). So here's what we do: we store a pointer to the parent
data structure in the child process command-line (cringe) and we read it
from the runtime exception module by reading the crashed process command-line
arguments and parsing them (double-cringe).
Armed with this information the WER runtime exception module can populate
the info for the generated minidump and then push it into the main process
by calling CreateRemoteThread() (which creates a new thread in the main
process, triple-cringe at this point).
Pushed by gsvelto@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/d75aef90ac53 Register the WER runtime exception module in child processes r=KrisWright
|
||
Comment 4•3 years ago
|
||
Backed out 5 changesets (Bug 1697895, Bug 1682518, Bug 1703761, Bug 1711418) for causing Windows 2012 x64 asan build bustages.
https://hg.mozilla.org/integration/autoland/rev/f4046bbf47c7b1b2f596168b3ffa7f046c191524
Push with failures:
https://treeherder.mozilla.org/jobs?repo=autoland&revision=4cc2cb3653f2f17e0cdf220eb1714c0a2d0dba30&selectedTaskRun=NBgmowhvR0-HwPjbLja2eQ.0
Failure log:
https://treeherder.mozilla.org/logviewer?job_id=342378389&repo=autoland&lineNumber=50884
|
|
Assignee | |
Comment 5•3 years ago
|
||
I forgot to stub the new WerNotifyProc() function in the dummy exception handler that gets used when crash reporting is off... which is the default in ASAN builds.
Pushed by gsvelto@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/a9be55acfd91 Register the WER runtime exception module in child processes r=KrisWright
|
||
Comment 7•3 years ago
|
||
Backed out for causing build bustages on nsEmbedFunctions.cpp.
|
|
Assignee | |
Comment 8•3 years ago
•
|
||
Ah, a parameter change to SetRemoteExceptionHandler()... I wonder why it only broke in no-opt builds, I thought only ASAN would be affected. Well, no matter, fix incoming.
Pushed by gsvelto@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/3dc9c315f3b6 Register the WER runtime exception module in child processes r=KrisWright
|
||
Comment 10•3 years ago
|
||
| bugherder | ||
Description
•