mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [PATCH] drm: amd: display: Fix memory leakage
@ 2022-11-30  2:50 Konstantin Meskhidze
  2023-01-12 16:00 ` Rodrigo Siqueira Jordao
  0 siblings, 1 reply; 2+ messages in thread
From: Konstantin Meskhidze @ 2022-11-30  2:50 UTC (permalink / raw)
  To: harry.wentland
  Cc: sunpeng.li, Rodrigo.Siqueira, alexander.deucher,
	christian.koenig, Xinhui.Pan, airlied, daniel, Jun.Lei,
	Alvin.Lee2, meenakshikumar.somasundaram, martin.leung,
	Dillon.Varone, wenjing.liu, alex.hung, amd-gfx, dri-devel,
	linux-kernel, yusongping, hukeping, artem.kuzin

This commit fixes memory leakage in dc_construct_ctx() function.

Signed-off-by: Konstantin Meskhidze <konstantin.meskhidze@huawei.com>
---
 drivers/gpu/drm/amd/display/dc/core/dc.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/gpu/drm/amd/display/dc/core/dc.c b/drivers/gpu/drm/amd/display/dc/core/dc.c
index 997ab031f816..359e28d3567e 100644
--- a/drivers/gpu/drm/amd/display/dc/core/dc.c
+++ b/drivers/gpu/drm/amd/display/dc/core/dc.c
@@ -878,6 +878,7 @@ static bool dc_construct_ctx(struct dc *dc,
 
 	dc_ctx->perf_trace = dc_perf_trace_create();
 	if (!dc_ctx->perf_trace) {
+		kfree(dc_ctx);
 		ASSERT_CRITICAL(false);
 		return false;
 	}
-- 
2.25.1


^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH] drm: amd: display: Fix memory leakage
  2022-11-30  2:50 [PATCH] drm: amd: display: Fix memory leakage Konstantin Meskhidze
@ 2023-01-12 16:00 ` Rodrigo Siqueira Jordao
  0 siblings, 0 replies; 2+ messages in thread
From: Rodrigo Siqueira Jordao @ 2023-01-12 16:00 UTC (permalink / raw)
  To: Konstantin Meskhidze, harry.wentland
  Cc: sunpeng.li, alexander.deucher, christian.koenig, Xinhui.Pan,
	airlied, daniel, Jun.Lei, Alvin.Lee2,
	meenakshikumar.somasundaram, martin.leung, Dillon.Varone,
	wenjing.liu, alex.hung, amd-gfx, dri-devel, linux-kernel,
	yusongping, hukeping, artem.kuzin



On 11/29/22 21:50, Konstantin Meskhidze wrote:
> This commit fixes memory leakage in dc_construct_ctx() function.
> 
> Signed-off-by: Konstantin Meskhidze <konstantin.meskhidze@huawei.com>
> ---
>   drivers/gpu/drm/amd/display/dc/core/dc.c | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/gpu/drm/amd/display/dc/core/dc.c b/drivers/gpu/drm/amd/display/dc/core/dc.c
> index 997ab031f816..359e28d3567e 100644
> --- a/drivers/gpu/drm/amd/display/dc/core/dc.c
> +++ b/drivers/gpu/drm/amd/display/dc/core/dc.c
> @@ -878,6 +878,7 @@ static bool dc_construct_ctx(struct dc *dc,
>   
>   	dc_ctx->perf_trace = dc_perf_trace_create();
>   	if (!dc_ctx->perf_trace) {
> +		kfree(dc_ctx);
>   		ASSERT_CRITICAL(false);
>   		return false;
>   	}

Reviewed-by: Rodrigo Siqueira <Rodrigo.Siqueira@amd.com>

And applied to amd-staging-drm-next.

Thanks
Siqueira

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2023-01-12 16:10 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-11-30  2:50 [PATCH] drm: amd: display: Fix memory leakage Konstantin Meskhidze
2023-01-12 16:00 ` Rodrigo Siqueira Jordao

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®