1336930 - Assertion failure: (frame != nullptr) == (((HRESULT)(hr)) >= 0), at WMFVideoMFTManager.cpp:988

Status: NEW

(Core :: Audio/Video: Playback, defect, P3)

Description

[Carsten Book [:Tomcat]]

Attachment: bughunter stack

Assertion failure: (frame != nullptr) == (((HRESULT)(hr)) >= 0), at c:/builds/moz2_slave/m-cen-w32-d-000000000000000000/build/src/dom/media/platforms/wmf/WMFVideoMFTManager.cpp:988

found on a nsfw werbesite, when loading the front page and also on https://ni.learn.com/files/scorm/EN/FPGA/story.html

seems this is a kind of OOM or so while just loading the front page as example

Comment 1

[Alastor Wu [:alwu]]

Keep NI, I'll check it later.

Comment 2

[Alastor Wu [:alwu]]

Do you have the crash report? 
Which Windows version do you use? Do you open hardware acceleration?
I can't reproduce this issue on my Windows7.
Thanks!

Comment 3

[Bob Clary [:bc] (inactive)]

Attachment: windows 7 64bit crash report

We've seen this mostly on Windows 7/10 32bit but also see it on 64bit. The 32bit crashes are on virtual machines with 3G memory but the 64bit crashes are on virtual machines with 8G memory.

Comment 4

[Bob Clary [:bc] (inactive)]

Attachment: the crash dump extra file

Note these are ESX virtual machines with the corresponding display adapter. We've seen this on 19 different urls. I'll retest and pick several.

Comment 5

[Carsten Book [:Tomcat]]

yeah my crash when trying to reproduce was also on windows 7

Comment 6

[Bob Clary [:bc] (inactive)]

http://www.flv.kr/ has several related crashes. You may need to load the various videos. Some of them are slightly NSFW so don't view them in the office.

Comment 7

[Alastor Wu [:alwu]]

OK, Let me check what's happened.

Comment 8

[Bob Clary [:bc] (inactive)]

Note we don't have flash installed on any of our vms and these play with the h264 decoder if I understand the situation correctly.

Comment 9

[Alastor Wu [:alwu]]

Hi, Bob,
So you can reproduce this issue just by loading the various videos on http://www.flv.kr/?
How many times you switch the video or do I need any extra steps?
Because now I can't reproduce this issue on my Win7. (Also can't success by visiting the link in comment0)
Thanks!

Comment 10

[Bob Clary [:bc] (inactive)]

I was just looking at the bughunter reports. I've started some manual tests on a win7 32bit and 64bit and will report back on my success.

Comment 11

[Bob Clary [:bc] (inactive)]

Attachment: 1336930.log

I got lazy and used Spider to load the pages from the site. It reproduced quickly. I haven't reviewed the urls 

Install https://bclary.com/projects/spider/spider-0.1.0.3-sm+tb+fx+an+fn.xpi

firefox -spider -start -quit -url 'http://www.flv.kr/' -depth 1 -wait 60 2>&1 | tee /tmp/1336930.log | grep Assertion.failure:

Assertion failure: (frame != nullptr) == (((HRESULT)(hr)) >= 0), at c:/builds/moz2_slave/m-cen-w64-d-000000000000000000/build/src/dom/media/platforms/wmf/WMFVideoMFTManager.cpp:988

$ grep Begin 1336930.log 
Spider: Begin loading http://www.flv.kr/
Spider: Begin loading http://www.flv.kr/category
Spider: Begin loading http://www.flv.kr/category/%EC%97%B0%EC%98%88
Spider: Begin loading http://www.flv.kr/category/%EC%A7%A4%EB%B0%A9
Spider: Begin loading http://dasibogi.kr/top100/kpop
Spider: Begin loading http://www.flv.kr/tag/mouseMove
Spider: Begin loading http://www.flv.kr/category?page=269
Spider: Begin loading http://www.flv.kr/tag/%EC%8D%A8%EB%8B%88

Hope this helps.

Comment 12

[Bob Clary [:bc] (inactive)]

Note that was on win61 i64 with 8G. I could also reproduce on win61 i32 with 3G just loading the original url in spider.

mozauto@win61i32-user /mozilla/builds/hg.mozilla.org/sisyphus/xpi/all (sisyphus-prd)
$ for i in {1..100}; do echo ======= $i =======;  nightly.sh -spider -start -quit -url 'https://ni.learn.com/files/scorm/EN/FPGA/story.html' -depth 1 2>&1 | tee /tmp/1336930.log | grep Assertion.failure:; done
======= 1 =======
======= 2 =======
======= 3 =======
======= 4 =======
======= 5 =======
======= 6 =======
======= 7 =======
Assertion failure: (frame != nullptr) == (((HRESULT)(hr)) >= 0), at c:/builds/moz2_slave/m-cen-w32-d-000000000000000000/build/src/dom/media/platforms/wmf/WMFVideoMFTManager.cpp:988
Assertion failure: (frame != nullptr) == (((HRESULT)(hr)) >= 0), at c:/builds/moz2_slave/m-cen-w32-d-000000000000000000/build/src/dom/media/platforms/wmf/WMFVideoMFTManager.cpp:988
======= 8 =======
Assertion failure: (frame != nullptr) == (((HRESULT)(hr)) >= 0), at c:/builds/moz2_slave/m-cen-w32-d-000000000000000000/build/src/dom/media/platforms/wmf/WMFVideoMFTManager.cpp:988
Assertion failure: (frame != nullptr) == (((HRESULT)(hr)) >= 0), at c:/builds/moz2_slave/m-cen-w32-d-000000000000000000/build/src/dom/media/platforms/wmf/WMFVideoMFTManager.cpp:988
======= 9 =======
======= 10 =======
Assertion failure: (frame != nullptr) == (((HRESULT)(hr)) >= 0), at c:/builds/moz2_slave/m-cen-w32-d-000000000000000000/build/src/dom/media/platforms/wmf/WMFVideoMFTManager.cpp:988
Assertion failure: (frame != nullptr) == (((HRESULT)(hr)) >= 0), at c:/builds/moz2_slave/m-cen-w32-d-000000000000000000/build/src/dom/media/platforms/wmf/WMFVideoMFTManager.cpp:988
======= 11 =======
Assertion failure: (frame != nullptr) == (((HRESULT)(hr)) >= 0), at c:/builds/moz2_slave/m-cen-w32-d-000000000000000000/build/src/dom/media/platforms/wmf/WMFVideoMFTManager.cpp:988
Assertion failure: (frame != nullptr) == (((HRESULT)(hr)) >= 0), at c:/builds/moz2_slave/m-cen-w32-d-000000000000000000/build/src/dom/media/platforms/wmf/WMFVideoMFTManager.cpp:988
Assertion failure: (frame != nullptr) == (((HRESULT)(hr)) >= 0), at c:/builds/moz2_slave/m-cen-w32-d-000000000000000000/build/src/dom/media/platforms/wmf/WMFVideoMFTManager.cpp:988
Assertion failure: (frame != nullptr) == (((HRESULT)(hr)) >= 0), at c:/builds/moz2_slave/m-cen-w32-d-000000000000000000/build/src/dom/media/platforms/wmf/WMFVideoMFTManager.cpp:988
======= 12 =======
Assertion failure: (frame != nullptr) == (((HRESULT)(hr)) >= 0), at c:/builds/moz2_slave/m-cen-w32-d-000000000000000000/build/src/dom/media/platforms/wmf/WMFVideoMFTManager.cpp:988
======= 13 =======
======= 14 =======
Assertion failure: (frame != nullptr) == (((HRESULT)(hr)) >= 0), at c:/builds/moz2_slave/m-cen-w32-d-000000000000000000/build/src/dom/media/platforms/wmf/WMFVideoMFTManager.cpp:988
Assertion failure: (frame != nullptr) == (((HRESULT)(hr)) >= 0), at c:/builds/moz2_slave/m-cen-w32-d-000000000000000000/build/src/dom/media/platforms/wmf/WMFVideoMFTManager.cpp:988
Assertion failure: (frame != nullptr) == (((HRESULT)(hr)) >= 0), at c:/builds/moz2_slave/m-cen-w32-d-000000000000000000/build/src/dom/media/platforms/wmf/WMFVideoMFTManager.cpp:988
======= 15 =======
Assertion failure: (frame != nullptr) == (((HRESULT)(hr)) >= 0), at c:/builds/moz2_slave/m-cen-w32-d-000000000000000000/build/src/dom/media/platforms/wmf/WMFVideoMFTManager.cpp:988
======= 16 =======
Assertion failure: (frame != nullptr) == (((HRESULT)(hr)) >= 0), at c:/builds/moz2_slave/m-cen-w32-d-000000000000000000/build/src/dom/media/platforms/wmf/WMFVideoMFTManager.cpp:988
Assertion failure: (frame != nullptr) == (((HRESULT)(hr)) >= 0), at c:/builds/moz2_slave/m-cen-w32-d-000000000000000000/build/src/dom/media/platforms/wmf/WMFVideoMFTManager.cpp:988
Assertion failure: (frame != nullptr) == (((HRESULT)(hr)) >= 0), at c:/builds/moz2_slave/m-cen-w32-d-000000000000000000/build/src/dom/media/platforms/wmf/WMFVideoMFTManager.cpp:988
Assertion failure: (frame != nullptr) == (((HRESULT)(hr)) >= 0), at c:/builds/moz2_slave/m-cen-w32-d-000000000000000000/build/src/dom/media/platforms/wmf/WMFVideoMFTManager.cpp:988
======= 17 =======
Assertion failure: (frame != nullptr) == (((HRESULT)(hr)) >= 0), at c:/builds/moz2_slave/m-cen-w32-d-000000000000000000/build/src/dom/media/platforms/wmf/WMFVideoMFTManager.cpp:988
======= 18 =======
Assertion failure: mPromise, at c:\builds\moz2_slave\m-cen-w32-d-000000000000000000\build\src\obj-firefox\dist\include\mozilla/MozPromise.h:1030
Assertion failure: mPromise, at c:\builds\moz2_slave\m-cen-w32-d-000000000000000000\build\src\obj-firefox\dist\include\mozilla/MozPromise.h:1030
Assertion failure: mPromise, at c:\builds\moz2_slave\m-cen-w32-d-000000000000000000\build\src\obj-firefox\dist\include\mozilla/MozPromise.h:1030
======= 19 =======
Assertion failure: (frame != nullptr) == (((HRESULT)(hr)) >= 0), at c:/builds/moz2_slave/m-cen-w32-d-000000000000000000/build/src/dom/media/platforms/wmf/WMFVideoMFTManager.cpp:988
Assertion failure: (frame != nullptr) == (((HRESULT)(hr)) >= 0), at c:/builds/moz2_slave/m-cen-w32-d-000000000000000000/build/src/dom/media/platforms/wmf/WMFVideoMFTManager.cpp:988

Comment 13

[Alastor Wu [:alwu]]

Thanks Bob!

From the log in comment11, I saw that 
> [1836] WARNING: Invalid plane size: file c:/builds/moz2_slave/m-cen-w64-d-000000000000000000/build/src/dom/media/MediaData.cpp, line 118

This line indicated that we didn't create the video data successfully, so that we hit the assertion (return OK but doesn't have valid video data)

Next step, I'll survey why the plane size is invalid, maybe we need some check to avoid that case?

Comment 14

[Anthony Jones (:ajones, :kentuckyfriedtakahe, :k17e)]

I'm marking this as P1 on the basis that it should be easy to fix. If that is not the case then feel free to drop the priority.

Comment 15

[Emma Humphries ☕️🎸🧞‍♀️✨ (she/they) [:emceeaich] (Pacific Time) use needinfo]

This is an assigned P1 bug without activity in two weeks. 

If you intend to continue working on this bug for the current release/iteration/sprint, remove the 'stale-bug' keyword.

Otherwise we'll reset the priority of the bug back to '--' on Monday, August 28th.

Comment 16

[Bulk Bug Changes for mreavy's org]

Mass change P1->P2 to align with new Mozilla triage process

Comment 17

[Intermittent Failures Robot]

1 failures in 2628 pushes (0.0 failures/push) were associated with this bug in the last 7 days.

Repository breakdown:

• autoland: 1

Platform breakdown:

• windows7-32: 1

For more details, see:
https://treeherder.mozilla.org/intermittent-failures.html#/bugdetails?bug=1336930&startday=2019-01-28&endday=2019-02-03&tree=all