Closed
Bug 1341080
Opened 7 years ago
Closed 7 years ago
Crash in wrap_strdup | mozilla::Version::Version
Categories
(Core :: JavaScript Engine, defect)
Tracking
()
RESOLVED
FIXED
mozilla54
People
(Reporter: jujjyl, Assigned: bbouvier)
Details
(Keywords: crash)
Crash Data
Attachments
(1 file)
1.98 KB,
patch
|
luke
:
review+
gchang
:
approval-mozilla-aurora+
jcristau
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
This bug was filed from the Socorro interface and is report bp-4992537d-b1a6-4e0e-8c58-8846b2170220. ============================================================= FF Nightly crashes on slow script dialog coming up in Wasm.
Reporter | ||
Comment 1•7 years ago
|
||
Occurs somewhat deterministically, another callstack: https://crash-stats.mozilla.com/report/index/81391e44-8780-4b8b-b64c-575182170220
Reporter | ||
Comment 2•7 years ago
|
||
Unfortunately the test case is not public, but can be provided as needed.
Comment 3•7 years ago
|
||
D'oh, looks like another case of forgetting filename can be null (and, in this case, calling strdup on filename in DescribeScriptedCaller()).
Assignee | ||
Comment 4•7 years ago
|
||
The hardest was to make a test case.
Assignee | ||
Updated•7 years ago
|
status-firefox52:
--- → affected
status-firefox53:
--- → affected
tracking-firefox52:
--- → ?
tracking-firefox53:
--- → ?
Comment 5•7 years ago
|
||
Comment on attachment 8839237 [details] [diff] [review] fix.patch Review of attachment 8839237 [details] [diff] [review]: ----------------------------------------------------------------- Wow, that was fast, thanks!
Attachment #8839237 -
Flags: review?(luke) → review+
Pushed by bbouvier@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/4bccce8de17e Provide an empty filename if the wasm caller doesn't have one in DescribeScriptedCaller; r=luke
Comment 7•7 years ago
|
||
This seems to repro pretty easily for slow-script-dialog on Aurora/Beta, so it'd be good to uplift.
Assignee | ||
Comment 8•7 years ago
|
||
Comment on attachment 8839237 [details] [diff] [review] fix.patch Approval Request Comment [Feature/Bug causing the regression]: wasm [User impact if declined]: crashes in slow script dialog [Is this code covered by automated tests?]: yes [Has the fix been verified in Nightly?]: yes (landed this morning and tests passed) [Needs manual test from QE? If yes, steps to reproduce]: n/a [List of other uplifts needed for the feature/fix]: n/a [Is the change risky?]: not much [Why is the change risky/not risky?]: conservative change, one-line of code [String changes made/needed]: n/a
Attachment #8839237 -
Flags: approval-mozilla-beta?
Attachment #8839237 -
Flags: approval-mozilla-aurora?
Comment 9•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/4bccce8de17e
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
status-firefox54:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla54
Comment 11•7 years ago
|
||
Comment on attachment 8839237 [details] [diff] [review] fix.patch Fix a crash related to wasm and also include a test. Aurora53+.
Attachment #8839237 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Comment 12•7 years ago
|
||
Comment on attachment 8839237 [details] [diff] [review] fix.patch wasm crash fix for beta52 Should be in 52.0b9
Attachment #8839237 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Updated•7 years ago
|
Comment 13•7 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-aurora/rev/8e4f44727c5b
Comment 14•7 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-beta/rev/6d4cf6378334
Flags: in-testsuite+
Comment 15•7 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-esr52/rev/6d4cf6378334
status-firefox-esr52:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•