[cairo] libpixman leak

Tim Janik timj at imendio.com
Fri Apr 18 04:20:10 PDT 2008


Hi Soeren.

We've had a leak reported with the following patch attached.
The test case used was running a Gtk+ example program in valgrind
with progressbars that show text. Does the following look like an
appropriate leak fix for you?

diff -u -r libcairo-1.4.10.orig/pixman/src/pixregion.c libcairo-1.4.10/pixman/src/pixregion.c
--- libcairo-1.4.10.orig/pixman/src/pixregion.c	2007-06-27 21:00:55.000000000 +0300
+++ libcairo-1.4.10/pixman/src/pixregion.c	2008-04-14 17:35:52.000000000 +0300
@@ -384,6 +384,7 @@
      }
      else if (!region->data->size)
      {
+	n++;
  	region->data = allocData(n);
  	if (!region->data)
  	    return pixman_break (region);


---
ciaoTJ


More information about the cairo mailing list