Make a copy of the region since pixman is currently taking ownership of it (ugh). Thanks to Vladimir Vukicevic <[email protected]> and Peter Dennis Bartok <[email protected]>.

Author: cworth-gh

AUTHORS

@@ -1,4 +1,5 @@
 Olivier Andrieu <[email protected]> PNG backend
+Peter Dennis Bartok <[email protected]> Bug fix for clipping
 Dave Beckett <[email protected]> Track rename of libpixman, build fixes
 Andrew Chant <[email protected]> Adding const where needed
 John Ellson <[email protected]> First font/glyph extents functions
@@ -13,6 +14,7 @@ David Reveman <[email protected]> New pattern API, OpenGL backend
 Jamey Sharp <[email protected]> Surface/font backend virtualization, XCB backend
 Bill Spitzak <[email protected]> Build fix to find Xrender.h without xrender.pc
 Sasha Vasko <[email protected]> Build fix to compile without xlib backend
+Vladimir Vukicevic <[email protected]> Bug fix for clipping
 Carl Worth <[email protected]> Original library, support for paths, images
 Richard D. Worth <[email protected]> Build fixes for cygwin
 

ChangeLog

@@ -1,5 +1,11 @@
 2004-08-14  Carl Worth  <[email protected]>
 
+	* src/cairo_image_surface.c
+	(_cairo_image_surface_set_clip_region): Make a copy of the region
+	since pixman is currently taking ownership of it (ugh). Thanks to
+	Vladimir Vukicevic <[email protected]> and Peter Dennis Bartok
+	<[email protected]>.
+
 	* autogen.sh (LANG): Explicitly set LANG=C to fix the awk
 	string->number conversion for user with locales that don't match
 	ASCII digit conventions.

src/cairo-image-surface.c

@@ -455,7 +455,20 @@ cairo_int_status_t
 _cairo_image_surface_set_clip_region (cairo_image_surface_t *surface,
 				      pixman_region16_t *region)
 {
-    pixman_image_set_clip_region (surface->pixman_image, region);
+    if (region) {
+        pixman_region16_t *rcopy;
+
+        rcopy = pixman_region_create();
+        /* pixman_image_set_clip_region expects to take ownership of the
+         * passed-in region, so we create a copy to give it. */
+	/* XXX: I think that's probably a bug in pixman. But its
+	 * memory management issues need auditing anyway, so a
+	 * workaround like this is fine for now. */
+        pixman_region_copy (rcopy, region);
+        pixman_image_set_clip_region (surface->pixman_image, rcopy);
+    } else {
+        pixman_image_set_clip_region (surface->pixman_image, region);
+    }
 
     return CAIRO_STATUS_SUCCESS;
 }

src/cairo_image_surface.c

@@ -455,7 +455,20 @@ cairo_int_status_t
 _cairo_image_surface_set_clip_region (cairo_image_surface_t *surface,
 				      pixman_region16_t *region)
 {
-    pixman_image_set_clip_region (surface->pixman_image, region);
+    if (region) {
+        pixman_region16_t *rcopy;
+
+        rcopy = pixman_region_create();
+        /* pixman_image_set_clip_region expects to take ownership of the
+         * passed-in region, so we create a copy to give it. */
+	/* XXX: I think that's probably a bug in pixman. But its
+	 * memory management issues need auditing anyway, so a
+	 * workaround like this is fine for now. */
+        pixman_region_copy (rcopy, region);
+        pixman_image_set_clip_region (surface->pixman_image, rcopy);
+    } else {
+        pixman_image_set_clip_region (surface->pixman_image, region);
+    }
 
     return CAIRO_STATUS_SUCCESS;
 }