[HarfBuzz] harfbuzz: Branch 'master'

Behdad Esfahbod behdad at behdad.org
Tue Sep 4 11:56:00 PDT 2012


Ah, my bad.  It really bothers me that GCC doesn't crash on it.  So I
reorganized code to make sure the same mistake won't happen again.  I'm
planning on removing the code in Arabic shape fallback soon though.

behdad

On 09/04/2012 02:31 AM, Kenichi Ishibashi wrote:
> Hi Behdad,
> 
> Thank you for the change, but it seems that we need to do it for
> hb_apply_context_t(), not hb_would_apply_context_t() to fix the crash :)
> 
> 
> 
> On Tue, Sep 4, 2012 at 9:19 AM, Behdad Esfahbod <behdad at kemper.freedesktop.org
> <mailto:behdad at kemper.freedesktop.org>> wrote:
> 
>      src/hb-ot-layout-gsubgpos-private.hh |    2 +-
>      1 file changed, 1 insertion(+), 1 deletion(-)
> 
>     New commits:
>     commit f8fa2b5cf67b02d74514dec7885d03de73ec7349
>     Author: Behdad Esfahbod <behdad at behdad.org <mailto:behdad at behdad.org>>
>     Date:   Mon Sep 3 20:19:46 2012 -0400
> 
>         Fix possible NULL dereference
> 
>         As reported by Kenichi Ishibashi.
> 
>     diff --git a/src/hb-ot-layout-gsubgpos-private.hh
>     b/src/hb-ot-layout-gsubgpos-private.hh
>     index 00bc563..40d5c57 100644
>     --- a/src/hb-ot-layout-gsubgpos-private.hh
>     +++ b/src/hb-ot-layout-gsubgpos-private.hh
>     @@ -92,7 +92,7 @@ struct hb_would_apply_context_t
>                                   glyphs (glyphs_),
>                                   len (len_),
>                                   zero_context (zero_context_),
>     -                             digest (*digest_),
>     +                             digest (digest_ ? *digest_ : hb_set_digest_t()),
>                                   debug_depth (0) {};
>      };
> 
>     _______________________________________________
>     HarfBuzz mailing list
>     HarfBuzz at lists.freedesktop.org <mailto:HarfBuzz at lists.freedesktop.org>
>     http://lists.freedesktop.org/mailman/listinfo/harfbuzz
> 
> 



More information about the HarfBuzz mailing list