[PATCH] drm/rockchip: gem: check drm_gem_object_init() return value
邱晓金
qiuxiaojin at qs-tech.com
Tue May 5 17:57:22 PDT 2026
From ab8b1592209438a04c635d4c6c40c398230c1ba7 Mon Sep 17 00:00:00 2001
From: Qiu Xiaojin <qiuxiaojin at qs-tech.com>
Date: Wed, 6 May 2026 08:52:45 +0800
Subject: [PATCH] drm/rockchip: gem: check drm_gem_object_init() return value
rockchip_gem_alloc_object() currently ignores the return value of
drm_gem_object_init(). If initialization fails, the function can
return an incompletely initialized GEM object.
Handle the failure by freeing the allocated object and returning
ERR_PTR(ret).
Fixes: 38f993b7c59e ("drm/rockchip: Do not use DMA mapping API if attached to IOMMU domain")
Signed-off-by: Qiu Xiaojin <qiuxiaojin at qs-tech.com>
---
drivers/gpu/drm/rockchip/rockchip_drm_gem.c | 7 ++++++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_gem.c b/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
index 09d14a072d27..ef16b1fabec3 100644
--- a/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
+++ b/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
@@ -286,6 +286,7 @@ static struct rockchip_gem_object *
{
struct rockchip_gem_object *rk_obj;
struct drm_gem_object *obj;
+ int ret;
size = round_up(size, PAGE_SIZE);
@@ -297,7 +298,11 @@ static struct rockchip_gem_object *
obj->funcs = &rockchip_gem_object_funcs;
- drm_gem_object_init(drm, obj, size);
+ ret = drm_gem_object_init(drm, obj, size);
+ if (ret) {
+ kfree(rk_obj);
+ return ERR_PTR(ret);
+ }
return rk_obj;
}
--
2.34.1
--------------
Qiu Xiaojin
More information about the Linux-rockchip
mailing list