From 0981068b75710397f08e0d2d776a0a2ea68d7117 Mon Sep 17 00:00:00 2001 From: Behdad Esfahbod Date: Sat, 28 Jul 2012 17:01:59 -0400 Subject: [PATCH] [GSUB/GPOS] Reject Context/ChainContext lookups with zero input --- src/hb-ot-layout-gsubgpos-private.hh | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/hb-ot-layout-gsubgpos-private.hh b/src/hb-ot-layout-gsubgpos-private.hh index 14d36b083..48caf6a8f 100644 --- a/src/hb-ot-layout-gsubgpos-private.hh +++ b/src/hb-ot-layout-gsubgpos-private.hh @@ -857,6 +857,7 @@ struct ContextFormat3 TRACE_SANITIZE (); if (!c->check_struct (this)) return TRACE_RETURN (false); unsigned int count = glyphCount; + if (unlikely (!glyphCount)) return TRACE_RETURN (false); if (!c->check_array (coverage, coverage[0].static_size, count)) return TRACE_RETURN (false); for (unsigned int i = 0; i < count; i++) if (!coverage[i].sanitize (c, this)) return TRACE_RETURN (false); @@ -1386,6 +1387,7 @@ struct ChainContextFormat3 if (!backtrack.sanitize (c, this)) return TRACE_RETURN (false); OffsetArrayOf &input = StructAfter > (backtrack); if (!input.sanitize (c, this)) return TRACE_RETURN (false); + if (unlikely (!input.len)) return TRACE_RETURN (false); OffsetArrayOf &lookahead = StructAfter > (input); if (!lookahead.sanitize (c, this)) return TRACE_RETURN (false); ArrayOf &lookup = StructAfter > (lookahead);