On Fri, 06 May 2011 09:18:51 +0200, Soeren Sandmann <[email protected]> wrote: > Chris Wilson <[email protected]> writes: > > > + if (width == stride) { > > + width *= height; > > + height = 1; > > + } > > Do we need a check here that the mulitplication doesn't overflow? > pixman_multiply_overflows_int() can be used for that.
I think we are safe here, as we check for height*stride (in bytes) overflow during create_bits(). I'll work these small patches into a real submission with a corresponding addition to the test suite. Thanks, -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Pixman mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/pixman
