[Devel] [PATCH RHEL7 COMMIT] ms/drm/qxl: qxl_release leak in qxl_draw_dirty_fb()
Konstantin Khorenko
khorenko at virtuozzo.com
Mon Apr 27 14:05:58 MSK 2020
The commit is pushed to "branch-rh7-3.10.0-1127.vz7.150.x-ovz" and will appear at https://src.openvz.org/scm/ovz/vzkernel.git
after rh7-3.10.0-1127.vz7.150.4
------>
commit c84dff6b84d88f51b64a34a75b6d46f90d447c71
Author: Vasily Averin <vvs at virtuozzo.com>
Date: Mon Apr 27 14:05:57 2020 +0300
ms/drm/qxl: qxl_release leak in qxl_draw_dirty_fb()
"ret" should be changed to release allocated struct qxl_release.
Cc: stable at vger.kernel.org
mFixes: 8002db6336dd ("qxl: convert qxl driver to proper use for reservations")
Signed-off-by: Vasily Averin <vvs at virtuozzo.com>
---
drivers/gpu/drm/qxl/qxl_draw.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/qxl/qxl_draw.c b/drivers/gpu/drm/qxl/qxl_draw.c
index c408bb83c7a92..4a43b5ba4aa07 100644
--- a/drivers/gpu/drm/qxl/qxl_draw.c
+++ b/drivers/gpu/drm/qxl/qxl_draw.c
@@ -348,9 +348,10 @@ void qxl_draw_dirty_fb(struct qxl_device *qdev,
goto out_release_backoff;
rects = drawable_set_clipping(qdev, num_clips, clips_bo);
- if (!rects)
+ if (!rects) {
+ ret = -EINVAL;
goto out_release_backoff;
-
+ }
drawable = (struct qxl_drawable *)qxl_release_map(qdev, release);
drawable->clip.type = SPICE_CLIP_TYPE_RECTS;
More information about the Devel
mailing list