1 files changed, 32 insertions, 38 deletions

diff --git a/apps/codecs/libpcm/dvi_adpcm.c b/apps/codecs/libpcm/dvi_adpcm.c
index 6774b14473..2784b21786 100644
--- a/apps/codecs/libpcm/dvi_adpcm.c
+++ b/apps/codecs/libpcm/dvi_adpcm.c
@@ -67,7 +67,7 @@ static struct pcm_pos *get_seek_pos(long seek_time,
     return &newpos;
 }
 
-static inline void decode_2bit(const uint8_t **inbuf,
+static inline void decode_2bit(const uint8_t **inbuf, size_t inbufsize,
                                int32_t **outbuf, int *outbufcount)
 {
     int ch;
@@ -75,7 +75,7 @@ static inline void decode_2bit(const uint8_t **inbuf,
     int32_t *pcmbuf;
     int samples;
 
-    samples = fmt->blockalign / (4 * fmt->channels) - 1;
+    samples = inbufsize / (4 * fmt->channels) - 1;
     *outbufcount += (samples << 4);
     while (samples-- > 0)
     {
@@ -99,7 +99,7 @@ static inline void decode_2bit(const uint8_t **inbuf,
     }
 }
 
-static inline void decode_3bit(const uint8_t **inbuf,
+static inline void decode_3bit(const uint8_t **inbuf, size_t inbufsize,
                                int32_t **outbuf, int *outbufcount)
 {
     const uint8_t *adpcmbuf;
@@ -109,7 +109,7 @@ static inline void decode_3bit(const uint8_t **inbuf,
     int32_t *pcmbuf;
     int samples;
 
-    samples = (fmt->blockalign - 4 * fmt->channels) / (12 * fmt->channels);
+    samples = (inbufsize - 4 * fmt->channels) / (12 * fmt->channels);
     *outbufcount += (samples << 5);
     while (samples--)
     {
@@ -157,7 +157,7 @@ static inline void decode_3bit(const uint8_t **inbuf,
     }
 }
 
-static inline void decode_4bit(const uint8_t **inbuf,
+static inline void decode_4bit(const uint8_t **inbuf, size_t inbufsize,
                                int32_t **outbuf, int *outbufcount)
 {
     int ch;
@@ -165,7 +165,7 @@ static inline void decode_4bit(const uint8_t **inbuf,
     int32_t *pcmbuf;
     int samples;
 
-    samples = fmt->blockalign / (4 * fmt->channels) - 1;
+    samples = inbufsize / (4 * fmt->channels) - 1;
     *outbufcount += (samples << 3);
     while (samples-- > 0)
     {
@@ -185,7 +185,7 @@ static inline void decode_4bit(const uint8_t **inbuf,
     }
 }
 
-static inline void decode_5bit(const uint8_t **inbuf,
+static inline void decode_5bit(const uint8_t **inbuf, size_t inbufsize,
                                int32_t **outbuf, int *outbufcount)
 {
     const uint8_t *adpcmbuf;
@@ -195,7 +195,7 @@ static inline void decode_5bit(const uint8_t **inbuf,
     int32_t *pcmbuf;
     int samples;
 
-    samples = (fmt->blockalign - 4 * fmt->channels) / (20 * fmt->channels);
+    samples = (inbufsize - 4 * fmt->channels) / (20 * fmt->channels);
     *outbufcount += (samples << 5);
     while (samples--)
     {
@@ -257,46 +257,40 @@ static int decode(const uint8_t *inbuf, size_t inbufsize,
                   int32_t *outbuf, int *outbufcount)
 {
     int ch;
-    unsigned int i;
     int32_t init_pcmdata[2];
     int8_t init_index[2];
-    unsigned int nblocks = fmt->chunksize / fmt->blockalign;
-
-    (void)inbufsize;
 
     *outbufcount = 0;
-    for (i = 0; i < nblocks; i++)
+    for (ch = 0; ch < fmt->channels; ch++)
     {
-        for (ch = 0; ch < fmt->channels; ch++)
+        init_pcmdata[ch] = inbuf[0] | (inbuf[1] << 8);
+        if (init_pcmdata[ch] > 32767)
+            init_pcmdata[ch] -= 65536;
+
+        init_index[ch] = inbuf[2];
+        if (init_index[ch] > 88 || init_index[ch] < 0)
         {
-            init_pcmdata[ch] = inbuf[0] | (inbuf[1] << 8);
-            if (init_pcmdata[ch] > 32767)
-                init_pcmdata[ch] -= 65536;
+            DEBUGF("CODEC_ERROR: dvi adpcm illegal step index=%d > 88\n",
+                                 init_index[ch]);
+            return CODEC_ERROR;
+        }
+        inbuf += 4;
 
-            init_index[ch] = inbuf[2];
-            if (init_index[ch] > 88 || init_index[ch] < 0)
-            {
-                DEBUGF("CODEC_ERROR: dvi adpcm illegal step index=%d > 88\n",
-                                     init_index[ch]);
-                return CODEC_ERROR;
-            }
-            inbuf += 4;
+        *outbuf++ = init_pcmdata[ch] << IMA_ADPCM_INC_DEPTH;
+    }
 
-            *outbuf++ = init_pcmdata[ch] << IMA_ADPCM_INC_DEPTH;
-        }
+    *outbufcount += 1;
+    set_decode_parameters(fmt->channels, init_pcmdata, init_index);
 
-        *outbufcount += 1;
-        set_decode_parameters(fmt->channels, init_pcmdata, init_index);
+    if (fmt->bitspersample == 4)
+        decode_4bit(&inbuf, inbufsize, &outbuf, outbufcount);
+    else if (fmt->bitspersample == 3)
+        decode_3bit(&inbuf, inbufsize, &outbuf, outbufcount);
+    else if (fmt->bitspersample == 5)
+        decode_5bit(&inbuf, inbufsize, &outbuf, outbufcount);
+    else /* fmt->bitspersample == 2 */
+        decode_2bit(&inbuf, inbufsize, &outbuf, outbufcount);
 
-        if (fmt->bitspersample == 4)
-            decode_4bit(&inbuf, &outbuf, outbufcount);
-        else if (fmt->bitspersample == 3)
-            decode_3bit(&inbuf, &outbuf, outbufcount);
-        else if (fmt->bitspersample == 5)
-            decode_5bit(&inbuf, &outbuf, outbufcount);
-        else /* fmt->bitspersample == 2 */
-            decode_2bit(&inbuf, &outbuf, outbufcount);
-    }
     return CODEC_OK;
 }