frontends/va: Fix *num_entrypoints check.
Fix defect reported by Coverity Scan.
Dereference before null check (REVERSE_INULL)
check_after_deref: Null-checking num_entrypoints suggests that it
may be null, but it has already been dereferenced on all paths
leading to the check.
Fixes: 5bcaa1b9e9
("st/va: add encode entrypoint v2")
Signed-off-by: Vinson Lee <vlee@freedesktop.org>
Reviewed-by: Pierre-Eric Pelloux-Prayer <pierre-eric.pelloux-prayer@amd.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/7360>
This commit is contained in:
parent
1beb477908
commit
7820c8c13f
|
@ -99,7 +99,7 @@ vlVaQueryConfigEntrypoints(VADriverContextP ctx, VAProfile profile,
|
||||||
PIPE_VIDEO_CAP_SUPPORTED))
|
PIPE_VIDEO_CAP_SUPPORTED))
|
||||||
entrypoint_list[(*num_entrypoints)++] = VAEntrypointEncSlice;
|
entrypoint_list[(*num_entrypoints)++] = VAEntrypointEncSlice;
|
||||||
|
|
||||||
if (num_entrypoints == 0)
|
if (*num_entrypoints == 0)
|
||||||
return VA_STATUS_ERROR_UNSUPPORTED_PROFILE;
|
return VA_STATUS_ERROR_UNSUPPORTED_PROFILE;
|
||||||
|
|
||||||
assert(*num_entrypoints <= ctx->max_entrypoints);
|
assert(*num_entrypoints <= ctx->max_entrypoints);
|
||||||
|
|
Loading…
Reference in New Issue