[PATCH] virgl: fix caps ioctl size check.
Stéphane Marchesin
stephane.marchesin at gmail.com
Tue Feb 13 00:28:42 UTC 2018
On Mon, Feb 12, 2018 at 4:26 PM, Dave Airlie <airlied at gmail.com> wrote:
> From: Dave Airlie <airlied at redhat.com>
>
> The size check seems backwards, if userspace gives us a size
> larger it is probably asking for a newer version of the caps,
> so we just fill in the caps version we have at the smaller
> size, and it can work it out from there.
>
> Userspace will probably still need a fallback as old
> kernel are out there.
>
> Cc: <stable at vger.kernel.org>
> Signed-off-by: Dave Airlie <airlied at redhat.com>
Reviewed-by: Stéphane Marchesin <marcheu at chromium.org>
Thanks!
> ---
> drivers/gpu/drm/virtio/virtgpu_ioctl.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/virtio/virtgpu_ioctl.c b/drivers/gpu/drm/virtio/virtgpu_ioctl.c
> index 5720a0d4ac0a..5d5f79508100 100644
> --- a/drivers/gpu/drm/virtio/virtgpu_ioctl.c
> +++ b/drivers/gpu/drm/virtio/virtgpu_ioctl.c
> @@ -497,7 +497,7 @@ static int virtio_gpu_get_caps_ioctl(struct drm_device *dev,
> }
>
> size = vgdev->capsets[found_valid].max_size;
> - if (args->size > size) {
> + if (size > args->size) {
> spin_unlock(&vgdev->display_info_lock);
> return -EINVAL;
> }
> --
> 2.14.3
>
> _______________________________________________
> dri-devel mailing list
> dri-devel at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
More information about the dri-devel
mailing list