From 5144a4f56cd41f23e4576213e5f813656346fda7 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Corentin=20No=C3=ABl?= Date: Wed, 9 Apr 2025 18:19:58 +0200 Subject: [PATCH] virgl: Close handle on resource info failure MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit We just opened the GEM handle a few line before (or used drmPrimeFDToHandle to acquire it), on failure it is just better to close it. Signed-off-by: Corentin Noël Part-of: --- src/gallium/winsys/virgl/drm/virgl_drm_winsys.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/gallium/winsys/virgl/drm/virgl_drm_winsys.c b/src/gallium/winsys/virgl/drm/virgl_drm_winsys.c index 7b65841d385..9061e5d93e0 100644 --- a/src/gallium/winsys/virgl/drm/virgl_drm_winsys.c +++ b/src/gallium/winsys/virgl/drm/virgl_drm_winsys.c @@ -560,6 +560,8 @@ virgl_drm_winsys_resource_create_handle(struct virgl_winsys *qws, info_arg.bo_handle = res->bo_handle; if (drmIoctl(qdws->fd, DRM_IOCTL_VIRTGPU_RESOURCE_INFO, &info_arg)) { + drmCloseBufferHandle(qdws->fd, res->bo_handle); + /* close */ FREE(res); res = NULL;