Closed
Bug 1486057
Opened 6 years ago
Closed 6 years ago
Crash in mozilla::extensions::URLInfo::CSpec const
Categories
(WebExtensions :: General, defect)
Tracking
(firefox-esr52 unaffected, firefox-esr60 unaffected, firefox61 unaffected, firefox62 unaffected, firefox63+ fixed)
RESOLVED
FIXED
mozilla63
Tracking | Status | |
---|---|---|
firefox-esr52 | --- | unaffected |
firefox-esr60 | --- | unaffected |
firefox61 | --- | unaffected |
firefox62 | --- | unaffected |
firefox63 | + | fixed |
People
(Reporter: calixte, Assigned: kmag)
References
(Blocks 1 open bug)
Details
(Keywords: crash, regression)
Crash Data
Attachments
(1 file)
This bug was filed from the Socorro interface and is
report bp-f5bb0a2d-04c3-40ce-8ce2-b0ed00180824.
=============================================================
Top 10 frames of crashing thread:
0 libxul.so mozilla::extensions::URLInfo::CSpec const toolkit/components/extensions/MatchPattern.cpp:174
1 libxul.so mozilla::extensions::URLInfo::Spec const toolkit/components/extensions/MatchPattern.cpp:183
2 libxul.so mozilla::extensions::MozDocumentMatcher::Matches const toolkit/components/extensions/MatchPattern.cpp:205
3 libxul.so mozilla::ExtensionPolicyService::InjectContentScripts const [clone .cold.289]
4 libxul.so mozilla::ForEachDocShell gcc/include/c++/6.4.0/functional:2127
5 libxul.so mozilla::ExtensionPolicyService::InjectContentScripts toolkit/components/extensions/ExtensionPolicyService.cpp:421
6 libxul.so mozilla::extensions::WebExtensionPolicy::InjectContentScripts toolkit/components/extensions/WebExtensionPolicy.cpp:337
7 libxul.so mozilla::dom::WebExtensionPolicy_Binding::injectContentScripts dom/bindings/WebExtensionPolicyBinding.cpp:1705
8 libxul.so bool mozilla::dom::binding_detail::GenericMethod<mozilla::dom::binding_detail::NormalThisPolicy, mozilla::dom::binding_detail::ThrowExceptions> dom/bindings/BindingUtils.cpp:3296
9 libxul.so js::InternalCallOrConstruct js/src/vm/Interpreter.cpp:449
=============================================================
There is 1 crash in nightly 63 with buildid 20180824100112. In analyzing the backtrace, the regression may have been introduced by patch [1] to fix bug 1484373.
[1] https://hg.mozilla.org/mozilla-central/rev?node=a7848dcb339e
Flags: needinfo?(kmaglione+bmo)
Reporter | ||
Updated•6 years ago
|
Crash Signature: [@ mozilla::extensions::URLInfo::CSpec const] → [@ mozilla::extensions::URLInfo::CSpec const]
[@ mozilla::extensions::URLInfo::Spec]
Reporter | ||
Updated•6 years ago
|
Crash Signature: [@ mozilla::extensions::URLInfo::CSpec const]
[@ mozilla::extensions::URLInfo::Spec] → [@ mozilla::extensions::URLInfo::CSpec const]
[@ mozilla::extensions::URLInfo::Spec]
[@ mozilla::extensions::URLInfo::Spec const]
Assignee | ||
Comment 1•6 years ago
|
||
Reporter | ||
Updated•6 years ago
|
Crash Signature: [@ mozilla::extensions::URLInfo::CSpec const]
[@ mozilla::extensions::URLInfo::Spec]
[@ mozilla::extensions::URLInfo::Spec const] → [@ mozilla::extensions::URLInfo::CSpec const]
[@ mozilla::extensions::URLInfo::Spec]
[@ mozilla::extensions::URLInfo::Spec const]
[@ mozilla::extensions::URLInfo::CSpec]
Updated•6 years ago
|
OS: Linux → All
Comment 2•6 years ago
|
||
Comment on attachment 9004022 [details]
Bug 1486057: Skip uninitialized windows when injecting content scripts. r=mixedpuppy
Shane Caraveo (:mixedpuppy) has approved the revision.
Attachment #9004022 -
Flags: review+
Updated•6 years ago
|
tracking-firefox63:
--- → +
Assignee | ||
Comment 3•6 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/292e35a4377f5f6c5534dad4f4df349010de41b9
Bug 1486057: Skip uninitialized windows when injecting content scripts. r=mixedpuppy
Comment 4•6 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
Reporter | ||
Updated•6 years ago
|
Assignee: nobody → kmaglione+bmo
Assignee | ||
Updated•6 years ago
|
Flags: needinfo?(kmaglione+bmo)
Comment 5•6 years ago
|
||
Is manual testing required on this bug? If yes, please provide some STR and the proper extension(if required) or set the “qe-verify -“ flag.
Thanks!
Flags: needinfo?(kmaglione+bmo)
Assignee | ||
Updated•6 years ago
|
Flags: needinfo?(kmaglione+bmo) → qe-verify-
You need to log in
before you can comment on or make changes to this bug.
Description
•