Commit 3b32b7f6 authored by Su Hui's avatar Su Hui Committed by Boris Brezillon
Browse files

drm/panthor: avoid garbage value in panthor_ioctl_dev_query() 



'priorities_info' is uninitialized, and the uninitialized value is copied
to user object when calling PANTHOR_UOBJ_SET(). Using memset to initialize
'priorities_info' to avoid this garbage value problem.

Fixes: f70000ef ("drm/panthor: Add DEV_QUERY_GROUP_PRIORITIES_INFO dev query")
Signed-off-by: default avatarSu Hui <suhui@nfschina.com>
Reviewed-by: default avatarDan Carpenter <dan.carpenter@linaro.org>
Reviewed-by: default avatarBoris Brezillon <boris.brezillon@collabora.com>
Reviewed-by: default avatarSteven Price <steven.price@arm.com>
Signed-off-by: default avatarBoris Brezillon <boris.brezillon@collabora.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20250119025828.1168419-1-suhui@nfschina.com
parent 02458fbf

drivers/gpu/drm/panthor/panthor_drv.c

+1 −0
Original line number Diff line number Diff line
@@ -802,6 +802,7 @@ static void panthor_query_group_priorities_info(struct drm_file *file, 
{

	int prio;



	memset(arg, 0, sizeof(*arg));

	for (prio = PANTHOR_GROUP_PRIORITY_REALTIME; prio >= 0; prio--) {

		if (!group_priority_permit(file, prio))

			arg->allowed_mask |= BIT(prio);