Closed Bug 1570440 Opened 5 years ago Closed 5 years ago

[Web content] Firefox Crashes on CNN.COM (Windows7)

Categories

(Core :: WebVR, defect, P2)

Desktop
Windows 7
defect

Tracking

()

RESOLVED FIXED
mozilla70
Tracking Status
firefox-esr60 --- unaffected
firefox-esr68 --- unaffected
firefox68 --- unaffected
firefox69 --- unaffected
firefox70 --- fixed

People

(Reporter: muirpablo, Assigned: thomasmo)

References

Details

(Keywords: crash, regression, Whiteboard: [qa-70b-p2])

Crash Data

Attachments

(6 files)

(deleted), text/plain
Details
(deleted), text/plain
Details
(deleted), text/plain
Details
(deleted), text/plain
Details
(deleted), text/plain
Details
(deleted), application/x-zip-compressed
Details
Attached file crash.txt (deleted) —

[Description:]
When the browser is playing videos on CNN.com it constantly crashes on windows7

[Steps to reproduce:]

[Actual results]:
Firefox crashes while watching the video sometimed crashes after watching 2 or 3 videos / or when video ends it crashes.

[Expected results:]
Videos should play without any problem at all.

MacOs 10.14
Nightly 70.0a1 no issue
Beta 69.0.b9 no issue
Release 68.0.1 no issue

Windows7 64bit
Nightly 70.0a1 Crashes firefox
Release 68.0.1 no issue
Beta 69.0.b9 no issue

Ubuntu 18.04 64bit
Release 68.0.1 no issue
Beta 69.0.b9 no issue
Nightly 70.0a1 no issue

Crash report attached.
Will do mozregression as soon as i can

Attached file crash2.txt (deleted) —
Attached file cr3.txt (deleted) —

I cannot reproduce it on my Window 10.
Can you find the crash report related to this bug on about:crashes ? It'll be helpful if we can get the crash stack.

Flags: needinfo?(pablo.muir)

It looks like it crashes in VR related code. Can you upload your about:support to this bug? Is there a specific video that can cause crash?

Flags: needinfo?(pablo.muir)

Bugbug thinks this bug is a regression, but please revert this change in case of error.

Keywords: regression
Attached file aboutSupporttext.txt (deleted) —
Flags: needinfo?(pablo.muir)
Attached file aboutSupportraw.txt (deleted) —

Attached about:support,

No specific video, sometimes i am still watching the 1st one, then scroll and watch the second one and crashes
and sometimes i am watching the 1st one, then scroll, then start to watch the second one, then scroll and watch the 3rd one and crashes.

Maybe a graphics problem, but I guess it makes sense to leave it here until we've had a chance to investigate it further.

Priority: -- → P2

Pablo, is this specific to CNN? Other sites with videos don't have the issue? Are you using any kind of VR hardware with the device that is crashing?

Looks like a null pointer deref IMO. The crash address is constant and low (0x18) which looks like a believable address for members if the shmem here is null and then we try access members here.

Flags: needinfo?(pablo.muir)

Daosheng, could you take a look on this? This might be a regression caused by VR related code.

Flags: needinfo?(dmu)

It is only happening on CNN.com, other sites didn´t crash.
We are using just a regular PC at work, I will try with my personal home PC tonight.

Flags: needinfo?(pablo.muir)

Crashed with my Home PC too. Attached Home.zip with about:crashes and about:support
Machine is a skylake Intel with a Radeon hd7950 with latest drivers.

By going to http://www.cnn.com/2016/10/10/us/weather-matthew/index.html
-hit play on first video, then 10 seconds later scroll down
-hit play on second video, then 10 seconds later scroll down
and it crashed.

Attached file home.zip (deleted) —
Flags: needinfo?(dmu) → needinfo?(thomasmo)
Assignee: nobody → thomasmo
Flags: needinfo?(thomasmo)
Component: Audio/Video → WebVR

Bug understood, testing and validating local fix

(In reply to thomasmo from comment #17)

Bug understood, testing and validating local fix

Thomas, is your local fix ready to land? Thanks

Flags: needinfo?(thomasmo)

(In reply to Pascal Chevrel:pascalc from comment #18)

Thomas, is your local fix ready to land? Thanks

Almost--It is in CR. Sorry that the bug didn't get updated (I guess arc/phab can't handle more than one bug at a time?)
Here is the CR link: https://phabricator.services.mozilla.com/D40681

Flags: needinfo?(thomasmo)
Crash Signature: [@ mozilla::gfx::VRShMem::JoinShMem ]
Keywords: crash
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla70
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: