[PATCH] Fix memory leak in cairo TextLayout::draw.

Mark Wielaard (via Code Review) gerrit at gerrit.libreoffice.org
Fri Jun 7 15:40:35 PDT 2013


Hi,

I have submitted a patch for review:

    https://gerrit.libreoffice.org/4193

To pull it, you can do:

    git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/93/4193/1

Fix memory leak in cairo TextLayout::draw.

Destroy cairo_font_options when done.

Change-Id: Ib49c07dee22fa59b4f6227c6b106a18b615aa060
---
M canvas/source/cairo/cairo_textlayout.cxx
1 file changed, 1 insertion(+), 0 deletions(-)



diff --git a/canvas/source/cairo/cairo_textlayout.cxx b/canvas/source/cairo/cairo_textlayout.cxx
index fe55665..35defcd 100644
--- a/canvas/source/cairo/cairo_textlayout.cxx
+++ b/canvas/source/cairo/cairo_textlayout.cxx
@@ -572,6 +572,7 @@
 
             cairo_restore( pSCairo.get() );
             cairo_font_face_destroy(font_face);
+            cairo_font_options_destroy(options);
         }
         return true;
     }

-- 
To view, visit https://gerrit.libreoffice.org/4193
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ib49c07dee22fa59b4f6227c6b106a18b615aa060
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Mark Wielaard <mark at klomp.org>



More information about the LibreOffice mailing list