[libav-devel] [PATCH 03/24] adpcm: convert adpcm_4xm to bytestream2.

[Ronald S. Bultje]

---
 libavcodec/adpcm.c |   14 +++++++++-----
 1 files changed, 9 insertions(+), 5 deletions(-)

diff --git a/libavcodec/adpcm.c b/libavcodec/adpcm.c
index 5870af4..e0aaffa 100644
--- a/libavcodec/adpcm.c
+++ b/libavcodec/adpcm.c
@@ -647,18 +647,22 @@ static int adpcm_decode_frame(AVCodecContext *avctx, void 
*data,
         break;
     case CODEC_ID_ADPCM_4XM:
         for (i = 0; i < avctx->channels; i++)
-            c->status[i].predictor= (int16_t)bytestream_get_le16(&src);
+            c->status[i].predictor = (int16_t)bytestream2_get_le16u(&gb);
 
         for (i = 0; i < avctx->channels; i++) {
-            c->status[i].step_index= (int16_t)bytestream_get_le16(&src);
-            c->status[i].step_index = av_clip(c->status[i].step_index, 0, 88);
+            c->status[i].step_index = (int16_t)bytestream2_get_le16u(&gb);
+            if (c->status[i].step_index > 88) {
+                av_log(avctx, AV_LOG_ERROR, "ERROR: step_index[%d] = %i\n",
+                       i, c->status[i].step_index);
+                return AVERROR_INVALIDDATA;
+            }
         }
 
         for (i = 0; i < avctx->channels; i++) {
             samples = (short *)c->frame.data[0] + i;
             cs = &c->status[i];
-            for (n = nb_samples >> 1; n > 0; n--, src++) {
-                uint8_t v = *src;
+            for (n = nb_samples >> 1; n > 0; n--) {
+                int v = bytestream2_get_byteu(&gb);
                 *samples = adpcm_ima_expand_nibble(cs, v & 0x0F, 4);
                 samples += avctx->channels;
                 *samples = adpcm_ima_expand_nibble(cs, v >> 4  , 4);
-- 
1.7.2.1

_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel