drm/amd/display: Fix null check for pipe_ctx->plane_state in resource_build_scaling_p...

commit 374c9faac5a763a05bc3f68ad9f73dab3c6aec90 upstream.

Null pointer dereference issue could occur when pipe_ctx->plane_state
is null. The fix adds a check to ensure 'pipe_ctx->plane_state' is not
null before accessing. This prevents a null pointer dereference.

Found by code review.

Fixes: 3be5262e353b ("drm/amd/display: Rename more dc_surface stuff to plane_state")
Reviewed-by: Alex Hung <alex.hung@amd.com>
Signed-off-by: Ma Ke <make24@iscas.ac.cn>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
(cherry picked from commit 63e6a77ccf239337baa9b1e7787cde9fa0462092)
Cc: stable@vger.kernel.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

diff --git a/drivers/gpu/drm/amd/display/dc/core/dc_resource.c b/drivers/gpu/drm/amd/display/dc/core/dc_resource.c
index d915020a429582a64a49925a4dfad86f7d4a3a36..f0eda0ba015600d6437b9891ad4f21ab85f04ccc 100644 (file)
--- a/drivers/gpu/drm/amd/display/dc/core/dc_resource.c
+++ b/drivers/gpu/drm/amd/display/dc/core/dc_resource.c
@@ -1455,7 +1455,8 @@ bool resource_build_scaling_params(struct pipe_ctx *pipe_ctx)
        DC_LOGGER_INIT(pipe_ctx->stream->ctx->logger);
 
        /* Invalid input */
-       if (!plane_state->dst_rect.width ||
+       if (!plane_state ||
+                       !plane_state->dst_rect.width ||
                        !plane_state->dst_rect.height ||
                        !plane_state->src_rect.width ||
                        !plane_state->src_rect.height) {