ref: 64b395cca690381c8bbc1b03852b669c995b9a3f
parent: c2b4753970b7f7bf1bbee3dda54d5eba53c797d1
author: Werner Lemberg <[email protected]>
date: Wed Dec 4 01:18:56 EST 2013
[sfnt] Fix handling of embedded bitmap strikes. This corrects the commit from 2013-11-21. Problem reported by Andrey Panov <[email protected]>. * src/sfnt/ttsbit.c (tt_sbit_decoder_load_bitmap): Fix logic to detect excessive bytes for bit-aligned bitmaps.
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,13 @@
+2013-12-04 Werner Lemberg <[email protected]>
+
+ [sfnt] Fix handling of embedded bitmap strikes.
+
+ This corrects the commit from 2013-11-21. Problem reported by
+ Andrey Panov <[email protected]>.
+
+ * src/sfnt/ttsbit.c (tt_sbit_decoder_load_bitmap): Fix logic to
+ detect excessive bytes for bit-aligned bitmaps.
+
2013-12-03 Werner Lemberg <[email protected]>
[truetype] Remove dead code.
@@ -101,7 +111,7 @@
2013-11-21 Werner Lemberg <[email protected]>
- [truetype] Improve handling of buggy embedded bitmap strikes.
+ [sfnt] Improve handling of buggy embedded bitmap strikes.
We are now able to successfully load `AppleMyoungJo.ttf'.
Problem reported by Hin-Tak Leung <[email protected]>.
--- a/src/sfnt/ttsbit.c
+++ b/src/sfnt/ttsbit.c
@@ -976,11 +976,21 @@
/* an excessive number of bytes in the image: If it is equal to */
/* the value for a byte-aligned glyph, use the other loading */
/* routine. */
+ /* */
+ /* Note that for some (width,height) combinations, where the */
+ /* width is not a multiple of 8, the sizes for bit- and */
+ /* byte-aligned data are equal, for example (7,7) or (15,6). We */
+ /* then prefer what `glyph_format' specifies. */
+
FT_UInt width = decoder->metrics->width;
FT_UInt height = decoder->metrics->height;
+ FT_UInt bit_size = ( width * height + 7 ) >> 3;
+ FT_UInt byte_size = height * ( ( width + 7 ) >> 3 );
- if ( height * ( ( width + 7 ) >> 3 ) == (FT_UInt)( p_limit - p ) )
+
+ if ( bit_size < byte_size &&
+ byte_size == (FT_UInt)( p_limit - p ) )
loader = tt_sbit_decoder_load_byte_aligned;
else
loader = tt_sbit_decoder_load_bit_aligned;