[PATCH v2] drm/amd/display: Fix dmub_psr_destroy()
Kazlauskas, Nicholas
nicholas.kazlauskas at amd.com
Fri Feb 28 13:08:46 UTC 2020
On 2020-02-28 5:58 a.m., Dan Carpenter wrote:
> This is freeing the wrong variable so it will crash. It should be
> freeing "*dmub" instead of "dmub".
>
> Fixes: 4c1a1335dfe0 ("drm/amd/display: Driverside changes to support PSR in DMCUB")
> Signed-off-by: Dan Carpenter <dan.carpenter at oracle.com>
Reviewed-by: Nicholas Kazlauskas <nicholas.kazlauskas at amd.com>
Thanks!
> ---
> drivers/gpu/drm/amd/display/dc/dce/dmub_psr.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/amd/display/dc/dce/dmub_psr.c b/drivers/gpu/drm/amd/display/dc/dce/dmub_psr.c
> index 2c932c29f1f9..f0936cb3c056 100644
> --- a/drivers/gpu/drm/amd/display/dc/dce/dmub_psr.c
> +++ b/drivers/gpu/drm/amd/display/dc/dce/dmub_psr.c
> @@ -235,6 +235,6 @@ struct dmub_psr *dmub_psr_create(struct dc_context *ctx)
> */
> void dmub_psr_destroy(struct dmub_psr **dmub)
> {
> - kfree(dmub);
> + kfree(*dmub);
> *dmub = NULL;
> }
>
More information about the amd-gfx
mailing list