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)

x86
macOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED
Tracking Status
status1.9.2 --- .13-fixed
status1.9.1 --- .16-fixed

People

(Reporter: jfkthame, Assigned: jfkthame)

References

Details

Attachments

(1 file)

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.
This font is being rejected by the sanitizer because the final table in the font is not padded to a 4-byte boundary.
Assignee: nobody → jfkthame
Attachment #480278 - Flags: review?(jdaggett)
Attachment #480278 - Flags: review?(jdaggett) → review+
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
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 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+
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 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+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: