[PATCH xserver 2/2 v3] glamor: Avoid overflow between box32 and box16 box

Adam Jackson ajax at nwnk.net
Wed Sep 13 15:09:48 UTC 2017


On Wed, 2017-09-13 at 07:46 -0400, Olivier Fourdan wrote:
> Hi all,
> 
> Before I forget, would it be possible to consider these two patches for glamor:
> 
>   https://patchwork.freedesktop.org/series/27940/
> 
> This patch was successfully tested by Fabrice here (the reproducer is in bug #101894):
> 
>   https://lists.x.org/archives/xorg-devel/2017-July/054174.html
> 
> And Keith gave his R-b on the same patch here: 
> 
>   https://lists.x.org/archives/xorg-devel/2017-August/054416.html
> 
> (although that was on a previous -but identical- iteration of this patch)
> 
> The two patches are necessary because we need to handle empty source regions (and not just because of possible overflow).

lgtm, merged:

remote: I: patch #169162 updated using rev bd353e9b84e013fc34ed730319d5b63d20977903.
remote: I: patch #168774 updated using rev 9869dcb349b49f6d4cc2fab5d927cd8b1d1f463c.
remote: I: 2 patch(es) updated to state Accepted.
To ssh://git.freedesktop.org/git/xorg/xserver
   4486d199bd..9869dcb349  master -> master

- ajax


More information about the xorg-devel mailing list