linux-kernel - Re: [PATCH v4.19-v6.1] drm/amdgpu: Using uninitialized value *size when calling

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <2024082955-luxury-agreed-36e6@gregkh>
Date: Thu, 29 Aug 2024 18:19:01 +0200
From: Greg KH <gregkh@...uxfoundation.org>
To: Vamsi Krishna Brahmajosyula <vamsi-krishna.brahmajosyula@...adcom.com>
Cc: stable@...r.kernel.org, jesse.zhang@....com, alexander.deucher@....com,
	sashal@...nel.org, christian.koenig@....com, Xinhui.Pan@....com,
	airlied@...il.com, daniel@...ll.ch, amd-gfx@...ts.freedesktop.org,
	dri-devel@...ts.freedesktop.org, linux-kernel@...r.kernel.org,
	ajay.kaher@...adcom.com, alexey.makhalov@...adcom.com,
	vasavi.sirnapalli@...adcom.com
Subject: Re: [PATCH v4.19-v6.1] drm/amdgpu: Using uninitialized value *size
 when calling

On Wed, Aug 28, 2024 at 10:15:56AM -0500, Vamsi Krishna Brahmajosyula wrote:
> From: Jesse Zhang <jesse.zhang@....com>
> 
> [ Upstream commit 88a9a467c548d0b3c7761b4fd54a68e70f9c0944 ]
> 
> Initialize the size before calling amdgpu_vce_cs_reloc, such as case 0x03000001.
> V2: To really improve the handling we would actually
>    need to have a separate value of 0xffffffff.(Christian)
> 
> Signed-off-by: Jesse Zhang <jesse.zhang@....com>
> Suggested-by: Christian König <christian.koenig@....com>
> Reviewed-by: Christian König <christian.koenig@....com>
> Signed-off-by: Alex Deucher <alexander.deucher@....com>
> Signed-off-by: Sasha Levin <sashal@...nel.org>
> Signed-off-by: Vamsi Krishna Brahmajosyula <vamsi-krishna.brahmajosyula@...adcom.com>
> ---
>  drivers/gpu/drm/amd/amdgpu/amdgpu_vce.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)

Now queued up, thanks.

greg k-h