Commit d232afb1 authored by gehao's avatar gehao Committed by Alex Deucher
Browse files

drm/amd/display: prevent memory leak 



In dce6(0,1,4)_create_resource_pool and dce80_create_resource_pool
the allocated memory should be released if construct pool fails.

Reviewed-by: default avatarRodrigo Siqueira <Rodrigo.Siqueira@amd.com>
Signed-off-by: default avatargehao <gehao@kylinos.cn>
Signed-off-by: default avatarRodrigo Siqueira <Rodrigo.Siqueira@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 58ddbecb

drivers/gpu/drm/amd/display/dc/dce60/dce60_resource.c

+3 −0
Original line number Diff line number Diff line
@@ -1128,6 +1128,7 @@ struct resource_pool *dce60_create_resource_pool( 
	if (dce60_construct(num_virtual_links, dc, pool))

		return &pool->base;



	kfree(pool);

	BREAK_TO_DEBUGGER();

	return NULL;

}
@@ -1325,6 +1326,7 @@ struct resource_pool *dce61_create_resource_pool( 
	if (dce61_construct(num_virtual_links, dc, pool))

		return &pool->base;



	kfree(pool);

	BREAK_TO_DEBUGGER();

	return NULL;

}
@@ -1518,6 +1520,7 @@ struct resource_pool *dce64_create_resource_pool( 
	if (dce64_construct(num_virtual_links, dc, pool))

		return &pool->base;



	kfree(pool);

	BREAK_TO_DEBUGGER();

	return NULL;

}

drivers/gpu/drm/amd/display/dc/dce80/dce80_resource.c

+2 −0
Original line number Diff line number Diff line
@@ -1137,6 +1137,7 @@ struct resource_pool *dce80_create_resource_pool( 
	if (dce80_construct(num_virtual_links, dc, pool))

		return &pool->base;



	kfree(pool);

	BREAK_TO_DEBUGGER();

	return NULL;

}
@@ -1336,6 +1337,7 @@ struct resource_pool *dce81_create_resource_pool( 
	if (dce81_construct(num_virtual_links, dc, pool))

		return &pool->base;



	kfree(pool);

	BREAK_TO_DEBUGGER();

	return NULL;

}