[AMD Official Use Only - AMD Internal Distribution Only] Series is,
Reviewed-by: Tim Huang <[email protected]> > -----Original Message----- > From: Jesse Zhang <[email protected]> > Sent: Tuesday, May 21, 2024 3:18 PM > To: [email protected] > Cc: Deucher, Alexander <[email protected]>; Koenig, Christian > <[email protected]>; Huang, Tim <[email protected]>; Zhang, > Jesse(Jie) <[email protected]>; Zhang, Jesse(Jie) > <[email protected]> > Subject: [PATCH 4/4 V2] drm/admgpu: fix dereferencing null pointer context > > When user space sets an invalid ta type, the pointer context will be empty. > So it need to check the pointer context before using it > > Signed-off-by: Jesse Zhang <[email protected]> > Suggested-by: Tim Huang <[email protected]> > --- > drivers/gpu/drm/amd/amdgpu/amdgpu_psp_ta.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_psp_ta.c > b/drivers/gpu/drm/amd/amdgpu/amdgpu_psp_ta.c > index ca5c86e5f7cd..8e8afbd237bc 100644 > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_psp_ta.c > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_psp_ta.c > @@ -334,7 +334,7 @@ static ssize_t ta_if_invoke_debugfs_write(struct file > *fp, const char *buf, size > > set_ta_context_funcs(psp, ta_type, &context); > > - if (!context->initialized) { > + if (!context || !context->initialized) { > dev_err(adev->dev, "TA is not initialized\n"); > ret = -EINVAL; > goto err_free_shared_buf; > -- > 2.25.1
