Serialize content crash annotations in binary format when sending them to the main process
Categories
(Toolkit :: Crash Reporting, task)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox69 | --- | fixed |
People
(Reporter: gsvelto, Assigned: gsvelto)
References
Details
Attachments
(1 file)
|
(deleted),
text/x-phabricator-request
|
Details |
Exception-time crash annotations are generated in the PrepareChildExceptionTimeAnnotations() functions and sent to the main process via a pipe. Before bug 1407693 they were written to a file instead so the format they're serialized to is the same INI-style format used for the .extra file.
This is inconvenient as it requires turning the annotations into strings and escaping them. However since we're in a crashed context only the MozCrashReason annotation is escaped, the others aren't leading to data loss in case they contain null or newline characters.
Additionally we need to parse and validate the strings in the main process before serializing them back into the .extra file.
Since the serialized annotations are not seen by any other code save for PrepareChildExceptionTimeAnnotations() and ReadAndValidateExceptionTimeAnnotations() it would be simpler to serialize them and read them back in a binary format such as <annotation>/<size>/<data>. The data would also not require escaping then.
|
Assignee | |
Comment 1•5 years ago
|
||
|
|
Assignee | |
Updated•5 years ago
|
|
||
Comment 3•5 years ago
|
||
| bugherder | ||
Description
•