Re: [Mesa-dev] [PATCH] anv: fix possible stack corruption

2017-05-04 Thread Lionel Landwerlin
Something funny happened with the indentation, if you could fix this, this is : Reviewed-by: Lionel Landwerlin Thanks! - Lionel On 04/05/17 17:34, Grazvydas Ignotas wrote: ping Emil: the code this fixes is yours. Gražvydas On Tue, May 2, 2017 at 8:17 PM, Grazvydas Ignotas

Re: [Mesa-dev] [PATCH] anv: fix possible stack corruption

2017-05-04 Thread Grazvydas Ignotas
ping Emil: the code this fixes is yours. Gražvydas On Tue, May 2, 2017 at 8:17 PM, Grazvydas Ignotas wrote: > drmGetDevices2 takes count and not size. Probably hasn't caused problems > yet in practice and was missed as setups with more than 8 DRM devices > are not very common. > > Fixes: b1fb6e

[Mesa-dev] [PATCH] anv: fix possible stack corruption

2017-05-02 Thread Grazvydas Ignotas
drmGetDevices2 takes count and not size. Probably hasn't caused problems yet in practice and was missed as setups with more than 8 DRM devices are not very common. Fixes: b1fb6e8d "anv: do not open random render node(s)" Signed-off-by: Grazvydas Ignotas --- src/intel/vulkan/anv_device.c | 2 +-