Closed
Bug 539451
Opened 15 years ago
Closed 14 years ago
Multi-process crash reporting should unhook and not leak
Categories
(Core :: IPC, defect)
Tracking
()
RESOLVED
FIXED
| Tracking | Status | |
|---|---|---|
| status1.9.2 | --- | .4-fixed |
People
(Reporter: benjamin, Assigned: cjones)
References
(Depends on 1 open bug)
Details
(Whiteboard: [fixed-lorentz])
Attachments
(1 file)
|
1.42 KB,
patch
|
ted
:
review+
|
Details | Diff | Splinter Review |
Multi-process crash reporting currently doesn't unhook on UnsetExceptionHandler and leaks childCrashNotifyPipe. It should clean up better.
|
Assignee | |
Comment 1•14 years ago
|
||
Assignee: nobody → jones.chris.g
Attachment #422642 -
Flags: review?(ted.mielczarek)
|
||
Comment 2•14 years ago
|
||
Comment on attachment 422642 [details] [diff] [review] Clean up the OOP crashreporting stuff OOPDeinit() rubs me the wrong way, but I don't have a better suggestion.
Attachment #422642 -
Flags: review?(ted.mielczarek) → review+
|
|
Assignee | |
Comment 3•14 years ago
|
||
Pushed http://hg.mozilla.org/projects/electrolysis/rev/3d68f4cc4d55
Whiteboard: low priority → low priority [land m-c]
|
Reporter | |
Comment 4•14 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/7f3377180c6a
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
|
|
Reporter | |
Comment 5•14 years ago
|
||
http://hg.mozilla.org/projects/firefox-lorentz/rev/629b79e60d8b
Whiteboard: low priority [land m-c] → [fixed-lorentz]
|
||
Comment 6•14 years ago
|
||
Blanket approval for Lorentz merge to mozilla-1.9.2 a=beltzner for 1.9.2.4 - please make sure to mark status1.9.2:.4-fixed
|
|
Reporter | |
Comment 7•14 years ago
|
||
Merged into 1.9.2 at http://hg.mozilla.org/releases/mozilla-1.9.2/rev/84ba4d805430
status1.9.2:
--- → .4-fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•