[GSUB] Fix context_length handling in Ligature too
authorBehdad Esfahbod <behdad@behdad.org>
Mon, 18 May 2009 08:21:53 +0000 (04:21 -0400)
committerBehdad Esfahbod <behdad@behdad.org>
Mon, 2 Nov 2009 19:40:12 +0000 (14:40 -0500)
src/hb-ot-layout-gsub-private.h
src/hb-ot-layout-gsubgpos-private.h

index a65a351..9f0b0d8 100644 (file)
@@ -318,14 +318,13 @@ struct Ligature {
 
     unsigned int i, j;
     unsigned int count = component.len;
-
-    if (HB_UNLIKELY (buffer->in_pos + count > buffer->in_length ||
-                    context_length < count))
-      return false; /* Not enough glyphs in input or context */
+    unsigned int end = MIN (buffer->in_length, buffer->in_pos + context_length);
+    if (HB_UNLIKELY (buffer->in_pos + count > end))
+      return false;
 
     for (i = 1, j = buffer->in_pos + 1; i < count; i++, j++) {
       while (!_hb_ot_layout_check_glyph_property (layout, IN_ITEM (j), lookup_flag, &property)) {
-       if (HB_UNLIKELY (j + count - i == buffer->in_length))
+       if (HB_UNLIKELY (j + count - i == end))
          return false;
        j++;
       }
index f5b900f..f9a1f6b 100644 (file)
@@ -221,7 +221,7 @@ static inline bool context_lookup (LOOKUP_ARGS_DEF,
 {
   /* First guess */
   if (HB_UNLIKELY (buffer->in_pos + inputCount > buffer->in_length ||
-                  context_length < inputCount))
+                  inputCount > context_length))
     return false;
 
   return match_input (LOOKUP_ARGS,
@@ -435,7 +435,8 @@ static inline bool chain_context_lookup (LOOKUP_ARGS_DEF,
 {
   /* First guess */
   if (HB_UNLIKELY (buffer->out_pos < backtrackCount ||
-                  buffer->in_pos + inputCount + lookaheadCount > buffer->in_length))
+                  buffer->in_pos + inputCount + lookaheadCount > buffer->in_length ||
+                  inputCount + lookaheadCount > context_length))
     return false;
 
   unsigned int offset;