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

2017-07-26 Thread Olivier Fourdan
Hi Michel, > > > 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

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

2017-07-26 Thread Olivier Fourdan
Hi Michel, > > 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. > >

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

2017-07-26 Thread Michel Dänzer
On 26/07/17 04:51 PM, Olivier Fourdan wrote: > 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