From: Michael Niedermayer Date: Mon, 14 Oct 2013 00:10:43 +0000 (+0200) Subject: Merge commit '1b20d0f581f01f2df601c9e68d0d321672d97af7' X-Git-Tag: android-x86-4.4-r1~618 X-Git-Url: http://git.osdn.net/view?a=commitdiff_plain;h=d794b7db1453ada81c49ae03e777250b5c2868c6;p=android-x86%2Fexternal-ffmpeg.git Merge commit '1b20d0f581f01f2df601c9e68d0d321672d97af7' * commit '1b20d0f581f01f2df601c9e68d0d321672d97af7': cavs: Return meaningful error values Conflicts: libavcodec/cavsdec.c Merged-by: Michael Niedermayer --- d794b7db1453ada81c49ae03e777250b5c2868c6 diff --cc libavcodec/cavsdec.c index 4a8d21b2ac,37eb118666..7a8501426a --- a/libavcodec/cavsdec.c +++ b/libavcodec/cavsdec.c @@@ -555,8 -550,7 +555,8 @@@ static int decode_residual_block(AVSCon const struct dec_2dvlc *r, int esc_golomb_order, int qp, uint8_t *dst, int stride) { - int i, esc_code, level, mask; - int i, level_code, esc_code, level, run, mask, ret; ++ int i, esc_code, level, mask, ret; + unsigned int level_code, run; int16_t level_buf[65]; uint8_t run_buf[65]; int16_t *block = h->block; @@@ -608,9 -602,9 +608,9 @@@ static inline int decode_residual_inter /* get coded block pattern */ int cbp = get_ue_golomb(&h->gb); - if (cbp > 63) { + if (cbp > 63U) { av_log(h->avctx, AV_LOG_ERROR, "illegal inter cbp\n"); - return -1; + return AVERROR_INVALIDDATA; } h->cbp = cbp_tab[cbp][1]; @@@ -679,9 -673,9 +679,9 @@@ static int decode_mb_i(AVSContext *h, i /* get coded block pattern */ if (h->cur.f->pict_type == AV_PICTURE_TYPE_I) cbp_code = get_ue_golomb(gb); - if (cbp_code > 63) { + if (cbp_code > 63U) { av_log(h->avctx, AV_LOG_ERROR, "illegal intra cbp\n"); - return -1; + return AVERROR_INVALIDDATA; } h->cbp = cbp_tab[cbp_code][0]; if (h->cbp && !h->qp_fixed)