Commit ab56e62e authored by Michael Niedermayer's avatar Michael Niedermayer

avcodec/vp5/6/8: use vpX_rac_is_end()

Reviewed-by: 's avatarPeter Ross <pross@xvid.org>
Signed-off-by: 's avatarMichael Niedermayer <michael@niedermayer.cc>
parent 5937f055
...@@ -183,7 +183,7 @@ static int vp5_parse_coeff(VP56Context *s) ...@@ -183,7 +183,7 @@ static int vp5_parse_coeff(VP56Context *s)
int b, i, cg, idx, ctx, ctx_last; int b, i, cg, idx, ctx, ctx_last;
int pt = 0; /* plane type (0 for Y, 1 for U or V) */ int pt = 0; /* plane type (0 for Y, 1 for U or V) */
if (c->end <= c->buffer && c->bits >= 0) { if (vpX_rac_is_end(c)) {
av_log(s->avctx, AV_LOG_ERROR, "End of AC stream reached in vp5_parse_coeff\n"); av_log(s->avctx, AV_LOG_ERROR, "End of AC stream reached in vp5_parse_coeff\n");
return AVERROR_INVALIDDATA; return AVERROR_INVALIDDATA;
} }
......
...@@ -473,7 +473,7 @@ static int vp6_parse_coeff(VP56Context *s) ...@@ -473,7 +473,7 @@ static int vp6_parse_coeff(VP56Context *s)
int b, i, cg, idx, ctx; int b, i, cg, idx, ctx;
int pt = 0; /* plane type (0 for Y, 1 for U or V) */ int pt = 0; /* plane type (0 for Y, 1 for U or V) */
if (c->end <= c->buffer && c->bits >= 0) { if (vpX_rac_is_end(c)) {
av_log(s->avctx, AV_LOG_ERROR, "End of AC stream reached in vp6_parse_coeff\n"); av_log(s->avctx, AV_LOG_ERROR, "End of AC stream reached in vp6_parse_coeff\n");
return AVERROR_INVALIDDATA; return AVERROR_INVALIDDATA;
} }
......
...@@ -658,7 +658,7 @@ static int vp7_decode_frame_header(VP8Context *s, const uint8_t *buf, int buf_si ...@@ -658,7 +658,7 @@ static int vp7_decode_frame_header(VP8Context *s, const uint8_t *buf, int buf_si
s->fade_present = vp8_rac_get(c); s->fade_present = vp8_rac_get(c);
} }
if (c->end <= c->buffer && c->bits >= 0) if (vpX_rac_is_end(c))
return AVERROR_INVALIDDATA; return AVERROR_INVALIDDATA;
/* E. Fading information for previous frame */ /* E. Fading information for previous frame */
if (s->fade_present && vp8_rac_get(c)) { if (s->fade_present && vp8_rac_get(c)) {
...@@ -693,7 +693,7 @@ static int vp7_decode_frame_header(VP8Context *s, const uint8_t *buf, int buf_si ...@@ -693,7 +693,7 @@ static int vp7_decode_frame_header(VP8Context *s, const uint8_t *buf, int buf_si
vp78_update_pred16x16_pred8x8_mvc_probabilities(s, VP7_MVC_SIZE); vp78_update_pred16x16_pred8x8_mvc_probabilities(s, VP7_MVC_SIZE);
} }
if (c->end <= c->buffer && c->bits >= 0) if (vpX_rac_is_end(c))
return AVERROR_INVALIDDATA; return AVERROR_INVALIDDATA;
if ((ret = vp7_fade_frame(s, alpha, beta)) < 0) if ((ret = vp7_fade_frame(s, alpha, beta)) < 0)
...@@ -2375,7 +2375,7 @@ static av_always_inline int decode_mb_row_no_filter(AVCodecContext *avctx, void ...@@ -2375,7 +2375,7 @@ static av_always_inline int decode_mb_row_no_filter(AVCodecContext *avctx, void
curframe->tf.f->data[2] + 8 * mb_y * s->uvlinesize curframe->tf.f->data[2] + 8 * mb_y * s->uvlinesize
}; };
if (c->end <= c->buffer && c->bits >= 0) if (vpX_rac_is_end(c))
return AVERROR_INVALIDDATA; return AVERROR_INVALIDDATA;
if (mb_y == 0) if (mb_y == 0)
...@@ -2406,7 +2406,7 @@ static av_always_inline int decode_mb_row_no_filter(AVCodecContext *avctx, void ...@@ -2406,7 +2406,7 @@ static av_always_inline int decode_mb_row_no_filter(AVCodecContext *avctx, void
td->mv_bounds.mv_max.x = ((s->mb_width - 1) << 6) + MARGIN; td->mv_bounds.mv_max.x = ((s->mb_width - 1) << 6) + MARGIN;
for (mb_x = 0; mb_x < s->mb_width; mb_x++, mb_xy++, mb++) { for (mb_x = 0; mb_x < s->mb_width; mb_x++, mb_xy++, mb++) {
if (c->end <= c->buffer && c->bits >= 0) if (vpX_rac_is_end(c))
return AVERROR_INVALIDDATA; return AVERROR_INVALIDDATA;
// Wait for previous thread to read mb_x+2, and reach mb_y-1. // Wait for previous thread to read mb_x+2, and reach mb_y-1.
if (prev_td != td) { if (prev_td != td) {
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment