Re: [Qemu-devel] [PATCH] qxl-render: add more sanity checks

2014-08-29 Thread Gerd Hoffmann
On Fr, 2014-08-29 at 09:38 +0200, Gerd Hoffmann wrote: > Damn, the dirty rectangle values are signed integers. So the checks > added by commit 788fbf042fc6d5aaeab56757e6dad622ac5f0c21 are not good > enouth, we also have to make sure they are not negative. > > [ Note: There must be something broke

[Qemu-devel] [PATCH] qxl-render: add more sanity checks

2014-08-29 Thread Gerd Hoffmann
Damn, the dirty rectangle values are signed integers. So the checks added by commit 788fbf042fc6d5aaeab56757e6dad622ac5f0c21 are not good enouth, we also have to make sure they are not negative. [ Note: There must be something broken in spice-server so we get negative values in the first place.