758fda728b
[glyf] Don't accept gids higher than maxp's glyphs number
...
This specially becomes concerning on sub-components where a gvar table
that is sanitized using maxp's glyphs number overflows when a high gid
accepted here goes to it, maybe an additional check can be put there
also, this however feels to be enough.
Fixes https://crbug.com/oss-fuzz/20944
5 years ago
e642aab116
[subset] Add source_blob as a hb_subset_context_t field ( #2203 )
...
So no more double sanitizing source table.
5 years ago
e90213868b
Revert "collect_unicodes() to check gid < num_glyphs with cmap 12"
...
Didn't fix the case actually, making bots to fail.
This reverts commit 15b43a4104
5 years ago
61208401f4
[gvar] Use hb_bytes_t.check_range instead having in house one
...
And use TupleVarHeader calculated size for validity check.
Fixes https://crbug.com/oss-fuzz/20919 and possibly other gvar related issues
5 years ago
15b43a4104
collect_unicodes() to check gid < num_glyphs with cmap 12
...
fixes #2204
5 years ago
868ecf7b26
[draw] Add fuzzer runner
5 years ago
b0749bfaa5
[subset] GDEF LigCaretList subsetting support
5 years ago
002f0e20c4
reimplment serialize_int using check_assign()
5 years ago
14b134379d
[gvar] Minor, check whether sub_array result also have enough room
5 years ago
8eba66c1c6
[gvar] Fix invalid memory access by refactoring GlyphVarData fetch logic
...
Fixes https://crbug.com/oss-fuzz/20906
5 years ago
f44e1dc07d
Fix spelling.
5 years ago
5ad761b943
[subset] GDEF MarkGlyphSets subsetting support
5 years ago
fcd7f33bbb
[subset] GDEF glyphClassDef subsetting support
...
glyphClassDef uses the same ClassDef format. However, glyphClassDef table
uses predefined class values so we do not remap class values.
5 years ago
50129b03a1
Add a reverse () call to hb_array_t.
5 years ago
38c6598c1c
Switch to C style comments.
5 years ago
52b6e0baa0
When serializing cmap14 order the offsets from smallest to largest.
...
Current versions of OTS fail fonts with cmap 14's who's last offset does not point to the a block at the end of the table.
5 years ago
a99134c5be
add oss-fuzz 20886 test file
5 years ago
d0aaba5c50
fixes oss-fuzz 20886
...
hb_set_t::resize () is needed after compact()
5 years ago
05a25c1a5b
[cff] minor, remove unused fields
5 years ago
9fe0dc3464
[draw] Pass draw_helper_t itself around instead recreating it
...
Specially helpful if we want to change the design
5 years ago
1b8b863898
minor
5 years ago
4cdaa9d1f4
[glyf] Simplify contour end logic
...
So no need for infinite loop here
5 years ago
132fcfbc47
[fuzz] minor don't abort main.cc when the file was empty or not found
5 years ago
84163c83d3
[draw] Skip commands and paths not contributing anything
...
They aren't contributing to rendering and making issue for stroking, let's skip them
ourselves as Skia does also https://skia-review.googlesource.com/c/skia/+/268166
They are useful for extracting extents and so which that functionality won't be effected by this change.
5 years ago
073d4954e0
[draw] Port glyf path extract to draw_helper_t
5 years ago
0ebf3a4e62
[draw] Move common CFF path building logic to draw_helper_t
5 years ago
c400cb8863
Re-implement hb_set_t::del_range ( #2194 )
...
* optimize hb_set_del_range()
fix issue #2193
* fixed bug & added tests
* coding & comment tweaks
5 years ago
c21eb86bfd
Merge pull request #2163 from harfbuzz/absolute-link
...
added add_link_abs()
5 years ago
4081439d2a
tweak reflecting review & add test cases
5 years ago
152000d9c7
[fuzz] Practice variations on font object
5 years ago
036d868913
[draw] Add a fuzzer
...
Specially checks correctness of the API semantics:
* no move happens when a path is already opened with move-to.
* no path will be left open and close-path will happen at the end of opened paths.
* no path opens with a move-to and will be closed with no length.
* paths start and ending points matches.
* no line/quadratic/cubic command will be issued when no path is started.
5 years ago
de896278f7
coding & comment tweaks
5 years ago
2f97aa65e5
[cff] Make sure previous is ended on processing a seac
5 years ago
acc2d4738e
[cff] Make path extract easier to read and more defensive
5 years ago
b59eb54f70
[glyf] Refactor, move get_points inside Glyph
5 years ago
173b745da8
fixed bug & added tests
5 years ago
a5012e97c4
optimize hb_set_del_range()
...
fix issue #2193
5 years ago
96b71e802f
[fuzz] make the custom loader to handle multiple files
...
Actually this was the way it used to work :)
5 years ago
1f5a54c768
[gvar] fix infinite loop introduced by 11f3fca
...
The attempt on removing end_points had made the code unreadable
and has intrdouced infinite, fixed by making the code clear what
it tries to achieve.
5 years ago
f00eb4ebfa
[gvar] Don't compare against Null address
5 years ago
19b78d56cd
[gvar] Accept coord_count even if is higher than gvar's axisCount
5 years ago
197e2e929b
[gvar] Return gracefully even if gvar wasn't used at all
5 years ago
346ab9884b
[gvar] Return gracefully even if coords_count != gvar's axis count
...
It shouldn't be that common to have the situation, even so, no need to
reject entire a glyph even if gvar isn't used.
5 years ago
5f23a76e99
[cff] Don't remove coords const qualification
...
We should find some way to audit all the const removals, perhaps by
enabling -Wold-style-cast warning on the project and turning all the
implicit const removals to const_cast.
5 years ago
ff5223ba60
remove invalid glyphs from closure ( #2188 )
...
fix issue #2186
5 years ago
e17fd0d91c
[tools] More on py3 compatibility
5 years ago
0cf050a7b1
[draw] Merge consequent move-to commands of CFF/CFF2
5 years ago
86bd5a0ba1
[draw] End CFF paths with a line-to
...
Issue a line-to command when start and end point of a path isn't same, matches freetype also.
5 years ago
a46ba770ab
[number] minor
5 years ago
4aa354beba
chain second OffsetTo::serialize_copy() to first
5 years ago
Ebrahim Byagowi
Michiharu Ariza
Qunxin Liu
Evgeniy Reizner
Garret Rieger