Intermittent /x-frame-options/sameorigin.sub.html | `XFO: SAMEORIGIN` allows same-origin nested in same-origin framing. - assert_equals: expected "Loaded" but got "Failed"
Categories
(Core :: DOM: Security, defect, P5)
Tracking
()
Tracking | Status | |
---|---|---|
firefox-esr60 | --- | unaffected |
firefox67 | --- | unaffected |
firefox68 | --- | unaffected |
firefox69 | --- | fixed |
People
(Reporter: intermittent-bug-filer, Assigned: sefeng)
References
(Regression)
Details
(Keywords: intermittent-failure, regression, Whiteboard: [domsecurity-intermittent][retriggered][wptsync upstream][stockwell fixed:patch])
Attachments
(1 file)
Filed by: cbrindusan [at] mozilla.com
Parsed log: https://treeherder.mozilla.org/logviewer.html#?job_id=245610084&repo=autoland
Full log: https://queue.taskcluster.net/v1/task/SJa8CzzTQWe2QdiAteNDOw/runs/0/artifacts/public/logs/live_backing.log
[task 2019-05-09T20:53:35.307Z] 20:53:35 INFO - TEST-START | /x-frame-options/sameorigin.sub.html
[task 2019-05-09T20:53:35.314Z] 20:53:35 INFO - Closing window 12884901889
[task 2019-05-09T20:53:35.411Z] 20:53:35 INFO - PID 24366 | -----------------------------------------------------
[task 2019-05-09T20:53:35.413Z] 20:53:35 INFO - PID 24366 | Suppressions used:
[task 2019-05-09T20:53:35.414Z] 20:53:35 INFO - PID 24366 | count bytes template
[task 2019-05-09T20:53:35.415Z] 20:53:35 INFO - PID 24366 | 28 864 nsComponentManagerImpl
[task 2019-05-09T20:53:35.417Z] 20:53:35 INFO - PID 24366 | 611 17713 libfontconfig.so
[task 2019-05-09T20:53:35.417Z] 20:53:35 INFO - PID 24366 | 1 29 libglib-2.0.so
[task 2019-05-09T20:53:35.418Z] 20:53:35 INFO - PID 24366 | -----------------------------------------------------
[task 2019-05-09T20:53:36.474Z] 20:53:36 INFO - PID 24366 | -----------------------------------------------------
[task 2019-05-09T20:53:36.474Z] 20:53:36 INFO - PID 24366 | Suppressions used:
[task 2019-05-09T20:53:36.477Z] 20:53:36 INFO - PID 24366 | count bytes template
[task 2019-05-09T20:53:36.478Z] 20:53:36 INFO - PID 24366 | 28 864 nsComponentManagerImpl
[task 2019-05-09T20:53:36.478Z] 20:53:36 INFO - PID 24366 | 611 17713 libfontconfig.so
[task 2019-05-09T20:53:36.478Z] 20:53:36 INFO - PID 24366 | 1 29 libglib-2.0.so
[task 2019-05-09T20:53:36.480Z] 20:53:36 INFO - PID 24366 | -----------------------------------------------------
[task 2019-05-09T20:53:36.706Z] 20:53:36 INFO -
[task 2019-05-09T20:53:36.706Z] 20:53:36 INFO - TEST-PASS | /x-frame-options/sameorigin.sub.html | XFO: SAMEORIGIN
allows same-origin framing.
[task 2019-05-09T20:53:36.706Z] 20:53:36 INFO - TEST-UNEXPECTED-FAIL | /x-frame-options/sameorigin.sub.html | XFO: SAMEORIGIN
allows same-origin nested in same-origin framing. - assert_equals: expected "Loaded" but got "Failed"
Comment hidden (Intermittent Failures Robot) |
Updated•5 years ago
|
Comment 2•5 years ago
|
||
Comment hidden (Intermittent Failures Robot) |
Comment 4•5 years ago
|
||
Retriggers show this is from Bug 1534012, Sean, please take a look.
Assignee | ||
Comment 5•5 years ago
|
||
hey apavel: Bug 1534012 got backed out, so I don't think it caused the regression?
Comment 6•5 years ago
|
||
Hi Sean, I know, I saw it. There are no occurrences since the bug got backed out. However, I looked into this and you should know when fixing bug 1534012 that this is an issue that needs to be fixed too.
Assignee | ||
Comment 7•5 years ago
|
||
apavel: yep, thanks. I'll keep that in mind!
Comment 8•5 years ago
|
||
The culprit seems to be https://bugzilla.mozilla.org/show_bug.cgi?id=1534012
Sean, can you please take a look at this?
Comment hidden (Intermittent Failures Robot) |
Comment 10•5 years ago
|
||
From IRC:
sefeng> apavel|sheriffduty: ah yeah looking...I thought I fixed it, but it is flaky
5:17 PM apavel|sheriffduty: sorry about that
Assignee | ||
Comment 11•5 years ago
|
||
We used to use 2 rAFs to stop the load event from racing with
onmessage event, but this was not the proper solution and it was
still racy, especially after Bug 1534012 is landed. So we fixed it
by instead letting racy postMessage to be sent, we wait for the
proper postMessage to be sent, and the test would automatically
timeout if the proper postMessage failed to sent.
Assignee | ||
Comment 12•5 years ago
|
||
Definitely my patch caused it. The actual problem was a buggy web platform test. The patch to fix the bug is up, and this is the try push https://treeherder.mozilla.org/#/jobs?repo=try&revision=ccc77a4b6a3be7927e888106e7b795e740222104. I'll wait for the try push result.
Comment 13•5 years ago
|
||
Pushed by sefeng@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/28a981abcc6c Fix a race condition in WPT related to postMessage r=smaug
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/16946 for changes under testing/web-platform/tests
Upstream web-platform-tests status checks passed, PR will merge once commit reaches central.
Comment 16•5 years ago
|
||
bugherder |
Updated•5 years ago
|
Comment hidden (Intermittent Failures Robot) |
Upstream PR merged
Assignee | ||
Updated•5 years ago
|
Updated•5 years ago
|
Comment hidden (Intermittent Failures Robot) |
Updated•5 years ago
|
Updated•2 years ago
|
Description
•