Marmelator/pdf.js.mirror
1
0
Fork 0
mirror of https://github.com/mozilla/pdf.js.git synced 2026-07-11 17:45:49 +02:00
Code Issues Packages Projects Releases Wiki Activity

Ensure that the built "hmtx" font table has valid widths (PR 21072 follow-up)

Browse Source 
With the changes in PR 21072 the `string16` helper is no longer being used when building the "hmtx" table, which accidentally removed the development mode assert.
This commit is contained in:
Jonas Jenwald 2026-04-09 23:07:30 +02:00
parent 94f8934d05
commit a69b9ad033
1 changed files with 10 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
src/core/fonts.js
View File

@ -332,6 +332,15 @@ function safeString16(value) {
return String.fromCharCode((value >> 8) & 0xff, value & 0xff); return String.fromCharCode((value >> 8) & 0xff, value & 0xff);
} }
function ensureInt16(v) {
if (typeof PDFJSDev === "undefined" || PDFJSDev.test("TESTING")) {
assert(
typeof v === "number" && Math.abs(v) < 2 ** 16,
`ensureInt16: Unexpected input "${v}".`
);
}
}
function isTrueTypeFile(file) { function isTrueTypeFile(file) {
const header = file.peekBytes(4); const header = file.peekBytes(4);
return ( return (
@ -3324,6 +3333,7 @@ class Font {
} else if (cffWidths) { } else if (cffWidths) {
width = Math.ceil(cffWidths[i] || 0); width = Math.ceil(cffWidths[i] || 0);
} }
ensureInt16(width);
data[pos++] = (width >> 8) & 0xff; data[pos++] = (width >> 8) & 0xff;
data[pos++] = width & 0xff; data[pos++] = width & 0xff;
// Use lsb=0, skip redundant assignment. // Use lsb=0, skip redundant assignment.