[HarfBuzz] harfbuzz: Branch 'master'

[Kenichi Ishibashi]

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> 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>
> 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
> http://lists.freedesktop.org/mailman/listinfo/harfbuzz
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/harfbuzz/attachments/20120904/bd47192e/attachment.html>