[cairo-commit] 2 commits - src/cairo-clip.c src/cairo-surface-clipper.c

Chris Wilson ickle at kemper.freedesktop.org
Wed Oct 12 08:05:23 PDT 2011


 src/cairo-clip.c            |    1 +
 src/cairo-surface-clipper.c |    2 +-
 2 files changed, 2 insertions(+), 1 deletion(-)

New commits:
commit 17173f961369fe4210588a8cf56f9b8276d1e10f
Author: Chris Wilson <chris at chris-wilson.co.uk>
Date:   Wed Oct 12 16:01:42 2011 +0100

    clip: Free the freed clip pool on reset
    
    To keep valgrind happy.
    
    Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>

diff --git a/src/cairo-clip.c b/src/cairo-clip.c
index 11ba54d..0fabe39 100644
--- a/src/cairo-clip.c
+++ b/src/cairo-clip.c
@@ -809,4 +809,5 @@ void
 _cairo_clip_reset_static_data (void)
 {
     _freed_pool_reset (&clip_path_pool);
+    _freed_pool_reset (&clip_pool);
 }
commit a7d04703579fae9c2d944feb02116fc33f86f45f
Author: Chris Wilson <chris at chris-wilson.co.uk>
Date:   Wed Oct 12 15:56:36 2011 +0100

    surface-clipper: Fix path leak due to typo
    
    When finishing the path, it is better to call fini() rather than init()
    
    Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>

diff --git a/src/cairo-surface-clipper.c b/src/cairo-surface-clipper.c
index 1282eb2..0d2890c 100644
--- a/src/cairo-surface-clipper.c
+++ b/src/cairo-surface-clipper.c
@@ -94,7 +94,7 @@ _cairo_surface_clipper_intersect_clip_boxes (cairo_surface_clipper_t *clipper,
 					   CAIRO_FILL_RULE_WINDING,
 					   0.,
 					   CAIRO_ANTIALIAS_DEFAULT);
-    _cairo_path_fixed_init (&path);
+    _cairo_path_fixed_fini (&path);
 
     return status;
 }


More information about the cairo-commit mailing list