Forked from
xorg / xserver
Source project has a limited visibility.
-
glamor_compute_transform_clipped_regions() uses a temporary box32 internally which is copied back to a box16 to init the regions16, thus causing a potential overflow. If an overflow occurs, the given region is invalid and the pixmap init region will fail. Simply check that the coordinates won't overflow when copying back to the box16, avoiding a crash later down the line in glamor. Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=101894 Signed-off-by:
Olivier Fourdan <ofourdan@redhat.com> Tested-by:
Fabrice Bellet <fabrice@bellet.info> Reviewed-by:
Adam Jackson <ajax@redhat.com>
glamor_compute_transform_clipped_regions() uses a temporary box32 internally which is copied back to a box16 to init the regions16, thus causing a potential overflow. If an overflow occurs, the given region is invalid and the pixmap init region will fail. Simply check that the coordinates won't overflow when copying back to the box16, avoiding a crash later down the line in glamor. Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=101894 Signed-off-by:
Olivier Fourdan <ofourdan@redhat.com> Tested-by:
Fabrice Bellet <fabrice@bellet.info> Reviewed-by:
Adam Jackson <ajax@redhat.com>