walkero
/
harfbuzz
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

[subset] don't visit lookup if covered glyph set has failed. 

If covered glyph set is in error then the same lookup can be recursed into repeatedly potentially causing a fuzzer timeout. Fixes: https://oss-fuzz.com/testcase-detail/5416421032067072.
This commit is contained in:
Garret Rieger 2021-04-05 12:53:08 -07:00 committed by Behdad Esfahbod
parent 4ec18c85e6
commit 64122b5a44
2 changed files with 2 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/hb-ot-layout-gsubgpos.hh
View File

@ -120,6 +120,8 @@ struct hb_closure_context_t :
} }
hb_set_t *covered_glyph_set = done_lookups_glyph_set->get (lookup_index); hb_set_t *covered_glyph_set = done_lookups_glyph_set->get (lookup_index);
if (covered_glyph_set->in_error ())
return true;
if (parent_active_glyphs ()->is_subset (covered_glyph_set)) if (parent_active_glyphs ()->is_subset (covered_glyph_set))
return true; return true;

BIN
test/fuzzing/fonts/clusterfuzz-testcase-minimized-hb-subset-fuzzer-5416421032067072 Normal file
View File

Binary file not shown.