diff options
| author | Hersen Wu <[email protected]> | 2024-04-26 11:24:21 -0400 |
|---|---|---|
| committer | Alex Deucher <[email protected]> | 2024-05-20 16:20:24 -0400 |
| commit | c6bce984b8917b4a2fba17b5f19f1b44cf69d736 (patch) | |
| tree | c3e289f80c808ec07d9207a6d0dcc2de15cf2099 | |
| parent | 67a4888dc38c1d90d22b051cdb68c9144a6bc317 (diff) | |
drm/amd/display: Add NULL check within get_target_mpc_factor
[Why] Coverity reports NULL_RETURN warning.
[How] Add pointer NULL check.
Reviewed-by: Rodrigo Siqueira <[email protected]>
Signed-off-by: Hersen Wu <[email protected]>
Tested-by: Daniel Wheeler <[email protected]>
Signed-off-by: Alex Deucher <[email protected]>
| -rw-r--r-- | drivers/gpu/drm/amd/display/dc/dml2/dml2_dc_resource_mgmt.c | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/drivers/gpu/drm/amd/display/dc/dml2/dml2_dc_resource_mgmt.c b/drivers/gpu/drm/amd/display/dc/dml2/dml2_dc_resource_mgmt.c index 190fa2962271..02fb2cb1c43c 100644 --- a/drivers/gpu/drm/amd/display/dc/dml2/dml2_dc_resource_mgmt.c +++ b/drivers/gpu/drm/amd/display/dc/dml2/dml2_dc_resource_mgmt.c @@ -819,7 +819,16 @@ static unsigned int get_target_mpc_factor(struct dml2_context *ctx, /* get stream id of main stream */ main_stream = ctx->config.svp_pstate.callbacks.get_paired_subvp_stream(state, stream); + if (!main_stream) { + ASSERT(false); + return 1; + } + main_stream_status = ctx->config.callbacks.get_stream_status(state, main_stream); + if (!main_stream_status) { + ASSERT(false); + return 1; + } /* get plane id for associated main plane */ get_plane_id(ctx, state, main_stream_status->plane_states[plane_idx], |