[PATCH 4/9] Fix pixmap double-frees on error paths.

Peter Hutterer peter.hutterer at who-t.net
Mon Sep 19 22:49:49 PDT 2011


On Mon, Sep 19, 2011 at 10:43:25PM -0700, Keith Packard wrote:
> On Tue, 20 Sep 2011 15:29:01 +1000, Peter Hutterer <peter.hutterer at who-t.net> wrote:
> 
> > if we create the pixmap but it cannot be added to the resource (and return
> > BadAlloc), shouldn't we destroy it?
> 
> AddResource frees the object if it fails

in that case, Reviewed-by: Peter Hutterer <peter.hutterer at who-t.net> for the
patch

Cheers,
  Peter


More information about the xorg-devel mailing list