Commit f79d7749 authored by Harry Wentland's avatar Harry Wentland Committed by Alex Deucher
Browse files

drm/amd/display: Don't use target before null check 



It might blow up.

Signed-off-by: default avatarHarry Wentland <harry.wentland@amd.com>
Reviewed-by: default avatarTony Cheng <Tony.Cheng@amd.com>
Acked-by: default avatarHarry Wentland <Harry.Wentland@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent e12078c5

drivers/gpu/drm/amd/display/dc/core/dc_target.c

+7 −3
Original line number Diff line number Diff line
@@ -151,9 +151,9 @@ bool dc_target_set_cursor_attributes( 
	const struct dc_cursor_attributes *attributes)

{

	int i, j;

	struct core_target *target = DC_TARGET_TO_CORE(dc_target);

	struct core_dc *core_dc = DC_TO_CORE(target->ctx->dc);

	struct resource_context *res_ctx = &core_dc->current_context->res_ctx;

	struct core_target *target;

	struct core_dc *core_dc;

	struct resource_context *res_ctx;

	bool ret = false;



	if (NULL == dc_target) {
@@ -165,6 +165,10 @@ bool dc_target_set_cursor_attributes( 
			return false;

	}



	target = DC_TARGET_TO_CORE(dc_target);

	core_dc = DC_TO_CORE(target->ctx->dc);

	res_ctx = &core_dc->current_context->res_ctx;



	for (i = 0; i < dc_target->stream_count; i++) {

		const struct dc_stream *stream = dc_target->streams[i];