Closed
Bug 854835
Opened 12 years ago
Closed 11 years ago
Intermittent test_bug472986.html,test_bug544462.html,test_unsecurePictureDup.html,test_removal_ondecode.html | application crashed [@ mozilla::image::RasterImage::DecodePool::RequestDecode(mozilla::image::RasterImage *)]
Categories
(Core :: Graphics: ImageLib, defect)
Tracking
()
RESOLVED
WORKSFORME
People
(Reporter: emorley, Assigned: seth)
References
Details
(Keywords: crash, intermittent-failure)
Crash Data
Attachments
(1 obsolete file)
Rev3 WINNT 6.1 mozilla-inbound debug test mochitest-5 on 2013-03-25 19:11:48 PDT for push 1555815d144a
slave: talos-r3-w7-072
https://tbpl.mozilla.org/php/getParsedLog.php?id=21084792&tree=Mozilla-Inbound
{
19:25:49 INFO - 163641 INFO TEST-START | /tests/security/ssl/mixedcontent/test_bug472986.html
19:25:49 INFO - ++DOMWINDOW == 72 (16C48BE0) [serial = 2817] [outer = 077F9DC8]
19:25:50 INFO - --DOMWINDOW == 71 (16C4F358) [serial = 2811] [outer = 16C4A658] [url = about:blank]
19:25:50 INFO - --DOCSHELL 1A6BA168 == 8 [id = 215]
19:25:50 INFO - --DOCSHELL 149AC248 == 7 [id = 213]
19:25:50 INFO - --DOMWINDOW == 70 (1261C330) [serial = 2797] [outer = 00000000] [url = https://example.com/tests/security/ssl/mixedcontent/test_bug329869.html?runtest]
19:25:50 INFO - --DOMWINDOW == 69 (1261E9B0) [serial = 2796] [outer = 00000000] [url = https://example.com/tests/security/ssl/mixedcontent/test_bug329869.html?runtest]
19:25:50 INFO - --DOMWINDOW == 68 (1261D1A0) [serial = 2795] [outer = 00000000] [url = about:blank]
19:25:50 INFO - --DOMWINDOW == 67 (1261D670) [serial = 2802] [outer = 00000000] [url = about:blank]
19:25:51 INFO - Assertion failure: NS_IsMainThread_P(), at e:/builds/moz2_slave/m-in-w32-d-0000000000000000000/build/image/src/RasterImage.cpp:3603
19:25:54 WARNING - TEST-UNEXPECTED-FAIL | /tests/security/ssl/mixedcontent/test_bug472986.html | Exited with code -2147483645 during test run
19:25:54 INFO - INFO | automation.py | Application ran for: 0:10:06.056000
19:25:54 INFO - INFO | automation.py | Reading PID log: c:\users\cltbld\appdata\local\temp\tmpln3nvupidlog
19:26:10 INFO - PROCESS-CRASH | /tests/security/ssl/mixedcontent/test_bug472986.html | application crashed [@ mozilla::image::RasterImage::DecodePool::RequestDecode(mozilla::image::RasterImage *)]
19:26:10 INFO - Crash dump filename: c:\users\cltbld\appdata\local\temp\tmp50tyjb\minidumps\93b4f2b6-88af-4b89-8e7f-243daa13e5d3.dmp
19:26:10 INFO - Operating system: Windows NT
19:26:10 INFO - 6.1.7600
19:26:10 INFO - CPU: x86
19:26:10 INFO - GenuineIntel family 6 model 23 stepping 10
19:26:10 INFO - 2 CPUs
19:26:10 INFO - Crash reason: EXCEPTION_BREAKPOINT
19:26:10 INFO - Crash address: 0x6a71fd71
19:26:10 INFO - Thread 32 (crashed)
19:26:10 INFO - 0 xul.dll!mozilla::image::RasterImage::DecodePool::RequestDecode(mozilla::image::RasterImage *) [RasterImage.cpp:1555815d144a : 3603 + 0x38]
19:26:10 INFO - eip = 0x6a71fd71 esp = 0x15bffbc4 ebp = 0x15bffbd4 ebx = 0x7707060c
19:26:10 INFO - esi = 0x693f1440 edi = 0x7706509b eax = 0x00000000 ecx = 0xe9515c04
19:26:10 INFO - edx = 0x694ee4d8 efl = 0x00200212
19:26:10 INFO - Found by: given as instruction pointer in context
19:26:10 INFO - 1 xul.dll!mozilla::image::RasterImage::DecodePool::DecodeJob::Run() [RasterImage.cpp:1555815d144a : 3728 + 0xd]
19:26:10 INFO - eip = 0x6a720c24 esp = 0x15bffbdc ebp = 0x15bffbfc
19:26:10 INFO - Found by: call frame info
19:26:10 INFO - 2 xul.dll!nsThreadPool::Run() [nsThreadPool.cpp:1555815d144a : 194 + 0xd]
19:26:10 INFO - eip = 0x6bc7f870 esp = 0x15bffc04 ebp = 0x15bffc30
19:26:10 INFO - Found by: call frame info
19:26:10 INFO - 3 xul.dll!nsThread::ProcessNextEvent(bool,bool *) [nsThread.cpp:1555815d144a : 627 + 0xd]
19:26:10 INFO - eip = 0x6bc7cd76 esp = 0x15bffc38 ebp = 0x15bffc88
19:26:10 INFO - Found by: call frame info
19:26:10 INFO - 4 xul.dll!NS_ProcessNextEvent_P(nsIThread *,bool) [nsThreadUtils.cpp:1555815d144a : 238 + 0xc]
19:26:10 INFO - eip = 0x6bc21e96 esp = 0x15bffc90 ebp = 0x15bffc9c
19:26:10 INFO - Found by: call frame info
19:26:10 INFO - 5 xul.dll!nsThread::ThreadFunc(void *) [nsThread.cpp:1555815d144a : 265 + 0x7]
19:26:10 INFO - eip = 0x6bc7c314 esp = 0x15bffca4 ebp = 0x15bffcc4
19:26:10 INFO - Found by: call frame info
19:26:10 INFO - 6 nss3.dll!_PR_NativeRunThread [pruthr.c:1555815d144a : 395 + 0x8]
19:26:10 INFO - eip = 0x696dc49b esp = 0x15bffccc ebp = 0x15bffd24
19:26:10 INFO - Found by: call frame info
19:26:10 INFO - 7 nss3.dll!pr_root [w95thred.c:1555815d144a : 90 + 0xc]
19:26:10 INFO - eip = 0x696e35ed esp = 0x15bffce8 ebp = 0x15bffd24
19:26:10 INFO - Found by: call frame info
19:26:10 INFO - 8 MSVCR100D.dll + 0x4a292
19:26:10 INFO - eip = 0x693da293 esp = 0x15bffcf0 ebp = 0x15bffd24
19:26:10 INFO - Found by: call frame info
19:26:10 INFO - 9 MSVCR100D.dll + 0x4a223
}
Updated•12 years ago
|
Crash Signature: [@ mozilla::image::RasterImage::DecodePool::RequestDecode(mozilla::image::RasterImage *)] → [@ mozilla::image::RasterImage::DecodePool::RequestDecode(mozilla::image::RasterImage*)]
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment 3•12 years ago
|
||
Summary: Intermittent ssl/mixedcontent/test_bug472986.html | application crashed [@ mozilla::image::RasterImage::DecodePool::RequestDecode(mozilla::image::RasterImage *)] → Intermittent test_bug472986.html,test_bug544462.html | application crashed [@ mozilla::image::RasterImage::DecodePool::RequestDecode(mozilla::image::RasterImage *)]
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment 14•12 years ago
|
||
Comment 15•12 years ago
|
||
Yes, this might be the same (or the same cause) as bug 854762, which is a reproducible crasher with new code. Let's hope to see a fix soon.
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment 19•12 years ago
|
||
Summary: Intermittent test_bug472986.html,test_bug544462.html | application crashed [@ mozilla::image::RasterImage::DecodePool::RequestDecode(mozilla::image::RasterImage *)] → Intermittent test_bug472986.html,test_bug544462.html,test_unsecurePictureDup.html | application crashed [@ mozilla::image::RasterImage::DecodePool::RequestDecode(mozilla::image::RasterImage *)]
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment 25•12 years ago
|
||
Summary: Intermittent test_bug472986.html,test_bug544462.html,test_unsecurePictureDup.html | application crashed [@ mozilla::image::RasterImage::DecodePool::RequestDecode(mozilla::image::RasterImage *)] → Intermittent test_bug472986.html,test_bug544462.html,test_unsecurePictureDup.html,test_removal_ondecode.html | application crashed [@ mozilla::image::RasterImage::DecodePool::RequestDecode(mozilla::image::RasterImage *)]
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment 29•12 years ago
|
||
Assignee | ||
Comment 30•12 years ago
|
||
(In reply to Robert Kaiser (:kairo@mozilla.com) from comment #15)
> Yes, this might be the same (or the same cause) as bug 854762, which is a
> reproducible crasher with new code. Let's hope to see a fix soon.
Nah, I doubt these two are the same.
This is an assertion failing, and the assertion itself is blatantly incorrect.
This is so trivial that I'm not to bother to create a separate bug for this. Will have a patch up shortly.
No longer depends on: 854762
Assignee | ||
Comment 31•12 years ago
|
||
Here's the patch. It's untrue that DecodePool::RequestDecode is only called from the main thread, so we shouldn't assert that. DecodeJob::Run can also call it. Furthermore, I don't see any reason to think that it's unsafe as long as we hold the decoding mutex (which we do, and we assert so).
Attachment #732126 -
Flags: review?(jmuizelaar)
Assignee | ||
Updated•12 years ago
|
Assignee: nobody → seth
Comment hidden (Legacy TBPL/Treeherder Robot) |
Updated•12 years ago
|
Attachment #732126 -
Flags: review?(jmuizelaar) → review+
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Assignee | ||
Comment 35•12 years ago
|
||
Nothing's ever easy. This turned out to be more complex than it appeared. I'll create a new bug for this.
Assignee | ||
Updated•12 years ago
|
Attachment #732126 -
Attachment is obsolete: true
Assignee | ||
Comment 36•12 years ago
|
||
I'll fix this in bug 857367.
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Assignee | ||
Comment 48•12 years ago
|
||
I believe that most recent star doesn't include the patch from bug 857367. Hopefully once things settle down a little we can confirm that this is now fixed. (Since it happened on Aurora we might want to consider uplifting bug 857367...)
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Reporter | ||
Comment 51•11 years ago
|
||
(OrangeWFM bugs not touched for 2 months)
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → WORKSFORME
You need to log in
before you can comment on or make changes to this bug.
Description
•