aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJames Almer <jamrial@gmail.com>2019-11-17 10:26:53 -0300
committerJames Almer <jamrial@gmail.com>2019-12-01 16:53:11 -0300
commit553c1431ac7dae44fb46285121bca176045fe6e7 (patch)
treef19a34b09194f058c9f8268d4368de831e75d484
parenteced91afa5fcdfe78961f88dbbfb40b421db6952 (diff)
downloadffmpeg-553c1431ac7dae44fb46285121bca176045fe6e7.tar.gz
Revert "avcodec/cbs_av1_syntax_template: Check ref_frame_idx before use"
This reverts commit 8174e5c77d8a94b57b6b1bcbb90728cf8b08ab6b. It's no longer needed after the previous commit. Signed-off-by: James Almer <jamrial@gmail.com>
-rw-r--r--libavcodec/cbs_av1_syntax_template.c7
1 files changed, 3 insertions, 4 deletions
diff --git a/libavcodec/cbs_av1_syntax_template.c b/libavcodec/cbs_av1_syntax_template.c
index 796897ab79..6c4816f964 100644
--- a/libavcodec/cbs_av1_syntax_template.c
+++ b/libavcodec/cbs_av1_syntax_template.c
@@ -530,17 +530,16 @@ static int FUNC(frame_size_with_refs)(CodedBitstreamContext *ctx, RWContext *rw,
for (i = 0; i < AV1_REFS_PER_FRAME; i++) {
flags(found_ref[i], 1, i);
if (current->found_ref[i]) {
- AV1ReferenceFrameState *ref;
+ AV1ReferenceFrameState *ref =
+ &priv->ref[current->ref_frame_idx[i]];
- if (current->ref_frame_idx[i] < 0 ||
- !priv->ref[current->ref_frame_idx[i]].valid) {
+ if (!ref->valid) {
av_log(ctx->log_ctx, AV_LOG_ERROR,
"Missing reference frame needed for frame size "
"(ref = %d, ref_frame_idx = %d).\n",
i, current->ref_frame_idx[i]);
return AVERROR_INVALIDDATA;
}
- ref = &priv->ref[current->ref_frame_idx[i]];
priv->upscaled_width = ref->upscaled_width;
priv->frame_width = ref->frame_width;