Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Font failure #2537

Closed
herk opened this issue Jan 8, 2013 · 2 comments
Closed

Font failure #2537

herk opened this issue Jan 8, 2013 · 2 comments

Comments

@herk
Copy link

herk commented Jan 8, 2013

http://redacted/test.pdf with PDF.js 0.7.1, FF 17.0.1 (Windows 7 and CentOS, both doing the same)

Console output:
[16:22:36.492] PDF 7d9399d5b4ca49d2172aab93c167f2e3 [1.6 Mac OS X 10.7.2 Quartz PDFContext / -
[16:22:36.894] downloadable font: table 'head': failed to parse table (font-family: "g_font_p0_1" style:normal weight:normal stretch:normal src index:0)
source: data:font/opentype;base64,AAEAAA
[16:22:36.895] downloadable font: rejected by sanitizer (font-family: "g_font_p0_1" style:normal weight:normal stretch:normal src index:0)
source: data:font/opentype;base64,AAEAAAANAIA

.... and a lot more of these in the web console.

@brendandahl
Copy link
Contributor

ERROR at ../src/head.cc:105 (bool ots::ots_head_parse(ots::OpenTypeFile *, const uint8_t *, size_t))
ERROR at ../src/ots.cc:540 (bool ::ProcessGeneric(ots::OpenTypeFile *, uint32_t, ots::OTSStream *, const uint8_t *, size_t, const std::vector &, ots::Buffer &))
Failed to sanitise file!

@mduan mduan closed this as completed in e4c3b45 Jan 31, 2013
yurydelendik added a commit that referenced this issue Jan 31, 2013
Add sanitization of truetype fonts to fix #2531 and #2537
@yurydelendik
Copy link
Contributor

Fixed by the pull request above

@yurydelendik yurydelendik reopened this Jan 31, 2013
yurydelendik added a commit to yurydelendik/pdf.js that referenced this issue May 2, 2014
yurydelendik added a commit that referenced this issue May 5, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants