1632161 - Crash in [@ geckoservo::glue::Servo_DeclarationBlock_GetCssText]

Status: RESOLVED WORKSFORME

(Core :: CSS Parsing and Computation, defect, P3)

Description

[Simon Giesecke [:sg] [he/him]]

This bug is for crash report bp-ef7e38a7-6415-44a8-9b27-392d20200422.

Top 10 frames of crashing thread:

0 xul.dll RustMozCrash mozglue/static/rust/wrappers.cpp:16
1 xul.dll mozglue_static::panic_hook mozglue/static/rust/lib.rs:89
2 xul.dll core::ops::function::Fn::call<fn ../f3e1a954d2ead4e2fc197c7da7d71e6c61bad196/src/libcore/ops/function.rs:232
3 xul.dll std::panicking::rust_panic_with_hook ../f3e1a954d2ead4e2fc197c7da7d71e6c61bad196//src/libstd/panicking.rs:475
4 xul.dll std::panicking::begin_panic<str*> ../f3e1a954d2ead4e2fc197c7da7d71e6c61bad196/src/libstd/panicking.rs:404
5 xul.dll geckoservo::glue::Servo_DeclarationBlock_GetCssText servo/ports/geckolib/glue.rs:4239
6 xul.dll nsAttrValue::ToString const dom/base/nsAttrValue.cpp:561
7 xul.dll AttrHasSubstring<const mozilla::ServoElementSnapshot>::<unnamed-tag>::operator const layout/style/GeckoBindings.cpp:880
8 xul.dll Gecko_AttrHasSubstring layout/style/GeckoBindings.cpp:945
9 xul.dll style::gecko::wrapper::{{impl}}::attr_matches servo/components/style/gecko/wrapper.rs:1953

There have been a few reports on Nightly starting with build id 20200417214650. Crash reason is "Locked::read_with called with a guard from an unrelated SharedRwLock "

Comment 1

[Emilio Cobos Álvarez (:emilio)]

That error makes close-to-zero sense, as we use a single static lock in Gecko (unlike Servo, which uses potentially multiple). So it's probably a corruption somewhere earlier, or similar... :(

Comment 2

[Mats Palmgren (inactive)]

There are 17 reported crashes in the past 6 months for this signature, with this version distribution:

77.0a1	7
75.0	4
60.4.0esr	1
68.5.0esr	1
68.6.0esr	1
71.0	1
72.0.2	1
73.0.1	1

so it appears 77.0a1 is over-represented given the relative user count on those versions.
3 out of the 7 crashes on v77 has URLs pointing some page on https://soundcloud.com/
All crashes are on Windows.

Comment 3

[BugBot [:suhaib / :marco/ :calixte]]

The severity field is not set for this bug.
:dholbert, could you have a look please?

For more information, please visit auto_nag documentation.

Comment 4

[BugBot [:suhaib / :marco/ :calixte]]

Closing because no crashes reported for 12 weeks.