Skip to content

Commit

Permalink
DdiMedia_QueryConfigAttributes: do not check input value of num_attribs
Browse files Browse the repository at this point in the history
num_attribs parameter is an "out" parameter.  Therefore, there is no
contract with caller about its input value.

Fixes #15

Signed-off-by: U. Artie Eoff <ullysses.a.eoff@intel.com>
  • Loading branch information
uartie authored and XinfengZhang committed Dec 16, 2017
1 parent 81796c8 commit 4d89071
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion media_driver/linux/common/ddi/media_libva.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -2001,7 +2001,6 @@ static VAStatus DdiMedia_QueryConfigAttributes (
DDI_CHK_NULL(entrypoint, "Null entrypoint", VA_STATUS_ERROR_INVALID_PARAMETER);
DDI_CHK_NULL(ctx, "Null Ctx", VA_STATUS_ERROR_INVALID_CONTEXT);
DDI_CHK_NULL(num_attribs, "Null num_attribs", VA_STATUS_ERROR_INVALID_PARAMETER);
DDI_CHK_LESS(*num_attribs, VAConfigAttribTypeMax, "Invalid attribute number", VA_STATUS_ERROR_INVALID_PARAMETER);

pMediaCtx = DdiMedia_GetMediaContext(ctx);
DDI_CHK_NULL(pMediaCtx, "Null pMediaCtx", VA_STATUS_ERROR_INVALID_CONTEXT);
Expand Down

0 comments on commit 4d89071

Please sign in to comment.