[prev in list] [next in list] [prev in thread] [next in thread]
List: cairo
Subject: Re: [cairo] an embarrassing memory leak
From: Behdad Esfahbod <behdad.esfahbod () gmail ! com>
Date: 2018-01-05 11:49:47
Message-ID: CAF63+7XqambC-6Cr-W5nWz6mpFFL1kuxTGJm8J1H+RJPniJNeA () mail ! gmail ! com
[Download RAW message or body]
[Attachment #2 (multipart/alternative)]
Hah. I didn't know that struct needs to be freed. Thought it's owned by
the FT_Face.
On Thu, Jan 4, 2018 at 9:03 PM, Matthias Clasen <matthias.clasen@gmail.com>
wrote:
> I forgot to free the FT_MM_Var struct in cairo_ft_apply_variations.
>
> Here is a fix:
>
> https://github.com/matthiasclasen/cairo/commit/
> 616fb7a9f2612f6cc3472542a70ba3e8ccf16584
>
--
behdad
http://behdad.org/
[Attachment #5 (text/html)]
<div dir="ltr">Hah. I didn't know that struct needs to be freed. Thought \
it's owned by the FT_Face.<br></div><div class="gmail_extra"><br><div \
class="gmail_quote">On Thu, Jan 4, 2018 at 9:03 PM, Matthias Clasen <span \
dir="ltr"><<a href="mailto:matthias.clasen@gmail.com" \
target="_blank">matthias.clasen@gmail.com</a>></span> wrote:<br><blockquote \
class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc \
solid;padding-left:1ex"><div dir="ltr"><div>I forgot to free the FT_MM_Var struct in \
cairo_ft_apply_variations.<br><br></div>Here is a fix:<br><br><a \
href="https://github.com/matthiasclasen/cairo/commit/616fb7a9f2612f6cc3472542a70ba3e8ccf16584" \
target="_blank">https://github.com/<wbr>matthiasclasen/cairo/commit/<wbr>616fb7a9f2612f6cc3472542a70ba3<wbr>e8ccf16584</a><br></div>
</blockquote></div><br><br clear="all"><br>-- <br><div class="gmail_signature" \
data-smartmail="gmail_signature">behdad<br><a href="http://behdad.org/" \
target="_blank">http://behdad.org/</a></div> </div>
[Attachment #6 (text/plain)]
--
cairo mailing list
cairo@cairographics.org
https://lists.cairographics.org/mailman/listinfo/cairo
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic