summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorClément Bœsch <u@pkh.me>2013-11-16 21:18:53 +0100
committerClément Bœsch <u@pkh.me>2013-11-16 22:34:23 +0100
commit7e704b1e109b3c48b3175bf0eac7bb5bf0c66e75 (patch)
tree97b2e827b3617392fc090541b7037bea899aecb5
parentcee63a14835267a8a3665ccc93b9f9c34a225b5e (diff)
avcodec/vp9: add a never triggerable assert.
vp8_rac_get_tree() is called with a tree of size 3, so the returned value can not be outside [0;3]. All of the [0;3] cases are handled in the switch, so the assert should not be triggerable by any means. A similar change was introduced in 97962b2 / 72ca830, with the av_log()+AVERROR_INVALIDDATA form, suggesting it could be triggerable somehow. This assert might help static analyzer, or simply the reader.
-rw-r--r--libavcodec/vp9.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/libavcodec/vp9.c b/libavcodec/vp9.c
index 4978915bc4..e2cb388fc9 100644
--- a/libavcodec/vp9.c
+++ b/libavcodec/vp9.c
@@ -2768,6 +2768,8 @@ static int decode_sb(AVCodecContext *ctx, int row, int col, struct VP9Filter *lf
}
}
break;
+ default:
+ av_assert0(0);
}
} else if (vp56_rac_get_prob_branchy(&s->c, p[1])) {
bp = PARTITION_SPLIT;