Only regard images as pixbufs if they have identity transformations
authorSøren Sandmann Pedersen <ssp@redhat.com>
Sun, 7 Oct 2012 21:58:32 +0000 (17:58 -0400)
committerSøren Sandmann Pedersen <ssp@redhat.com>
Sun, 7 Oct 2012 22:00:09 +0000 (18:00 -0400)
In order for a src/mask pair to be considered a pixbuf, they have to
have identical transformations, but we don't check for that. Since the
only fast paths we have for pixbufs require identity transformations,
it sufficies to check that both source and mask are
untransformed.

This is also the reason that this bug can't be triggered by any test
code - if the source and mask had different transformations, we would
consider them a pixbuf, but then wouldn't take the fast path because
at least one of the transformations would be different from the
identity.

pixman/pixman.c

index bbe8b08..e3b6516 100644 (file)
@@ -605,6 +605,7 @@ pixman_image_composite32 (pixman_op_t      op,
     if ((mask_format == PIXMAN_a8r8g8b8 || mask_format == PIXMAN_a8b8g8r8) &&
        (src->type == BITS && src->bits.bits == mask->bits.bits)           &&
        (src->common.repeat == mask->common.repeat)                        &&
+       (src_flags & mask_flags & FAST_PATH_ID_TRANSFORM)                  &&
        (src_x == mask_x && src_y == mask_y))
     {
        if (src_format == PIXMAN_x8b8g8r8)