Closed
Bug 1457284
Opened 7 years ago
Closed 7 years ago
Get latest Skia m66 cherry-picks
Categories
(Core :: Graphics, defect, P1)
Tracking
()
RESOLVED
FIXED
mozilla61
Tracking | Status | |
---|---|---|
firefox-esr52 | --- | unaffected |
firefox-esr60 | 61+ | fixed |
firefox59 | --- | unaffected |
firefox60 | - | wontfix |
firefox61 | + | fixed |
People
(Reporter: lsalzman, Assigned: lsalzman)
References
Details
(Keywords: sec-audit, Whiteboard: [gfx-noted][adv-main61-][adv-esr60.1-][post-critsmash-triage])
Attachments
(1 file)
39.54 KB,
patch
|
rhunt
:
review+
jcristau
:
approval-mozilla-beta-
jcristau
:
approval-mozilla-esr60+
|
Details | Diff | Splinter Review |
[Tracking Requested - why for this release]:
Skia's m66 branch (https://skia.googlesource.com/skia/+/chrome/m66) has included some new cherry-picks to deal with various security bugs, so we should take these before 60 beta hits release.
These are just changes that have already been rolled into their branch since weeks ago, so they should be safe for us to take.
Attachment #8971370 -
Flags: review?(rhunt)
Updated•7 years ago
|
Attachment #8971370 -
Flags: review?(rhunt) → review+
Comment 1•7 years ago
|
||
Julien, is this something you'd still be willing to take for a 60RC build, or should we punt for Fx61 & ESR 60.1?
Group: core-security → gfx-core-security
status-firefox59:
--- → unaffected
status-firefox-esr60:
--- → affected
tracking-firefox61:
--- → +
tracking-firefox-esr60:
--- → ?
Flags: needinfo?(jcristau)
Comment 2•7 years ago
|
||
Group: gfx-core-security → core-security-release
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: mozilla60 → mozilla61
Comment 3•7 years ago
|
||
(In reply to Ryan VanderMeulen [:RyanVM] from comment #1)
> Julien, is this something you'd still be willing to take for a 60RC build,
> or should we punt for Fx61 & ESR 60.1?
Leaning towards the latter at this stage.
Flags: needinfo?(jcristau)
Assignee | ||
Comment 4•7 years ago
|
||
Comment on attachment 8971370 [details] [diff] [review]
add Skia m66 cherry-picks
Approval Request Comment
[Feature/Bug causing the regression]: bug 1444506
[User impact if declined]: Potential security vulnerabilities.
[Is this code covered by automated tests?]: yes
[Has the fix been verified in Nightly?]: yes
[Needs manual test from QE? If yes, steps to reproduce]: no
[List of other uplifts needed for the feature/fix]:
[Is the change risky?]: no
[Why is the change risky/not risky?]: These are mostly security and correctness fixes that have had a few weeks of upstream testing.
[String changes made/needed]: none
Attachment #8971370 -
Flags: approval-mozilla-beta?
Comment 5•7 years ago
|
||
I checked with :abillings and he agrees with leaving this until 61/60.1, so marking 60 as wontfix.
Comment 6•7 years ago
|
||
Comment on attachment 8971370 [details] [diff] [review]
add Skia m66 cherry-picks
let's get this on esr60 for 60.1 though (attachment flag's not available yet but should be soon).
Attachment #8971370 -
Flags: approval-mozilla-beta? → approval-mozilla-beta-
Comment 7•7 years ago
|
||
Comment on attachment 8971370 [details] [diff] [review]
add Skia m66 cherry-picks
See comment 4. Note that this request is for the 60.1 release shipping alongside Fx61 in June.
Attachment #8971370 -
Flags: approval-mozilla-esr60?
Comment 8•7 years ago
|
||
Comment on attachment 8971370 [details] [diff] [review]
add Skia m66 cherry-picks
skia fixes for 60.1esr
Attachment #8971370 -
Flags: approval-mozilla-esr60? → approval-mozilla-esr60+
Comment 9•7 years ago
|
||
uplift |
Updated•6 years ago
|
Whiteboard: [gfx-noted] → [gfx-noted][adv-main61-]
Updated•6 years ago
|
Whiteboard: [gfx-noted][adv-main61-] → [gfx-noted][adv-main61-][adv-esr60.1-]
Updated•6 years ago
|
Flags: qe-verify-
Whiteboard: [gfx-noted][adv-main61-][adv-esr60.1-] → [gfx-noted][adv-main61-][adv-esr60.1-][post-critsmash-triage]
Updated•5 years ago
|
Group: core-security-release
You need to log in
before you can comment on or make changes to this bug.
Description
•