Closed
Bug 601099
Opened 14 years ago
Closed 14 years ago
reference font used for reftest "load-badchecksum.html" is rejected by OTS sanitizer
Categories
(Core :: Layout: Text and Fonts, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: jfkthame, Assigned: jfkthame)
References
Details
Attachments
(1 file)
(deleted),
patch
|
jtd
:
review+
dveditz
:
approval1.9.2.13+
dveditz
:
approval1.9.1.16+
|
Details | Diff | Splinter Review |
The "good" reference font VeraBd-validchecksum.ttf, used by layout/reftests/font-face/load-badchecksum.html, is rejected by the OTS sanitizer. According to Apple's (not-very-thorough) ftxvalidator tool, there's a problem with the 'name' table.
Because this reference font is rejected, the reftest no longer works as intended once the sanitizer is deployed.
Assignee | ||
Updated•14 years ago
|
Blocks: CVE-2010-3768
Assignee | ||
Comment 1•14 years ago
|
||
This font is being rejected by the sanitizer because the final table in the font is not padded to a 4-byte boundary.
Assignee | ||
Comment 2•14 years ago
|
||
Assignee: nobody → jfkthame
Attachment #480278 -
Flags: review?(jdaggett)
Updated•14 years ago
|
Attachment #480278 -
Flags: review?(jdaggett) → review+
Assignee | ||
Comment 3•14 years ago
|
||
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 4•14 years ago
|
||
Comment on attachment 480278 [details] [diff] [review]
patch, v1 - add proper padding to the VeraBd-validchecksum.ttf font file
Requesting approval for 1.9.2.13. This is a test-only patch, fixing an error in one of the test fonts we use; we need to fix this on branch, otherwise landing bug 527276 will lead to reftest failures because the incorrect font will no longer be loaded.
Attachment #480278 -
Flags: approval1.9.2.13?
Comment 5•14 years ago
|
||
Comment on attachment 480278 [details] [diff] [review]
patch, v1 - add proper padding to the VeraBd-validchecksum.ttf font file
Approved for 1.9.2.13, a=dveditz for release-drivers
Attachment #480278 -
Flags: approval1.9.2.13? → approval1.9.2.13+
Assignee | ||
Comment 6•14 years ago
|
||
Pushed to 1.9.2:
http://hg.mozilla.org/releases/mozilla-1.9.2/rev/df5318538e6e
Assignee | ||
Updated•14 years ago
|
status1.9.2:
--- → .13-fixed
Assignee | ||
Comment 7•14 years ago
|
||
Comment on attachment 480278 [details] [diff] [review]
patch, v1 - add proper padding to the VeraBd-validchecksum.ttf font file
Requesting approval to land on 1.9.1, so that we can take the OTS font sanitizer there.
Attachment #480278 -
Flags: approval1.9.1.16?
Comment 8•14 years ago
|
||
Comment on attachment 480278 [details] [diff] [review]
patch, v1 - add proper padding to the VeraBd-validchecksum.ttf font file
Approved for 1.9.1.16, a=dveditz for release-drivers
Attachment #480278 -
Flags: approval1.9.1.16? → approval1.9.1.16+
Assignee | ||
Comment 9•14 years ago
|
||
status1.9.1:
--- → .16-fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•