Closed
Bug 1222487
Opened 9 years ago
Closed 9 years ago
Thunderbird crash in mozilla::gfx::UserData::Add when printing
Categories
(Core :: Graphics, defect)
Tracking
()
RESOLVED
WORKSFORME
Tracking | Status | |
---|---|---|
thunderbird_esr38 | --- | wontfix |
People
(Reporter: wsmwk, Assigned: sotaro)
References
Details
(4 keywords, Whiteboard: [regression:TB38.3.0][gfx-noted][tbird topcrash])
Crash Data
Not a new crash [1], but has risen significantly in rankings.
#8 for version 38.3.0. Was #15 in 38.2.0.
All have printing on the stack and printing is often mentioned in user comments.
bp-89d71f90-04e9-4a24-afd5-8a97a2151101.
0 xul.dll mozilla::gfx::UserData::Add(mozilla::gfx::UserDataKey*, void*, void (*)(void*)) gfx/2d/UserData.h
1 xul.dll nsDeviceContext::CreateRenderingContext() gfx/src/nsDeviceContext.cpp
2 xul.dll nsSimplePageSequenceFrame::PrintNextPage() layout/generic/nsSimplePageSequenceFrame.cpp
3 xul.dll nsPrintEngine::PrintPage(nsPrintObject*, bool&) layout/printing/nsPrintEngine.cpp
4 xul.dll nsPagePrintTimer::Run() layout/printing/nsPagePrintTimer.cpp
Other examples, all with user comments and email addresses:
bp-e09683d4-d99a-4994-96a6-475a42151101
bp-0f2137b0-8177-4c1a-b531-ea59b2151105
bp-b299b005-d630-4168-98aa-8a6432151104
bp-21b81ce3-395e-414f-b69a-d749b2151105
bp-1effa082-6b12-44ae-a43a-8215d2151104
[1] crash exists in version 31 https://crash-stats.mozilla.com/search/?signature=~mozilla%3A%3Agfx%3A%3AUserData%3A%3AAdd&product=Thunderbird&date=%3E2015-05-01&date=%3C2015-08-01&_facets=signature&_columns=date&_columns=signature&_columns=product&_columns=version&_columns=build_id&_columns=platform#crash-reports
Note mozilla::gfx::UserData::Add(mozilla::gfx::UserDataKey*, void*, void (*)(void*)) exists in TB 31.* and 38.2.0 but virtually gone in TB 38.3.0 - probably from changes in signature creation for crash-stats. But that should not quite so dramatic a rise in ranking. So I'm suggesting a regression is involved.
Reporter | ||
Comment 1•9 years ago
|
||
However, for firefox the crash pretty much does not exist in version 41 and newer https://crash-stats.mozilla.com/report/list?signature=mozilla%3A%3Agfx%3A%3AUserData%3A%3AAdd
If it matters, note that hardware acceleration is disabled starting in TB38.2.0
status-thunderbird_esr38:
--- → affected
tracking-thunderbird_esr38:
--- → ?
Component: General → Graphics
Product: Thunderbird → Core
Version: unspecified → 38 Branch
Updated•9 years ago
|
Whiteboard: [regression:TB38.3.0] → [regression:TB38.3.0][gfx-noted]
Comment 2•9 years ago
|
||
We fail to create a DrawTarget here http://hg.mozilla.org/releases/mozilla-esr38/annotate/fc916052746/gfx/src/nsDeviceContext.cpp#l410
Since we are in the middle of printing I assume we need to do something more than null-check and early-return, because printing will be broken so we should at least warn the user or something. Jeff, do you know if there's a mechanism for handling errors during printing?
Flags: needinfo?(jmuizelaar)
Comment 3•9 years ago
|
||
(In reply to Wayne Mery (:wsmwk, use Needinfo for questions) from comment #1)
> However, for firefox the crash pretty much does not exist in version 41 and
> newer
It exists for v45.0a2
https://crash-stats.mozilla.com/report/index/b3d161d7-4c7f-448e-92ca-d15bc2151220
It exists for v44.0b2
https://crash-stats.mozilla.com/report/index/27376db0-6895-48ef-b1d0-1b2fc2151226
https://crash-stats.mozilla.com/report/index/2f746eb8-996f-4c09-83cf-034522151226
https://crash-stats.mozilla.com/report/index/b636f886-21f1-4932-af0b-092a02151223
https://crash-stats.mozilla.com/report/index/99b65ea5-c666-4112-b4c3-fa3722151224
https://crash-stats.mozilla.com/report/index/60362a54-8c5c-4f9b-87bb-8a0a02151223
https://crash-stats.mozilla.com/report/index/4822ef93-f613-40b5-b8c4-a44052151225
The crash clearly exists on all versions from V29.0.1 onward to v45.0a2 and it has never been fixed.
Updated•9 years ago
|
Assignee: nobody → sotaro.ikeda.g
Updated•9 years ago
|
Flags: needinfo?(jmuizelaar)
Reporter | ||
Updated•9 years ago
|
Whiteboard: [regression:TB38.3.0][gfx-noted] → [regression:TB38.3.0][gfx-noted][tbird topcrash]
Assignee | ||
Comment 4•9 years ago
|
||
The call stack of the crash did not specific to the print. It was just normal rendering update. printing just seemed to trigger oom.
Assignee | ||
Comment 5•9 years ago
|
||
(In reply to Sotaro Ikeda [:sotaro] from comment #4)
> The call stack of the crash did not specific to the print. It was just
> normal rendering update. printing just seemed to trigger oom.
The above was for the stacks in comment 3. Call stacks in comment 0 was different than comment 3.
comment 0 was printing related crash as in comment 2. comment 3 is a different crash that might not be related to printing.
Assignee | ||
Comment 6•9 years ago
|
||
Bug 1257112 is created for crashes of comment 3.
Assignee | ||
Comment 7•9 years ago
|
||
Bug 1019063 already addressed the cause of the crash. It handles oom failure in nsDeviceContext::CreateRenderingContext(). Uplifting Bug 1019063 should address the crash.
Assignee | ||
Comment 8•9 years ago
|
||
From Bug 1019063 Comment 78, we could not uplift the fix to esr38.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → WONTFIX
Reporter | ||
Comment 9•9 years ago
|
||
I believe this signature is no longer relevant to Thunderbird 45. All 4 of those crashes are startup, and have different signatures than the THunderbird "printing" crashes. Either the crash is gone for Thunderbird or the signature changed (but I find not one TB45 crash so far whose comments mentions printing).
Similarly, the rate for Thunderbird printing bug 1063723 has changed in 45, but not entirely gone.
tracking-thunderbird_esr38:
? → ---
Resolution: WONTFIX → WORKSFORME
Summary: crash in mozilla::gfx::UserData::Add when printing → Thunderbird crash in mozilla::gfx::UserData::Add when printing
You need to log in
before you can comment on or make changes to this bug.
Description
•