1488438 - Crash in xul.dll@0x5ad2f2 | style::media_queries::media_list::MediaList::parse

Status: RESOLVED DUPLICATE of bug 1471027

(Core :: CSS Parsing and Computation, defect)

Description

[Marcia Knous [:marcia]]

This bug was filed from the Socorro interface and is
report bp-f68e83d5-a018-4bc4-b368-610900180904.
=============================================================

Seen while looking at rc crash stats: https://bit.ly/2PxWFqb. This appears as far back as 62b8, but as we moved to rc the volume has increased. 

Code in this area was touched in 62 in Bug 1468846. ni on :emilio

Top 10 frames of crashing thread:

0 xul.dll xul.dll@0x5ad2f2 
1 xul.dll static struct style::media_queries::media_list::MediaList style::media_queries::media_list::MediaList::parse servo/components/style/media_queries/media_list.rs:58
2 xul.dll void geckoservo::glue::Servo_MediaList_SetText servo/ports/geckolib/glue.rs:3779
3 xul.dll mozilla::dom::MediaList::SetTextInternal layout/style/MediaList.cpp:115
4 xul.dll mozilla::dom::MediaList::Create layout/style/MediaList.cpp:95
5 xul.dll mozilla::dom::MediaQueryList::MediaQueryList layout/style/MediaQueryList.cpp:30
6 xul.dll nsIDocument::MatchMedia dom/base/nsDocument.cpp:6427
7 xul.dll nsGlobalWindowOuter::MatchMediaOuter dom/base/nsGlobalWindowOuter.cpp:3499
8 xul.dll nsGlobalWindowInner::MatchMedia dom/base/nsGlobalWindowInner.cpp:3548
9 xul.dll static bool mozilla::dom::WindowBinding::matchMedia dom/bindings/WindowBinding.cpp:2913

=============================================================

Comment 1

[Marcia Knous [:marcia]]

Philipp pointed out this probably a continuation of Bug 1471027.

Comment 2

[Emilio Cobos Álvarez [:emilio]]

It's pretty likely, that signature just changed because I moved the struct around. Are these OOM? an illegal instruction there sounds somewhat weird.

Comment 3

[Marcia Knous [:marcia]]

All the reports I looked at reason = EXCEPTION_ILLEGAL_INSTRUCTION. No helpful comments. Most of the URLs appear to be movie related.

Comment 4

[Mike Taylor [:miketaylr]]

(In reply to Emilio Cobos Álvarez (:emilio) from comment #2)
> It's pretty likely, that signature just changed because I moved the struct
> around. Are these OOM? an illegal instruction there sounds somewhat weird.

Let's dupe this and move over the signatures.