[Git][freetype/freetype][gsoc-anurag-2022-docs] 3 commits: [sfnt] Handle

Anurag Thakur pushed to branch gsoc-anurag-2022-docs at FreeType / FreeType

Commits:

91a7fae7

by Dominik Röttsches at 2022-09-05T15:52:58+02:00

[sfnt] Handle variable `COLR` v1 fonts without delta set index map

Fixes #1178.

* src/sfnt/ttcolr.c (get_deltas_for_var_index_base): Set outer index to
0 and inner index to the delta index when retrieving deltas if the
`COLR` table has no delta set index map.

8e68439a

by Dominik Röttsches at 2022-09-06T17:18:17+02:00

[truetype] Reject 'fvar' loading when `num_instances` is too small.

* src/truetype/ttgxvar.c (TT_Get_MM_Var): Reject retrieving master when
'fvar' values locally do not match with sanitized values from initialization
at `sfnt_init_face` time.

Reported as

  https://bugs.chromium.org/p/chromium/issues/detail?id=1360295

ea114d2c
by Anurag Thakur at 2022-09-11T11:58:22+05:30
```
Begin adding comments for documentation
```

Changes:

src/sfnt/ttcolr.c


       }
       else
       {
-        /* TODO: Direct lookup case not implemented or tested yet. */
-        FT_ASSERT( 0 );
-        return 0;
+        outer_index = 0;
+        inner_index = loop_var_index;
       }
 
       deltas[i] = mm->get_item_delta( FT_FACE( face ), &colr->var_store,

src/smooth/ftsmooth.c

@@ -477,6 +477,7 @@
      slot->internal->flags |= FT_GLYPH_OWN_BITMAP;
 +    /* Outline points are expressed as 1/64th of a pixel (26.6 fixed-point format) */
      x_shift = 64 * -slot->bitmap_left;
      y_shift = 64 * -slot->bitmap_top;
      if ( bitmap->pixel_mode == FT_PIXEL_MODE_LCD_V )

src/truetype/ttgxvar.c

@@ -2207,6 +2207,11 @@
        FT_FRAME_END
      };
 +    /* `num_instances' holds the number of all named instances including  */
 +    /* the default instance, which might be missing in the table of named */
 +    /* instances (in 'fvar').  This value is validated in `sfobjs.c` and  */
 +    /* may be reset to 0 if consistency checks fail.                      */
 +    num_instances = (FT_UInt)face->root.style_flags >> 16;
      /* read the font data and set up the internal representation */
      /* if not already done                                       */
@@ -2231,6 +2236,17 @@
        if ( FT_STREAM_READ_FIELDS( fvar_fields, &fvar_head ) )
          goto Exit;
 +      /* If `num_instances` is larger, synthetization of the default  */
 +      /* instance is required.  If `num_instances` is smaller,        */
 +      /* however, the value has been reset to 0 in `sfnt_init_face`   */
 +      /* (in `sfobjs.c`); in this case we have underallocated `mmvar` */
 +      /* structs.                                                     */
 +      if ( num_instances < fvar_head.instanceCount )
 +      {
 +        error = FT_THROW( Invalid_Table );
 +        goto Exit;
 +      }
++
        usePsName = FT_BOOL( fvar_head.instanceSize ==
 + 4 * fvar_head.axisCount );
@@ -2249,11 +2265,6 @@
      else
        num_axes = face->blend->num_axis;
 -    /* `num_instances' holds the number of all named instances, */
 -    /* including the default instance which might be missing    */
 -    /* in fvar's table of named instances                       */
 -    num_instances = (FT_UInt)face->root.style_flags >> 16;
+-
      /* prepare storage area for MM data; this cannot overflow   */
      /* 32-bit arithmetic because of the size limits used in the */
      /* `fvar' table validity check in `sfnt_init_face'          */

From:	Anurag Thakur (@AdbhutDev)
Subject:	[Git][freetype/freetype][gsoc-anurag-2022-docs] 3 commits: [sfnt] Handle variable `COLR` v1 fonts without delta set index map
Date:	Sun, 11 Sep 2022 06:28:48 +0000